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?
