lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Fri, 03 Jan 2014 17:14:59 +0000 From: David Howells <dhowells@...hat.com> To: Paul Bolle <pebolle@...cali.nl> Cc: dhowells@...hat.com, akpm@...ux-foundation.org, Greg Kroah-Hartman <gregkh@...uxfoundation.org>, Tomas Winkler <tomas.winkler@...el.com>, linux-kernel@...r.kernel.org Subject: Re: [PATCH] mei: mei-amt-version: remove unneeded call of mei_deinit() Paul Bolle <pebolle@...cali.nl> wrote: > From: Paul Bolle <pebolle@...cali.nl> > > 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@...cali.nl> Builds without warning for me. Acked-by: David Howells <dhowells@...hat.com> -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists