Hi David, On Sun, 2012-04-15 at 12:27 +0000, David Binderman wrote: > Hello there, > > I recently tried out the static analysis tool cppcheck-1.54 > on the linux-3.4-rc2 source code. > > It said > > 1. > > [drivers/net/wireless/wl12xx/cmd.c:1252]: (warning) Using size of > pointer tmpl instead of size of its data. > > The source code is > > memset(tmpl, 0, sizeof(tmpl)); > > It might be the case that > > memset(tmpl, 0, sizeof(*tmpl)); > > is better code. > > 2. > > [drivers/net/wireless/wl12xx/cmd.c:1301]: (warning) Using size of > pointer hdr instead of size of its data. > > Duplicate. Thank you very much for the report. This was indeed wrong. I have fixed it and queued it for integration. -- Cheers, Luca. -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html