linux-kernel - Re: [PATCH 00/17] Add memberof(), split some headers, and slightly simplify code

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]

Date:   Fri, 19 Nov 2021 16:52:22 +0100
From:   "Alejandro Colomar (man-pages)" <alx.manpages@...il.com>
To:     Andy Shevchenko <andriy.shevchenko@...ux.intel.com>
Cc:     "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 00/17] Add memberof(), split some headers, and slightly
 simplify code



On 11/19/21 16:49, Andy Shevchenko wrote:
>>
>> I think the problem was in
>> <linux/memberof.h> requiring NULL from <linux/stddef.h>
>> <linux/stddef.h> requiring memberof() from <linux/memberof.h>
>> or something like that.
> 
> There is no memberof.h in the kernel. Something is done wrongly on your series.

memberof.h was my first addition in this patch series.

Since I replaced (((T *)0)->m) by memberof(),
and that construction is used in <linux/stddef.h>
for example for sizeof_field(),
I included <linux/memberof.h> from <linux/stddef.h>.



-- 
Alejandro Colomar
Linux man-pages comaintainer; https://www.kernel.org/doc/man-pages/
http://www.alejandro-colomar.es/