From: Florian Weimer <fweimer@redhat.com>
To: pgowda cve <pgowda.cve@gmail.com>
Cc: libc-stable@sourceware.org, samuel.thibault@ens-lyon.org
Subject: Re: [2.33 and 2.34][PATCH] Simplify __opensock and fix race condition [BZ #28353]
Date: Thu, 03 Feb 2022 12:47:47 +0100 [thread overview]
Message-ID: <871r0kuq70.fsf@oldenburg.str.redhat.com> (raw)
In-Reply-To: <CALY4985v+jp1X4SSj2BsnXKYSKE3fihgQez5fM2mAYZTrZNALw@mail.gmail.com> (pgowda cve's message of "Thu, 3 Feb 2022 17:13:18 +0530")
* pgowda cve:
> (cherry picked from commit 5bf07e1b3a74232bfb8332275110be1a5da50f83)
>
> Please find attached the backported patch for glibc versions 2.33 and 2.34.
> Please let me know if it's okay.
I think we would have to backport all these commits:
commit 3d981795cd00cc9b73c3ee5087c308361acd62e5
Author: Florian Weimer <fweimer@redhat.com>
Date: Mon Nov 22 14:41:14 2021 +0100
socket: Do not use AF_NETLINK in __opensock
It is not possible to use interface ioctls with netlink sockets
on all Linux kernels.
Reviewed-by: Adhemerval Zanella <adhemerval.zanella@linaro.org>
commit 1d3decee997ba2fc24af81803299b2f4f3c47063
Author: Samuel Thibault <samuel.thibault@ens-lyon.org>
Date: Mon Oct 18 01:39:02 2021 +0200
hurd if_index: Explicitly use AF_INET for if index discovery
5bf07e1b3a74 ("Linux: Simplify __opensock and fix race condition [BZ #28353]")
made __opensock try NETLINK then UNIX then INET. On the Hurd, only INET
knows about network interfaces, so better actually specify that in
if_index.
commit 5bf07e1b3a74232bfb8332275110be1a5da50f83
Author: Florian Weimer <fweimer@redhat.com>
Date: Tue Sep 28 18:55:49 2021 +0200
Linux: Simplify __opensock and fix race condition [BZ #28353]
AF_NETLINK support is not quite optional on modern Linux systems
anymore, so it is likely that the first attempt will always succeed.
Consequently, there is no need to cache the result. Keep AF_UNIX
and the Internet address families as a fallback, for the rare case
that AF_NETLINK is missing. The other address families previously
probed are totally obsolete be now, so remove them.
Use this simplified version as the generic implementation, disabling
Netlink support as needed.
I can do that, no need to send patches.
Thanks,
Florian
prev parent reply other threads:[~2022-02-03 11:47 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-02-03 11:43 pgowda cve
2022-02-03 11:47 ` Florian Weimer [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=871r0kuq70.fsf@oldenburg.str.redhat.com \
--to=fweimer@redhat.com \
--cc=libc-stable@sourceware.org \
--cc=pgowda.cve@gmail.com \
--cc=samuel.thibault@ens-lyon.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).