Re: [Gluster-devel] Possible leaks and so on

[Vijay Bellur]

Hello Patrick,

Thanks for bringing this to our notice.

Some of the issues reported are valid and they will be addressed through 
this bug:

http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=748

Thanks,
Vijay

Patrick Matthäi wrote:
> Hello,
>
> I have checked glusterfs 3.0.3 with cppcheck and it gave me the 
> following errors (which must not be true at all):
>
> [./libglusterfs/src/dict.c:179]: (error) Memory leak: newdata
> [./libglusterfs/src/dict.c:251]: (error) Memory leak: pair
> [./libglusterfs/src/fd.c:464]: (error) Possible null pointer 
> dereference: fd - otherwise it is redundant to check if fd is null at 
> line 466
> [./libglusterfs/src/y.tab.c:1743]: (error) Memory leak: node
> [./libglusterfs/src/y.tab.c:2067]: (error) Memory leak: cmd
> [./xlators/protocol/client/src/client-protocol.c:3434]: (error) 
> Possible null pointer dereference: iobuf - otherwise it is redundant 
> to check if iobuf is null at line 3505
> [./xlators/protocol/server/src/server-protocol.c:324]: (error) 
> Possible null pointer dereference: state - otherwise it is redundant 
> to check if state is null at line 344
>
>
> Could you please have a look at them?