| 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 linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <20100924185008.GA20110@ericsson.com> Date: Fri, 24 Sep 2010 11:50:08 -0700 From: Guenter Roeck <guenter.roeck@...csson.com> To: Jan Beulich <jbeulich@...ell.com> CC: fenghua.yu@...el.com, khali@...ux-fr.org, gong.chen@...ux.intel.com, lm-sensors@...sensors.org, linux-kernel@...r.kernel.org Subject: Re: x86/hwmon: fix module init for hotplug-but-no-device-found case On Mon, Sep 13, 2010 at 10:05:51AM -0000, Jan Beulich wrote: > In commit 0dca94baeab4a1a514841b0a4c8e3a51dfb4d5ae the call to > platform_driver_unregister() was made conditional upon !HOTPLUG_CPU, > but the return value from coretemp_init() was left to indicate an > error. This isn't correct, as the negative return value indicates to > the module loader that initialization failed, which isn't intended > here and results in dangling pointers. > > Signed-off-by: Jan Beulich <jbeulich@...ell.com> > Cc: Chen Gong <gong.chen@...ux.intel.com> > Looks like my previous mail got lost or stuck in our e-mail system. Trying again. Sorry if this message is received twice. I have not seen an Acked-by for this set of patches, nor a Nack. Since (most of) the patches fix real bugs, I applied the ones which do into my staging tree, and the others into my -next tree. Guenter -- 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