From 29636c5ddc336a6a43da62f8be5172740e38b1ac Mon Sep 17 00:00:00 2001 From: Nicolas Iooss Date: Sat, 26 May 2018 18:22:29 +0200 Subject: [PATCH] libsepol: remove unused variable 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 --- 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 56887366..dcf6ebb1 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);