aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorZhiqiang Liu <liuzhiqiang26@huawei.com>2021-06-30 16:27:24 +0800
committerTheodore Ts'o <tytso@mit.edu>2021-07-15 23:59:21 -0400
commit29a61d8940b8a6a967a56c927d4703597f1d82e5 (patch)
treede4b3694771bb98014bc9b0d6869e01127071bda
parenta6aee13bc2b9557c25942a197e6d7a5ec7504fe8 (diff)
downloade2fsprogs-29a61d8940b8a6a967a56c927d4703597f1d82e5.tar.gz
ext2ed: fix potential NULL pointer dereference in dupstr()
In dupstr(), we should check return value of malloc(). Signed-off-by: Zhiqiang Liu <liuzhiqiang26@huawei.com> Signed-off-by: Wu Guanghao <wuguanghao3@huawei.com> Reviewed-by: Wu Bo <wubo40@huawei.com> Signed-off-by: Theodore Ts'o <tytso@mit.edu>
-rw-r--r--ext2ed/main.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/ext2ed/main.c b/ext2ed/main.c
index f7e7d7dfa..9d33a8e1a 100644
--- a/ext2ed/main.c
+++ b/ext2ed/main.c
@@ -524,6 +524,8 @@ char *dupstr (char *src)
char *ptr;
ptr=(char *) malloc (strlen (src)+1);
+ if (!ptr)
+ return NULL;
strcpy (ptr,src);
return (ptr);
}