samsung_sm8750/android_kernel_samsung_sm8750
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fs/ntfs3: handle hdr_first_de() return value

Browse Source 
[ Upstream commit af5cab0e5b6f8edb0be51a9f47f3f620e0b4fd70 ]

The hdr_first_de() function returns a pointer to a struct NTFS_DE. This
pointer may be NULL. To handle the NULL error effectively, it is important
to implement an error handler. This will help manage potential errors
consistently.

Additionally, error handling for the return value already exists at other
points where this function is called.

Found by Linux Verification Center (linuxtesting.org) with SVACE.

Fixes: 82cae269cf ("fs/ntfs3: Add initialization of super block")
Signed-off-by: Andrey Vatoropin <a.vatoropin@crpt.ru>
Signed-off-by: Konstantin Komarov <almaz.alexandrovich@paragon-software.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
Andrey Vatoropin
2025-03-18 13:42:18 +00:00
committed by Greg Kroah-Hartman
parent 94be75619e
commit 701340a25b
1 changed files with 8 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
fs/ntfs3/index.c
View File

@@ -2184,6 +2184,10 @@ static int indx_get_entry_to_replace(struct ntfs_index *indx,
e = hdr_first_de(&n->index->ihdr); e = hdr_first_de(&n->index->ihdr);
fnd_push(fnd, n, e); fnd_push(fnd, n, e);
if (!e) {
err = -EINVAL;
goto out;
}
if (!de_is_last(e)) { if (!de_is_last(e)) {
/* /*
@@ -2205,6 +2209,10 @@ static int indx_get_entry_to_replace(struct ntfs_index *indx,
n = fnd->nodes[level]; n = fnd->nodes[level];
te = hdr_first_de(&n->index->ihdr); te = hdr_first_de(&n->index->ihdr);
if (!te) {
err = -EINVAL;
goto out;
}
/* Copy the candidate entry into the replacement entry buffer. */ /* Copy the candidate entry into the replacement entry buffer. */
re = kmalloc(le16_to_cpu(te->size) + sizeof(u64), GFP_NOFS); re = kmalloc(le16_to_cpu(te->size) + sizeof(u64), GFP_NOFS);
if (!re) { if (!re) {