netdev - Re: [PATCH V2 net-next] selftests/net: fix bugs in address and port initialization

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: <CAF=yD-Kcr2TkgUASgNPRANtWP9TYS_rzc_2Zxt5EORvvBUGvjw@mail.gmail.com>
Date:   Tue, 26 Dec 2017 14:35:41 -0500
From:   Willem de Bruijn <willemdebruijn.kernel@...il.com>
To:     Sowmini Varadhan <sowmini.varadhan@...cle.com>
Cc:     Network Development <netdev@...r.kernel.org>,
        Willem de Bruijn <willemb@...gle.com>,
        David Miller <davem@...emloft.net>
Subject: Re: [PATCH V2 net-next] selftests/net: fix bugs in address and port initialization

On Mon, Dec 25, 2017 at 5:43 PM, Sowmini Varadhan
<sowmini.varadhan@...cle.com> wrote:
> Address/port initialization should work correctly regardless
> of the order in which command line arguments are supplied,
> E.g, cfg_port should be used to connect to the remote host
> even if it is processed after -D, src/dst address initialization
> should not require that [-4|-6] be specified before
> the -S or -D args, receiver should be able to bind to *.<cfg_port>
>
> Achieve this by making sure that the address/port structures
> are initialized after all command line options are parsed.
>
> Store cfg_port in host-byte order, and use htons()
> to set up the sin_port/sin6_port before bind/connect,
> so that the network system calls get the correct values
> in network-byte order.
>
> Signed-off-by: Sowmini Varadhan <sowmini.varadhan@...cle.com>

Acked-by: Willem de Bruijn <willemb@...gle.com>