linux-kernel - Re: linux-next: Tree for Sep 17 (net/ipv4/devinet.c)

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] [day] [month] [year] [list]

Message-ID: <15e457d1-e3a1-9cbc-0589-2033b67a24c5@infradead.org>
Date:   Fri, 18 Sep 2020 13:46:45 -0700
From:   Randy Dunlap <rdunlap@...radead.org>
To:     nicolas.dichtel@...nd.com, Stephen Rothwell <sfr@...b.auug.org.au>,
        Linux Next Mailing List <linux-next@...r.kernel.org>
Cc:     Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        David Miller <davem@...emloft.net>,
        Jakub Kicinski <kuba@...nel.org>
Subject: Re: linux-next: Tree for Sep 17 (net/ipv4/devinet.c)

On 9/18/20 1:25 AM, Nicolas Dichtel wrote:
> Le 17/09/2020 à 22:45, Randy Dunlap a écrit :
>> On 9/17/20 3:23 AM, Stephen Rothwell wrote:
>>> Hi all,
>>>
>>> Changes since 20200916:
>>>
>>
>>
>> Maybe some older versions of gcc just can't handle IS_ENABLED() inside an
>> if (expression) very well.
>>
>>> gcc --version
>> gcc (SUSE Linux) 7.5.0


> I tried with a (very old) gcc and I did not reproduce this error:
> $ gcc --version
> gcc (Debian 4.9.2-10+deb8u2) 4.9.2
> 

OK, no problem then.
I cannot reproduce it now either (I just tried 3-4 times).
We can forget about it. Sorry for the noise.

> 
>>
>> This patch:
>>
>> commit 9efd6a3cecdde984d67e63d17fe6af53c7c50968
>> Author: Nicolas Dichtel <nicolas.dichtel@...nd.com>
>> Date:   Wed May 13 15:58:43 2020 +0200
>>
>>     netns: enable to inherit devconf from current netns
>>
>> causes a build error:
>>
>> ../net/ipv4/devinet.c: In function ‘devinet_init_net’:
>> ../net/ipv4/devinet.c:2672:7: error: ‘sysctl_devconf_inherit_init_net’ undeclared (first use in this function); did you mean ‘devinet_init_net’?
>>        sysctl_devconf_inherit_init_net == 3) {
>>        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>>        devinet_init_net
>> ../net/ipv4/devinet.c:2672:7: note: each undeclared identifier is reported only
>>
>>

thanks.
-- 
~Randy