linux-kernel - Re: [PATCH] iwlagn: fix *_UCODE_API

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: <1307889852.13397.128.camel@wwguy-huron>
Date:	Sun, 12 Jun 2011 07:44:12 -0700
From:	"Guy, Wey-Yi" <wey-yi.w.guy@...el.com>
To:	Evgeni Golov <sargentd@...-welt.net>
Cc:	"ilw@...ux.intel.com" <ilw@...ux.intel.com>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] iwlagn: fix *_UCODE_API_MAX output in the firmware
 field

On Sun, 2011-06-12 at 05:34 -0700, Evgeni Golov wrote:
> Currently (3.0-rc2), modinfo iwlagn shows:
>     firmware:       iwlwifi-5150-IWL5150_UCODE_API_MAX.ucode
>     firmware:       iwlwifi-5000-IWL5000_UCODE_API_MAX.ucode
>     firmware:       iwlwifi-6000g2b-IWL6000G2_UCODE_API_MAX.ucode
>     firmware:       iwlwifi-6000g2a-IWL6000G2_UCODE_API_MAX.ucode
>     firmware:       iwlwifi-6050-IWL6050_UCODE_API_MAX.ucode
>     firmware:       iwlwifi-6000-IWL6000_UCODE_API_MAX.ucode
>     firmware:       iwlwifi-100-IWL100_UCODE_API_MAX.ucode
>     firmware:       iwlwifi-1000-IWL1000_UCODE_API_MAX.ucode
>     firmware:       iwlwifi-105-IWL105_UCODE_API_MAX.ucode
>     firmware:       iwlwifi-2030-IWL2030_UCODE_API_MAX.ucode
>     firmware:       iwlwifi-2000-IWL2000_UCODE_API_MAX.ucode
> 
> which is obviously wrong, the user should not see the *_UCODE_API_MAX
> macros but the actual ucode API versions here.
> 
> The problem are the
>     #define *_MODULE_FIRMWARE(api) *_FW_PRE #api ".ucode"
> which do not expand api correctly (because this is a macro itself).
> 
> Fixed by using __stringify() from linux/stringify.h.
> 
> Further information about macro stringification can be found here:
>     http://gcc.gnu.org/onlinedocs/cpp/Stringification.html
> 
> Signed-off-by: Evgeni Golov <sargentd@...-welt.net>
Signed-off-by: Wey-Yi Guy <wey-yi.w.guy@...el.com>

Good catch, thank you very much

Wey




--
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/