summaryrefslogtreecommitdiff
path: root/scripts/rt-tester
diff options
context:
space:
mode:
authorPaul Bolle <pebolle@tiscali.nl>2014-01-03 14:44:19 -0800
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2014-01-08 15:20:20 -0800
commitaa98a28563d2abcd904d07fd5e54553290c85f91 (patch)
treee8df39fe22fd43abf2eb54647b1bccee9fa186a0 /scripts/rt-tester
parentd276b6c54d47a427e4cffd7c84f3b631a1edf98e (diff)
Documentation/misc-devices/mei/mei-amt-version.c: remove unneeded call of mei_deinit()
Building mei-amt-version.o triggers a GCC warning: Documentation/misc-devices/mei/mei-amt-version.c: In function 'main': Documentation/misc-devices/mei/mei-amt-version.c:103:5: warning: 'acmd.fd' is used uninitialized in this function [-Wuninitialized] if (cl->fd != -1) ^ Documentation/misc-devices/mei/mei-amt-version.c:443:21: note: 'acmd.fd' was declared here struct amt_host_if acmd; ^ GCC is correct. See, the call chain that GCC detects must be main() amt_host_if_init() mei_init() mei_deinit() But when we enter mei_deinit() struct amt_host_if acmd is still unitialized. That makes the test for (effectively) amt_host_if->mei_cl->fd bogus. But it turns out that call of mei_deinit() isn't needed at all. All of the members of mei_cl will be set later in mei_init() and none will be used before they are set. So we can simply drop this call of mei_deinit(). Signed-off-by: Paul Bolle <pebolle@tiscali.nl> Cc: Tomas Winkler <tomas.winkler@intel.com> Reported-by: David Howells <dhowells@redhat.com> Acked-by: David Howells <dhowells@redhat.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'scripts/rt-tester')
0 files changed, 0 insertions, 0 deletions