c8h4/btrfs-progs
1
0
Fork 0
Code Issues Pull requests Actions Packages Projects Releases Wiki Activity

btrfs-progs: handle strndup() failure in ino_resolve()

Browse source 
Reported by 'gcc -fanalyzer':
cmds/subvolume.c:1078:39: warning: use of possibly-NULL ‘name’ where non-null expected [CWE-690] [-Wanalyzer-possible-null-argument]

The failure name duplication is not handled and can potentially lead to
a NULL dereference later. Handle the error properly and return template
error message.

Signed-off-by: David Sterba <dsterba@suse.com>
This commit is contained in:
David Sterba 2024-04-04 01:36:33 +02:00
parent c6d2d05ee6
commit 8767dda133
1 changed files with 4 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
cmds/subvolume.c
View file

@ -1138,6 +1138,10 @@ static char *ino_resolve(int fd, u64 ino, u64 *cache_dirid, char **cache_name)
name = (char *)(ref + 1);
name = strndup(name, namelen);
if (!name) {
error_msg(ERROR_MSG_MEMORY, NULL);
return NULL;
}
/* use our cached value */
if (dirid == *cache_dirid && *cache_name) {