vhost: fix false-positive warning from clang 5
authorBruce Richardson <bruce.richardson@intel.com>
Wed, 11 Oct 2017 11:28:17 +0000 (12:28 +0100)
committerThomas Monjalon <thomas@monjalon.net>
Wed, 11 Oct 2017 11:56:34 +0000 (13:56 +0200)
commitd65b3b1668f2037745407c3909c43d85a18692a6
tree816b559f5844e637d7288d5f8c42f118261c464a
parentfc8030eb8fe996534b5c6c873ea7f56ad407415e
vhost: fix false-positive warning from clang 5

When compiling with clang extra warning flags, such as used by default with
meson, a warning is given in iotlb.c:

lib/librte_vhost/iotlb.c:318:6: warning:
variable 'socket' is used uninitialized whenever
'if' condition is false [-Wsometimes-uninitialized]

This is a false positive, as the socket value will be initialized by the
call to get_mempolicy in the case where the NUMA build-time flag is set,
and in cases where it is not set, "if (ret)" will always be true as ret is
initialized to -1 and never changed.

However, this is not immediately obvious, and is perhaps a little fragile,
as it will break if other code using ret is subsequently added above the
call to get_mempolicy by someone unaware of this subtle dependency.
Therefore, we can fix the warning and making the code more robust by
explicitly initializing socket to zero, and moving the extra condition
check on the return from get_mempolicy() into the #ifdef

Fixes: d012d1f293f4 ("vhost: add IOTLB helper functions")

Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
lib/librte_vhost/iotlb.c