Skip to content

Commit 7c78b84

Browse files
committed
Remove useless warnings
These warnings give zero value to the user, because they don't specify what went wrong. Moreover, the severity of the message depends on the parameters of the macro. For example, in environemntvariable58 probe the macro is used in a way that "Entity has no value" is a valid and expected situation, because if the pid element is set to xsi:nil the PID should be set to the PID of the calling process. Resolves: RHBZ#1764139
1 parent 527d4f7 commit 7c78b84

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

src/OVAL/probes/public/probe-api.h

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -488,11 +488,9 @@ OSCAP_API SEXP_t *probe_item_create(oval_subtype_t item_subtype, probe_elmatr_t
488488
SEXP_t *___r; \
489489
\
490490
if ((___r = probe_ent_getval(ent)) == NULL) { \
491-
dW("Entity has no value!"); \
492491
invalid_exp \
493492
} else { \
494493
if (!SEXP_stringp(___r)) { \
495-
dE("Invalid type"); \
496494
SEXP_free(___r); \
497495
invalid_exp \
498496
} \
@@ -511,11 +509,9 @@ OSCAP_API SEXP_t *probe_item_create(oval_subtype_t item_subtype, probe_elmatr_t
511509
SEXP_t *___r; \
512510
\
513511
if ((___r = probe_ent_getval(ent)) == NULL) { \
514-
dW("Entity has no value!"); \
515512
nil_exp; \
516513
} else { \
517514
if (!SEXP_numberp(___r)) { \
518-
dE("Invalid type"); \
519515
SEXP_free(___r); \
520516
invalid_exp; \
521517
} else { \

0 commit comments

Comments
 (0)