diff options
| author | Ruben Devos <[email protected]> | 2025-06-01 17:18:55 +0000 |
|---|---|---|
| committer | Steve French <[email protected]> | 2025-06-01 20:12:11 +0000 |
| commit | f1e7a277a1736e12cc4bd6d93b8a5c439b8ca20c (patch) | |
| tree | 73fe278a302879e35348d2cb19896fe620fdeac6 | |
| parent | smb: client: Remove an unused function and variable (diff) | |
| download | kernel-f1e7a277a1736e12cc4bd6d93b8a5c439b8ca20c.tar.gz kernel-f1e7a277a1736e12cc4bd6d93b8a5c439b8ca20c.zip | |
smb: client: add NULL check in automount_fullpath
page is checked for null in __build_path_from_dentry_optional_prefix
when tcon->origin_fullpath is not set. However, the check is missing when
it is set.
Add a check to prevent a potential NULL pointer dereference.
Signed-off-by: Ruben Devos <[email protected]>
Cc: [email protected]
Signed-off-by: Steve French <[email protected]>
| -rw-r--r-- | fs/smb/client/namespace.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/fs/smb/client/namespace.c b/fs/smb/client/namespace.c index e3f9213131c4..a6655807c086 100644 --- a/fs/smb/client/namespace.c +++ b/fs/smb/client/namespace.c @@ -146,6 +146,9 @@ static char *automount_fullpath(struct dentry *dentry, void *page) } spin_unlock(&tcon->tc_lock); + if (unlikely(!page)) + return ERR_PTR(-ENOMEM); + s = dentry_path_raw(dentry, page, PATH_MAX); if (IS_ERR(s)) return s; |