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: Mon, 18 Sep 2006 15:54:17 -0700 From: "Om Narasimhan" <om.turyx@...il.com> To: "Matthew Wilcox" <matthew@....cx> Cc: linux-pcmcia@...ts.infradead.org, linux-kernel@...r.kernel.org, kernel-janitors@...ts.osdl.org Subject: Re: [KJ] potential crash fix : drivers/pcmcia/au1000_generic On 9/18/06, Matthew Wilcox <matthew@....cx> wrote: > On Sun, Sep 17, 2006 at 05:54:17PM -0700, Om Narasimhan wrote: > > Tested by compiling. > > > > I have not subscribed to pcmcia list. Please cc me any comments. > > You also haven't described what the 'potential crach' is that you're > fixing. Okay, I should have documented that. Here is the explanation. The previous code did something like, if (error) goto out_err; .... do { struct au1000_pcmcia_socket *skt = PCMCIA_SOCKET(i); del_timer_sync(&skt->poll_timer); pcmcia_unregister_socket(&skt->socket); out_err: flush_scheduled_work(); ops->hw_shutdown(skt); i--; } while (i > 0) ..... 1. On the error path, skt would not contain a valid value for the first iteration (skt is masked by uninitialized automatic skt) 2. does not do hw_shutdown() for 0th element of PCMCIA_SOCKET Does it sound good? Regards, Om. - 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