Callers are now well known, require non-NULL arguments
authorPanu Matilainen <pmatilai@redhat.com>
Tue, 10 Jun 2008 12:05:31 +0000 (15:05 +0300)
committerPanu Matilainen <pmatilai@redhat.com>
Tue, 10 Jun 2008 13:50:18 +0000 (16:50 +0300)
lib/tagexts.c

index 9f3dbf4..f0e437e 100644 (file)
@@ -65,8 +65,8 @@ static void rpmfiBuildFNames(Header h, rpmTag tagN,
     }
 
     if (!hge(h, tagN, &bnt, (rpm_data_t *) &baseNames, &count)) {
-       if (fnp) *fnp = NULL;
-       if (fcp) *fcp = 0;
+       *fnp = NULL;
+       *fcp = 0;
        return;         /* no file list */
     }
 
@@ -87,11 +87,8 @@ static void rpmfiBuildFNames(Header h, rpmTag tagN,
     baseNames = hfd(baseNames, bnt);
     dirNames = hfd(dirNames, dnt);
 
-    if (fnp)
-       *fnp = fileNames;
-    else
-       fileNames = _free(fileNames);
-    if (fcp) *fcp = count;
+    *fnp = fileNames;
+    *fcp = count;
 }
 static int filedepTag(Header h, rpmTag tagN, rpmtd td)
 {