| 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: <1161859199.12781.7.camel@localhost.localdomain> Date: Thu, 26 Oct 2006 11:39:59 +0100 From: Alan Cox <alan@...rguk.ukuu.org.uk> To: Andrew Morton <akpm@...l.org> Cc: proski@....org, linux-kernel@...r.kernel.org Subject: Re: incorrect taint of ndiswrapper Ar Mer, 2006-10-25 am 20:59 -0700, ysgrifennodd Andrew Morton: > May be so. But this patch was supposed to print a helpful taint message to > draw our attention to the fact that ndis-wrapper was in use. The patch was > not intended to cause gpl'ed modules to stop loading The stopping loading is purely because it now uses _GPLONLY symbols, which is fine until the user wants to load a windows driver except for the old CIPE driver. Some assumptions broke somewhere along the way and the chain of events that was never forseen unfolded. > Now, if we do want to disallow gpl module loading after ndis-wrapper has > been used then fine The problem is we do the dynamic link at module load time. We would have to unlink the module if it tried to taint itself, which is clearly not what the end user needs to suffer. Having the taint function actually taint and printk + return a "Linked gplonly you can't" error seems the better solution. Really ndiswrapper shouldn't be using _GPLONLY symbols, that would actually make it useful to the binary driver afflicted again and more likely to be legal. - 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