Whamcloud - gitweb
LU-3069 build: fix 'integer handling' issues 88/5888/15
authorSebastien Buisson <sebastien.buisson@bull.net>
Fri, 29 Mar 2013 15:21:15 +0000 (16:21 +0100)
committerOleg Drokin <oleg.drokin@intel.com>
Sat, 10 Aug 2013 04:35:28 +0000 (04:35 +0000)
commit233576b564b713ad765d908629b5b290acdabc1e
treebe1989f190f1b215a000177178547341ce90e39b
parent124909c94a1322205484d5014152a3a77ff1e0fe
LU-3069 build: fix 'integer handling' issues

Fix 'program hangs' defects found by Coverity version 6.5.1:
Improper use of negative value (NEGATIVE_RETURNS)
Variable is passed to a parameter that cannot be negative.
Macro compares unsigned to 0 (NO_EFFECT)
This comparison is always true.
Operands don't affect result (CONSTANT_EXPRESSION_RESULT)
Test is always false regardless of the values of its operands.
Wrong operator used (CONSTANT_EXPRESSION_RESULT)
Test is always true regardless of the values of its operand.

Signed-off-by: Sebastien Buisson <sebastien.buisson@bull.net>
Signed-off-by: Dmitry Eremin <dmitry.eremin@intel.com>
Change-Id: I687a05aaffad13caaebe16edbc4f93ef7ca3413f
Reviewed-on: http://review.whamcloud.com/5888
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
lustre/ldlm/ldlm_lockd.c
lustre/llite/rw.c
lustre/llite/xattr.c
lustre/lod/lod_dev.c
lustre/mdt/mdt_handler.c
lustre/obdclass/llog_ioctl.c
lustre/osp/osp_dev.c
lustre/utils/liblustreapi.c