53 lines
2.5 KiB
Diff
53 lines
2.5 KiB
Diff
|
From 378ef5e438a2f5af7a50374d2bd23bdd3403201f Mon Sep 17 00:00:00 2001
|
||
|
|
From: Evgeny Kolesnikov <ekolesni@redhat.com>
|
||
|
|
Date: Tue, 4 May 2021 08:41:06 +0200
|
||
|
|
Subject: [PATCH] Fix covscan-reported issues in yamlfilecontent probe and
|
||
|
|
schematron
|
||
|
|
|
||
|
|
Error: FORWARD_NULL (CWE-476): [#def1]
|
||
|
|
/OVAL/probes/independent/yamlfilecontent_probe.c:392: var_compare_op: Comparing "yaml_file" to null implies that "yaml_file" might be null.
|
||
|
|
/OVAL/probes/independent/yamlfilecontent_probe.c:417: var_deref_model: Passing null pointer "yaml_file" to "fclose", which dereferences it.
|
||
|
|
# 416| cleanup:
|
||
|
|
# 417|-> fclose(yaml_file);
|
||
|
|
# 418| yaml_parser_delete(&parser);
|
||
|
|
|
||
|
|
Error: RESOURCE_LEAK (CWE-772): [#def2] [important]
|
||
|
|
/source/schematron.c:549: alloc_fn: Storage is returned from allocation function "xmlXPathNodeEval".
|
||
|
|
/source/schematron.c:549: var_assign: Assigning: "component_refs" = storage returned from "xmlXPathNodeEval(data_stream_node, (xmlChar *)"ds:checklists/ds:component-ref", context)".
|
||
|
|
/source/schematron.c:551: leaked_storage: Variable "component_refs" going out of scope leaks the storage it points to.
|
||
|
|
# 550| if (component_refs == NULL || component_refs->nodesetval == NULL) {
|
||
|
|
# 551|-> return res;
|
||
|
|
# 552| }
|
||
|
|
---
|
||
|
|
src/OVAL/probes/independent/yamlfilecontent_probe.c | 3 ++-
|
||
|
|
src/source/schematron.c | 2 ++
|
||
|
|
2 files changed, 4 insertions(+), 1 deletion(-)
|
||
|
|
|
||
|
|
diff --git a/src/OVAL/probes/independent/yamlfilecontent_probe.c b/src/OVAL/probes/independent/yamlfilecontent_probe.c
|
||
|
|
index ed5ce0d68..62a8f4ff2 100644
|
||
|
|
--- a/src/OVAL/probes/independent/yamlfilecontent_probe.c
|
||
|
|
+++ b/src/OVAL/probes/independent/yamlfilecontent_probe.c
|
||
|
|
@@ -414,7 +414,8 @@ static int process_yaml_file(const char *prefix, const char *path, const char *f
|
||
|
|
}
|
||
|
|
|
||
|
|
cleanup:
|
||
|
|
- fclose(yaml_file);
|
||
|
|
+ if (yaml_file != NULL)
|
||
|
|
+ fclose(yaml_file);
|
||
|
|
yaml_parser_delete(&parser);
|
||
|
|
free(filepath_with_prefix);
|
||
|
|
free(filepath);
|
||
|
|
diff --git a/src/source/schematron.c b/src/source/schematron.c
|
||
|
|
index 6cb22658b..c32d5aed6 100644
|
||
|
|
--- a/src/source/schematron.c
|
||
|
|
+++ b/src/source/schematron.c
|
||
|
|
@@ -548,6 +548,8 @@ static bool _req_src_346_1_sub1(xmlNodePtr data_stream_node, xmlXPathContextPtr
|
||
|
|
/* every $m in ds:checklists/ds:component-ref satisfies ... */
|
||
|
|
xmlXPathObjectPtr component_refs = xmlXPathNodeEval(data_stream_node, BAD_CAST "ds:checklists/ds:component-ref", context);
|
||
|
|
if (component_refs == NULL || component_refs->nodesetval == NULL) {
|
||
|
|
+ if (component_refs != NULL)
|
||
|
|
+ xmlXPathFreeObject(component_refs);
|
||
|
|
return res;
|
||
|
|
}
|
||
|
|
for (int i = 0; i < component_refs->nodesetval->nodeNr; i++) {
|