linux-kernel - Re: [PATCH net] socket: fix compat SO_RCVTIMEO_NEW/SO_SNDTIMEO

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: <CABeXuvqa14ybq9d_BniwuURwjM+PfWTTbkeVXOF9SwbLVvok+Q@mail.gmail.com>
Date:   Tue, 16 Apr 2019 16:04:44 -0700
From:   Deepa Dinamani <deepa.kernel@...il.com>
To:     Arnd Bergmann <arnd@...db.de>
Cc:     "David S. Miller" <davem@...emloft.net>,
        Eric Dumazet <edumazet@...gle.com>,
        Willem de Bruijn <willemb@...gle.com>,
        Yafang Shao <laoar.shao@...il.com>,
        Linux Network Devel Mailing List <netdev@...r.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH net] socket: fix compat SO_RCVTIMEO_NEW/SO_SNDTIMEO_NEW

Acked-by: Deepa Dinamani <deepa.kernel@...il.com>

Thanks,
-Deepa

On Tue, Apr 16, 2019 at 1:31 PM Arnd Bergmann <arnd@...db.de> wrote:
>
> It looks like the new socket options only work correctly
> for native execution, but in case of compat mode fall back
> to the old behavior as we ignore the 'old_timeval' flag.
>
> Rework so we treat SO_RCVTIMEO_NEW/SO_SNDTIMEO_NEW the
> same way in compat and native 32-bit mode.
>
> Cc: Deepa Dinamani <deepa.kernel@...il.com>
> Fixes: a9beb86ae6e5 ("sock: Add SO_RCVTIMEO_NEW and SO_SNDTIMEO_NEW")
> Signed-off-by: Arnd Bergmann <arnd@...db.de>
> ---
>
>  net/core/sock.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/net/core/sock.c b/net/core/sock.c
> index dc7c31d66200..1e418c0b3b73 100644
> --- a/net/core/sock.c
> +++ b/net/core/sock.c
> @@ -348,7 +348,7 @@ static int sock_get_timeout(long timeo, void *optval, bool old_timeval)
>                 tv.tv_usec = ((timeo % HZ) * USEC_PER_SEC) / HZ;
>         }
>
> -       if (in_compat_syscall() && !COMPAT_USE_64BIT_TIME) {
> +       if (old_timeval && in_compat_syscall() && !COMPAT_USE_64BIT_TIME) {
>                 struct old_timeval32 tv32 = { tv.tv_sec, tv.tv_usec };
>                 *(struct old_timeval32 *)optval = tv32;
>                 return sizeof(tv32);
> @@ -372,7 +372,7 @@ static int sock_set_timeout(long *timeo_p, char __user *optval, int optlen, bool
>  {
>         struct __kernel_sock_timeval tv;
>
> -       if (in_compat_syscall() && !COMPAT_USE_64BIT_TIME) {
> +       if (old_timeval && in_compat_syscall() && !COMPAT_USE_64BIT_TIME) {
>                 struct old_timeval32 tv32;
>
>                 if (optlen < sizeof(tv32))
> --
> 2.20.0
>