seccomp: don't leak info on an error path

CID 154852 (#1 of 3): Resource leak (RESOURCE_LEAK) 32. leaked_storage: Variable info going out of scope leaks the storage it points to. Reported-by: coverity
avagin · Nov 26, 2015 · 759a1c8 · 759a1c8
1 parent d075176
commit 759a1c8
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/seccomp.c b/seccomp.c
@@ -97,8 +97,10 @@ static int collect_filter_for_pstree(struct pstree_item *item)
 
 		info->filter.filter.len = len * sizeof(struct sock_filter);
 		info->filter.filter.data = xmalloc(info->filter.filter.len);
-		if (!info->filter.filter.data)
+		if (!info->filter.filter.data) {
+			xfree(info);
 			goto out;
+		}
 
 		memcpy(info->filter.filter.data, buf, info->filter.filter.len);