Merge pull request #1214 from jan-cerny/d68

matusmarhefka · web-flow · commit a1f3f682dc22 · 2018-10-03T10:41:23.000+02:00
Plug a memory leak
diff --git a/src/OVAL/probes/unix/file_probe.c b/src/OVAL/probes/unix/file_probe.c
@@ -393,6 +393,8 @@ static int file_cb(const char *prefix, const char *p, const char *f, void *ptr,
 		if (se_acl == NULL) {
 			probe_item_ent_add(item, "has_extended_acl", NULL, SEXP_number_newb(true));
 			probe_itement_setstatus(item, "has_extended_acl", 1, SYSCHAR_STATUS_DOES_NOT_EXIST);
+		} else {
+			SEXP_free(se_acl);
 		}
 
                 SEXP_free(se_grp_id);

Original file line number	Diff line number	Diff line change
`@@ -393,6 +393,8 @@ static int file_cb(const char prefix, const char p, const char f, void ptr,`
`393`	`393`	`if (se_acl == NULL) {`
`394`	`394`	`probe_item_ent_add(item, "has_extended_acl", NULL, SEXP_number_newb(true));`
`395`	`395`	`probe_itement_setstatus(item, "has_extended_acl", 1, SYSCHAR_STATUS_DOES_NOT_EXIST);`
	`396`	`+ } else {`
	`397`	`+ SEXP_free(se_acl);`
`396`	`398`	`}`
`397`	`399`
`398`	`400`	`SEXP_free(se_grp_id);`