| 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, 17 Aug 2009 16:54:38 -0700 (PDT) From: Roland McGrath <roland@...hat.com> To: James Bottomley <James.Bottomley@...senPartnership.com> Cc: Helge Deller <deller@....de>, linux-parisc <linux-parisc@...r.kernel.org>, linux-kernel <linux-kernel@...r.kernel.org> Subject: Re: kernel segv with 2.6.31-rc6 ? > The root cause is a duplicate section name (.text); is this legal? It's a complicated subject, but I don't think they should ever actually exist in a .ko. The linker script for making .ko's should take care of that. But if modules do some crazy things like using section groups (e.g. for COMDAT, i.e. compile C++ code into modules) then I'm not sure how easy it is to get ld to combine things ideally as we'd like. Suffice it to say that any occurrence of this merits further investigation. It is certainly a red flag at first blush (as it were). > However, there's a problem with commit > 6d76013381ed28979cd122eb4b249a88b5e384fa in that if you fail to allocate > a mod->sect_attrs (in this case it's null because of the duplication), > it still gets used without checking in add_notes_attrs() > > This should fix it Yes, good catch. Sorry about that complete failure of defensive programming. Acked-by: Roland McGrath <roland@...hat.com> Thanks, Roland -- 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