diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2021-06-15 09:39:52 +0200 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2021-06-15 16:42:26 +0200 |
| commit | a33d62662d275cee22888fa7760fe09d5b9cd1f9 (patch) | |
| tree | 4a961598f8378f3e807fe430d8d6ba9ed9bb1c7c /fs | |
| parent | Linux 5.13-rc6 (diff) | |
| download | linux-a33d62662d275cee22888fa7760fe09d5b9cd1f9.tar.xz linux-a33d62662d275cee22888fa7760fe09d5b9cd1f9.zip | |
afs: Fix an IS_ERR() vs NULL check
The proc_symlink() function returns NULL on error, it doesn't return
error pointers.
Fixes: 5b86d4ff5dce ("afs: Implement network namespacing")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: David Howells <dhowells@redhat.com>
cc: linux-afs@lists.infradead.org
Link: https://lore.kernel.org/r/YLjMRKX40pTrJvgf@mwanda/
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs')
| -rw-r--r-- | fs/afs/main.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/afs/main.c b/fs/afs/main.c index b2975256dadb..179004b15566 100644 --- a/fs/afs/main.c +++ b/fs/afs/main.c @@ -203,8 +203,8 @@ static int __init afs_init(void) goto error_fs; afs_proc_symlink = proc_symlink("fs/afs", NULL, "../self/net/afs"); - if (IS_ERR(afs_proc_symlink)) { - ret = PTR_ERR(afs_proc_symlink); + if (!afs_proc_symlink) { + ret = -ENOMEM; goto error_proc; } |