Whamcloud - gitweb
LU-1907 build: avoid function resolution mistakes by Coverity
authorSebastien Buisson <sebastien.buisson@bull.net>
Wed, 12 Sep 2012 12:49:17 +0000 (14:49 +0200)
committerOleg Drokin <green@whamcloud.com>
Thu, 20 Sep 2012 04:05:02 +0000 (00:05 -0400)
commit6be01be8325fe9f2e579835757b2dd9922bdd693
treeb45791676bb3e332660a67d186a36003d86938e3
parentfa32b478599bbdfe6d51e41d8715665632d011b3
LU-1907 build: avoid function resolution mistakes by Coverity

Remove function resolution mistakes made by Coverity version
6.0.3.
The .hs_keycmp function is incorrectly resolved by Coverity,
making the variable passed as its first 'key' parameter to
be considered of the wrong type. Data structures being of
different sizes, it ends up with this Coverity defect:
Out-of-bounds access (OVERRUN_STATIC)
Overrunning static array by passing it as an argument to a
function which indexes it at a higher byte position.

Signed-off-by: Sebastien Buisson <sebastien.buisson@bull.net>
Change-Id: Ia64d90f2faed17981cc4b4eaa960a797ab056daf
Reviewed-on: http://review.whamcloud.com/3955
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Keith Mannthey <keith@whamcloud.com>
Reviewed-by: Bob Glossman <bob.glossman@intel.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
lustre/ldlm/ldlm_lock.c
lustre/ldlm/ldlm_lockd.c
lustre/ldlm/ldlm_request.c
lustre/lov/lov_obd.c
lustre/lov/lov_object.c
lustre/lov/lov_pack.c
lustre/lov/lov_qos.c
lustre/mdt/mdt_handler.c
lustre/mdt/mdt_open.c
lustre/obdecho/echo_client.c
lustre/ptlrpc/connection.c