Merge pull request OpenSCAP#2218 from evgenyz/backport-593e515

jan-cerny · web-flow · commit 061e9ca3f88f · 2025-04-10T10:34:11.000+02:00
Backport of OpenSCAP#2168
diff --git a/src/OVAL/probes/probe/worker.c b/src/OVAL/probes/probe/worker.c
@@ -985,7 +985,7 @@ static SEXP_t *probe_set_eval(probe_t *probe, SEXP_t *set, size_t depth)
 
 static void _add_blocked_paths(struct oscap_list *bpaths)
 {
-	char *envar = getenv("OSCAP_PROBE_IGNORE_PATHS");
+	char *envar = oscap_strdup(getenv("OSCAP_PROBE_IGNORE_PATHS"));
 	if (envar == NULL) {
 		return;
 	}
@@ -996,6 +996,7 @@ static void _add_blocked_paths(struct oscap_list *bpaths)
 	for (int i = 0; paths[i]; ++i) {
 		oscap_list_add(bpaths, strdup(paths[i]));
 	}
+	free(envar);
 	free(paths);
 #endif
 }

Original file line number	Diff line number	Diff line change
`@@ -985,7 +985,7 @@ static SEXP_t probe_set_eval(probe_t probe, SEXP_t *set, size_t depth)`
`985`	`985`
`986`	`986`	`static void _add_blocked_paths(struct oscap_list *bpaths)`
`987`	`987`	`{`
`988`		`- char *envar = getenv("OSCAP_PROBE_IGNORE_PATHS");`
	`988`	`+ char *envar = oscap_strdup(getenv("OSCAP_PROBE_IGNORE_PATHS"));`
`989`	`989`	`if (envar == NULL) {`
`990`	`990`	`return;`
`991`	`991`	`}`
`@@ -996,6 +996,7 @@ static void _add_blocked_paths(struct oscap_list *bpaths)`
`996`	`996`	`for (int i = 0; paths[i]; ++i) {`
`997`	`997`	`oscap_list_add(bpaths, strdup(paths[i]));`
`998`	`998`	`}`
	`999`	`+ free(envar);`
`999`	`1000`	`free(paths);`
`1000`	`1001`	`#endif`
`1001`	`1002`	`}`