linux-kernel - Re: [PATCH] Fixed bug in Wireless Extension Code (CONFIG_WEXT

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20091123154225.GA26103@MSServer>
Date:	Mon, 23 Nov 2009 10:42:25 -0500
From:	Michael Spradling <mike@...radling.com>
To:	Johannes Berg <johannes@...solutions.net>
Cc:	mike@...radling.com, davem@...emloft.net,
	hugh.dickins@...ali.co.uk, linville@...driver.com,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH] Fixed bug in Wireless Extension Code
 (CONFIG_WEXT_CORE).

I don't think its a Kconfig issue, see notes below.  However,
I have included my config file just in case I may be wrong.

Thank you
On Mon, Nov 23, 2009 at 10:45:41AM +0100, Johannes Berg wrote:
> Hi,
> 
> Please send wireless patches to the correct list (linux-wireless) in the
> future.
> 
> On Sun, 2009-11-22 at 19:36 -0500, mike@...radling.com wrote:
> 
> > When CONFIG_WEXT_CORE is not enabled and CONFIG_WEXT_PRIV is enabled
> > there currently is a few build errors.
> 
> This should never happen unless there's as bug in Kconfig, please
> provide the config that creates this problem so we can fix that issue
> rather than work around it. When PRIV is enabled but not CORE, this
> patch may fix compilation but it doesn't fix that the configuration
> makes no sense at all.
> 
> johannes
> 

I at first assumed there was a bug in Kconfig and looked into this.
However, I found this solution is not a hack.  PRIV does not depend
on CORE.  Some header files in "PRIV CODE" check and see if CORE is
defined.  If it is not defined it appears to impliment dummy functions
that return an error code.  All but one of these dummy functions are
implimented.  This patch just implements one that was forgoten.

Michael Spradling

View attachment "config" of type "text/plain" (35438 bytes)