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
| ||
|
Date: Wed, 3 Mar 2010 09:59:58 +0100 From: Jiri Pirko <jpirko@...hat.com> To: Dan Carpenter <error27@...il.com> Cc: netdev@...r.kernel.org, Florian Fainelli <florian@...nwrt.org>, "David S. Miller" <davem@...emloft.net>, Ralf Baechle <ralf@...ux-mips.org>, Stefan Weil <weil@...l.berlios.de>, kernel-janitors@...r.kernel.org Subject: Re: [patch] cpmac: use after free Wed, Mar 03, 2010 at 09:46:10AM CET, error27@...il.com wrote: >The original code dereferenced "cpmac_mii" after calling >"mdiobus_free(cpmac_mii);" > >Signed-off-by: Dan Carpenter <error27@...il.com> >--- >Found by a static checker and not tested. Sorry. :/ > >diff --git a/drivers/net/cpmac.c b/drivers/net/cpmac.c >index b85c81f..9d48942 100644 >--- a/drivers/net/cpmac.c >+++ b/drivers/net/cpmac.c >@@ -1290,8 +1290,8 @@ void __devexit cpmac_exit(void) > { > platform_driver_unregister(&cpmac_driver); > mdiobus_unregister(cpmac_mii); >- mdiobus_free(cpmac_mii); > iounmap(cpmac_mii->priv); >+ mdiobus_free(cpmac_mii); > } > > module_init(cpmac_init); Looks good. Thanks. Reviewed-by: Jiri Pirko <jpirko@...hat.com> -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists