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: Tue, 20 Oct 2009 13:21:24 +0300 From: Aaro Koskinen <aaro.koskinen@...ia.com> To: ext Samuel Ortiz <sameo@...ux.intel.com> CC: "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org> Subject: Re: [PATCH 1/2] mfd: twl4030-power: do not allow negative or zero length script Hello, Samuel Ortiz wrote: > On Wed, Oct 14, 2009 at 03:15:49PM +0300, Aaro Koskinen wrote: >> The script length cannot be negative. If the length is zero, return >> an error. >> >> The patch eliminates the following compiler warning: > Out of curiosity, which toolchain are you using ? I cross compile my mfd tree > for ARM, and I dont get this error... It seems I was accidentally using an older version of CodeSourcery toolchain (2007q3-51). I tried now with the recent one and warnings do not appear. I'm still not sure if they were totally bogus or not... >> drivers/mfd/twl4030-power.c: In function 'twl4030_power_init': > twl4030_power_init ? Isnt it twl4030_write_script ? You are right, it seems the compiler does some inlining and reports a wrong function name. A. > Cheers, > Samuel. > > >> drivers/mfd/twl4030-power.c:151: warning: 'err' may be used uninitialized in this function >> drivers/mfd/twl4030-power.c:151: note: 'err' was declared here >> >> Signed-off-by: Aaro Koskinen <aaro.koskinen@...ia.com> >> --- >> drivers/mfd/twl4030-power.c | 4 ++-- >> 1 files changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/drivers/mfd/twl4030-power.c b/drivers/mfd/twl4030-power.c >> index d423e0c..eae6cda 100644 >> --- a/drivers/mfd/twl4030-power.c >> +++ b/drivers/mfd/twl4030-power.c >> @@ -146,9 +146,9 @@ out: >> } >> >> static int __init twl4030_write_script(u8 address, struct twl4030_ins *script, >> - int len) >> + unsigned len) >> { >> - int err; >> + int err = EINVAL; >> >> for (; len; len--, address++, script++) { >> if (len == 1) { >> -- >> 1.6.0.4 >> > -- 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