Remove redundant NULL checks

- Fclose() handles NULL by itself, no need to check in callers.
  Also no need to NULL'ify the local variables here.

diff --git a/build/pack.c b/build/pack.c
index 9d4f404..5a0e844 100644 (file)
--- a/build/pack.c
+++ b/build/pack.c
@@ -499,14 +499,9 @@ exit:
     }
 
     sig = rpmFreeSignature(sig);
-    if (ifd) {
-       (void) Fclose(ifd);
-       ifd = NULL;
-    }
-    if (fd) {
-       (void) Fclose(fd);
-       fd = NULL;
-    }
+    Fclose(ifd);
+    Fclose(fd);
+
     if (sigtarget) {
        (void) unlink(sigtarget);
        sigtarget = _free(sigtarget);