Skip to content

Commit b6c3d5c

Browse files
authored
Merge pull request #1220 from matusmarhefka/fix_SEXP_list_coverity_false_positive
OVAL/probes/SEAP/sexp-manip.c: fixed false positives reported by coverity
2 parents 38cf515 + 5f2c66d commit b6c3d5c

File tree

1 file changed

+8
-2
lines changed

1 file changed

+8
-2
lines changed

src/OVAL/probes/SEAP/sexp-manip.c

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -932,7 +932,10 @@ SEXP_t *SEXP_list_new (SEXP_t *memb, ...)
932932

933933
va_start(ap, memb);
934934
list = SEXP_new ();
935-
list = SEXP_list_new_rv(list, memb, ap);
935+
if (SEXP_list_new_rv(list, memb, ap) == NULL) {
936+
SEXP_free(list);
937+
list = NULL;
938+
}
936939
va_end(ap);
937940

938941
return (list);
@@ -1035,7 +1038,10 @@ SEXP_t *SEXP_list_rest (const SEXP_t *list)
10351038
SEXP_t *sexp;
10361039

10371040
sexp = SEXP_new();
1038-
sexp = SEXP_list_rest_r(sexp, list);
1041+
if (SEXP_list_rest_r(sexp, list) == NULL) {
1042+
SEXP_free(sexp);
1043+
sexp = NULL;
1044+
}
10391045

10401046
return (sexp);
10411047
}

0 commit comments

Comments
 (0)