When write_contexts() frees variables context and new_context_str after a line has been successfully emitted, these variables are not reset to NULL. This leads the function to free them again if an error occurs when processing the next line. Fix this by always resetting these variables at the beginning of the loop.
This issue has been found using clang's static analyzer. Signed-off-by: Nicolas Iooss <nicolas.io...@m4x.org> --- libsemanage/src/genhomedircon.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/libsemanage/src/genhomedircon.c b/libsemanage/src/genhomedircon.c index 465dd8829403..e8c95ee46130 100644 --- a/libsemanage/src/genhomedircon.c +++ b/libsemanage/src/genhomedircon.c @@ -607,10 +607,12 @@ static int write_contexts(genhomedircon_settings_t *s, FILE *out, const genhomedircon_user_entry_t *user) { char *line, *temp; - sepol_context_t *context = NULL; - char *new_context_str = NULL; + sepol_context_t *context; + char *new_context_str; for (; tpl; tpl = tpl->next) { + context = NULL; + new_context_str = NULL; line = replace_all(tpl->data, repl); if (!line) { goto fail; -- 2.12.0 _______________________________________________ Selinux mailing list Selinux@tycho.nsa.gov To unsubscribe, send email to selinux-le...@tycho.nsa.gov. To get help, send an email containing "help" to selinux-requ...@tycho.nsa.gov.