diff options
| author | Jeff Layton <jlayton@kernel.org> | 2025-02-06 21:12:13 +0300 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2025-04-20 11:18:29 +0300 |
| commit | e31957a819e60cf0bc9a49408765e6095fd3d046 (patch) | |
| tree | 30a2803da1690e1ce82b6e94dd78b6eae7fb5346 /tools/perf/scripts/python/libxed.py | |
| parent | f0ae5bc36655fb55ab6bee4ea977b1738061dd84 (diff) | |
| download | linux-e31957a819e60cf0bc9a49408765e6095fd3d046.tar.xz | |
nfsd: don't ignore the return code of svc_proc_register()
commit 930b64ca0c511521f0abdd1d57ce52b2a6e3476b upstream.
Currently, nfsd_proc_stat_init() ignores the return value of
svc_proc_register(). If the procfile creation fails, then the kernel
will WARN when it tries to remove the entry later.
Fix nfsd_proc_stat_init() to return the same type of pointer as
svc_proc_register(), and fix up nfsd_net_init() to check that and fail
the nfsd_net construction if it occurs.
svc_proc_register() can fail if the dentry can't be allocated, or if an
identical dentry already exists. The second case is pretty unlikely in
the nfsd_net construction codepath, so if this happens, return -ENOMEM.
Reported-by: syzbot+e34ad04f27991521104c@syzkaller.appspotmail.com
Closes: https://lore.kernel.org/linux-nfs/67a47501.050a0220.19061f.05f9.GAE@google.com/
Cc: stable@vger.kernel.org # v6.9
Signed-off-by: Jeff Layton <jlayton@kernel.org>
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'tools/perf/scripts/python/libxed.py')
0 files changed, 0 insertions, 0 deletions