diff options
| author |   Christian Brauner <brauner@kernel.org> | 2025-04-16 22:05:40 +0200 |
|---|---|---|
| committer | Christian Brauner <brauner@kernel.org> | 2025-04-17 09:43:26 +0200 |
| commit | b590c928cca7bdc7fd580d52e42bfdc3ac5eeacb (patch) | |
| tree | 31fbccae82a427bade167971a887303817c3dc96 /net/core/sock.c | |
| parent | release_task: kill the no longer needed get/put_pid(thread_pid) (diff) | |
| download | wireguard-linux-b590c928cca7bdc7fd580d52e42bfdc3ac5eeacb.tar.xz wireguard-linux-b590c928cca7bdc7fd580d52e42bfdc3ac5eeacb.zip | |
net, pidfd: report EINVAL for ESRCH
dbus-broker relies -EINVAL being returned to indicate ESRCH in [1].
This causes issues for some workloads as reported in [2].
Paper over it until this is fixed in userspace.
Link: https://lore.kernel.org/20250416-gegriffen-tiefbau-70cfecb80ac8@brauner
Link: https://github.com/bus1/dbus-broker/blob/5d34d91b138fc802a016aa68c093eb81ea31139c/src/util/sockopt.c#L241 [1]
Link: https://lore.kernel.org/20250415223454.GA1852104@ax162 [2]
Tested-by: Nathan Chancellor <nathan@kernel.org>
Signed-off-by: Christian Brauner <brauner@kernel.org>
Diffstat (limited to 'net/core/sock.c')
| -rw-r--r-- | net/core/sock.c | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/net/core/sock.c b/net/core/sock.c index f67a3c5b0988..b969d2210656 100644 --- a/net/core/sock.c +++ b/net/core/sock.c @@ -1893,8 +1893,16 @@ int sk_getsockopt(struct sock *sk, int level, int optname, pidfd = pidfd_prepare(peer_pid, 0, &pidfd_file); put_pid(peer_pid); - if (pidfd < 0) + if (pidfd < 0) { + /* + * dbus-broker relies on -EINVAL being returned + * to indicate ESRCH. Paper over it until this + * is fixed in userspace. + */ + if (pidfd == -ESRCH) + pidfd = -EINVAL; return pidfd; + } if (copy_to_sockptr(optval, &pidfd, len) || copy_to_sockptr(optlen, &len, sizeof(int))) { |