sepol_ppfile_to_module_package() does not use its variable "FILE *f = NULL;" but to fclose() it. This variable has been unneeded since the introduction of function ppfile_to_module_package() in commit 893851c0a146 ("policycoreutils: add a HLL compiler to convert policy packages (.pp) to CIL"). Signed-off-by: Nicolas Iooss <nicolas.iooss@xxxxxxx> --- libsepol/src/module_to_cil.c | 4 ---- 1 file changed, 4 deletions(-) diff --git a/libsepol/src/module_to_cil.c b/libsepol/src/module_to_cil.c index 56887366707a..dcf6ebb10b60 100644 --- a/libsepol/src/module_to_cil.c +++ b/libsepol/src/module_to_cil.c @@ -4232,7 +4232,6 @@ exit: int sepol_ppfile_to_module_package(FILE *fp, struct sepol_module_package **mod_pkg) { int rc = -1; - FILE *f = NULL; struct sepol_policy_file *pf = NULL; struct sepol_module_package *pkg = NULL; char *data = NULL; @@ -4284,9 +4283,6 @@ exit: free(data); sepol_policy_file_free(pf); - if (f != NULL) { - fclose(f); - } if (rc != 0) { sepol_module_package_free(pkg); -- 2.17.0 _______________________________________________ Selinux mailing list Selinux@xxxxxxxxxxxxx To unsubscribe, send email to Selinux-leave@xxxxxxxxxxxxx. To get help, send an email containing "help" to Selinux-request@xxxxxxxxxxxxx.