netdev - Re: [v2,0/2] net: if_arp: use define instead of hard-coded value

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: <A1A93434-9496-4029-9A53-2D64D56CB846@oracle.com>
Date:   Tue, 23 Oct 2018 14:30:57 +0200
From:   Håkon Bugge <haakon.bugge@...cle.com>
To:     netdev@...r.kernel.org
Cc:     Stephen Hemminger <stephen@...workplumber.org>,
        "David S . Miller" <davem@...emloft.net>,
        Kate Stewart <kstewart@...uxfoundation.org>,
        Thomas Gleixner <tglx@...utronix.de>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        Philippe Ombredanne <pombredanne@...b.com>,
        linux-kernel@...r.kernel.org
Subject: Re: [v2,0/2] net: if_arp: use define instead of hard-coded value



> On 9 Oct 2018, at 16:27, Håkon Bugge <haakon.bugge@...cle.com> wrote:
> 
> Struct arpreq contains the name of the device. All other places in the
> kernel, the define IFNAMSIZ is used to designate its size. But in
> if_arp.h, a literal constant is used.
> 
> Fixed by explicitly including uapi/linu/if.h, where IFNAMSIZ is defined.
> 
> The series also fixes some incorrect indents.
> 
> 
> v1 -> v2:
>   * Include uapi/linux/if.h from if_arp.h
>   * Added Stephen's t-b
> 
> Håkon Bugge (2):
>  net: if_arp: Fix incorrect indents
>  net: if_arp: use define instead of hard-coded value
> 
> include/uapi/linux/if_arp.h | 19 ++++++++++---------
> 1 file changed, 10 insertions(+), 9 deletions(-)
> ---
> 2.14.3

Just a friendly reminder.


Thxs, Håkon


> 
> 
> From a36b99413ea5beca1539f4faf065f33c7c19e6a5 Mon Sep 17 00:00:00 2001
> From: =?UTF-8?q?H=C3=A5kon=20Bugge?= <Haakon.Bugge@...cle.com>
> Date: Fri, 21 Sep 2018 11:28:36 +0200
> Subject: [PATCH 0/2]
> MIME-Version: 1.0
> Content-Type: text/plain; charset=UTF-8
> Content-Transfer-Encoding: 8bit
> 
> Struct arpreq contains the name of the device. All other places in the
> kernel, the define IFNAMSIZ is used to designate its size. But in
> if_arp.h, a literal constant is used.
> 
> As it could be good reasons to use constants instead of the defines in
> include files under uapi, it seems to be OK to use the define here,
> without opening a can of worms in user-land.
> 
> This because if_arp.h includes netdevice.h, which also uses
> IFNAMSIZ. For the distros I have checked, this also holds true for the
> use-land side.
> 
> The series also fixes some incorrect indents.
> 
> Håkon Bugge (2):
>  net: if_arp: Fix incorrect indents
>  net: if_arp: use define instead of hard-coded value
> 
> include/uapi/linux/if_arp.h | 18 +++++++++---------
> 1 file changed, 9 insertions(+), 9 deletions(-)
> 
> --
> 2.14.3