diff options
| author | Paul Moore <paul@paul-moore.com> | 2026-05-29 11:24:37 -0400 |
|---|---|---|
| committer | Paul Moore <paul@paul-moore.com> | 2026-05-29 11:24:37 -0400 |
| commit | 033182baeab63ce96a6eb8aef1a6cd444fcf9519 (patch) | |
| tree | beaafe31367f6d973e93b9a68e01cfd062e8cb70 | |
| parent | 447e04178b776c82dad431f8af26086b5ce9a391 (diff) | |
selinux: revert use of __getname() in selinux_genfs_get_sid()
Revert commit 54067bacb49c ("selinux: hooks: use __getname() to allocate
path buffer") as it improperly assumed that PATH_MAX == PAGE_SIZE
everywhere. Moving away from __get_free_page() is still a good thing and
will be revisited in the future.
Cc: Mike Rapoport (Microsoft) <rppt@kernel.org>
Reported-by: Venkat Rao Bagalkote <venkat88@linux.ibm.com>
Signed-off-by: Paul Moore <paul@paul-moore.com>
| -rw-r--r-- | security/selinux/hooks.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/security/selinux/hooks.c b/security/selinux/hooks.c index 95d0fb20d84e..97801966bf32 100644 --- a/security/selinux/hooks.c +++ b/security/selinux/hooks.c @@ -1336,7 +1336,7 @@ static int selinux_genfs_get_sid(struct dentry *dentry, struct super_block *sb = dentry->d_sb; char *buffer, *path; - buffer = __getname(); + buffer = (char *)__get_free_page(GFP_KERNEL); if (!buffer) return -ENOMEM; @@ -1361,7 +1361,7 @@ static int selinux_genfs_get_sid(struct dentry *dentry, rc = 0; } } - __putname(buffer); + free_page((unsigned long)buffer); return rc; } |