On Sun, Mar 14, 2021 at 4:22 PM Nicolas Iooss <nicolas.iooss@xxxxxxx> wrote: > > All functions of the CIL compiler use cil_log or cil_tree_log to report > errors, but in two places which still uses printf. Replace these printf > invocation with cil_tree_log. > > Signed-off-by: Nicolas Iooss <nicolas.iooss@xxxxxxx> Acked-by: James Carter <jwcart2@xxxxxxxxx> > --- > libsepol/cil/src/cil_resolve_ast.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/libsepol/cil/src/cil_resolve_ast.c b/libsepol/cil/src/cil_resolve_ast.c > index 47cdf0e7c0b9..2ea106d63505 100644 > --- a/libsepol/cil/src/cil_resolve_ast.c > +++ b/libsepol/cil/src/cil_resolve_ast.c > @@ -2497,7 +2497,7 @@ int cil_resolve_in(struct cil_tree_node *current, void *extra_args) > > rc = cil_copy_ast(db, current, block_node); > if (rc != SEPOL_OK) { > - printf("Failed to copy in, rc: %d\n", rc); > + cil_tree_log(current, CIL_ERR, "Failed to copy in-statement"); > goto exit; > } > > @@ -2788,7 +2788,7 @@ int cil_resolve_call1(struct cil_tree_node *current, void *extra_args) > macro_node = NODE(macro_datum); > > if (macro_node->flavor != CIL_MACRO) { > - printf("Failed to resolve %s to a macro\n", new_call->macro_str); > + cil_tree_log(current, CIL_ERR, "Failed to resolve %s to a macro", new_call->macro_str); > rc = SEPOL_ERR; > goto exit; > } > -- > 2.30.2 >