diff options
author | Gui Hecheng <guihc.fnst@cn.fujitsu.com> | 2013-09-05 10:38:54 +0800 |
---|---|---|
committer | Chris Mason <chris.mason@fusionio.com> | 2013-10-16 08:20:42 -0400 |
commit | 0f2569b1266b0a6288013a81e4c93f4ac09fdb9b (patch) | |
tree | e36fb323a40982ad8571b00651e7e45096eb78a2 /btrfs-image.c | |
parent | 5991043ea4e825f002d58472554cee4475dffb0d (diff) |
btrfs-progs: free local variable buf upon unsuccessful returns
The variable "buf" passed into find_collision() as parameter "name"
should be freed on unsuccessful returns.
Signed-off-by: Gui Hecheng <guihc.fnst@cn.fujitsu.com>
Signed-off-by: David Sterba <dsterba@suse.cz>
Signed-off-by: Chris Mason <chris.mason@fusionio.com>
Diffstat (limited to 'btrfs-image.c')
-rw-r--r-- | btrfs-image.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/btrfs-image.c b/btrfs-image.c index ab229f3f..b2c34d75 100644 --- a/btrfs-image.c +++ b/btrfs-image.c @@ -284,6 +284,7 @@ static char *find_collision(struct metadump_struct *md, char *name, val = malloc(sizeof(struct name)); if (!val) { fprintf(stderr, "Couldn't sanitize name, enomem\n"); + free(name); return NULL; } @@ -295,6 +296,7 @@ static char *find_collision(struct metadump_struct *md, char *name, if (!val->sub) { fprintf(stderr, "Couldn't sanitize name, enomem\n"); free(val); + free(name); return NULL; } |