summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGui Hecheng <guihc.fnst@cn.fujitsu.com>2013-09-05 10:38:54 +0800
committerChris Mason <chris.mason@fusionio.com>2013-10-16 08:20:42 -0400
commit0f2569b1266b0a6288013a81e4c93f4ac09fdb9b (patch)
treee36fb323a40982ad8571b00651e7e45096eb78a2
parent5991043ea4e825f002d58472554cee4475dffb0d (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>
-rw-r--r--btrfs-image.c2
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;
}