More dead NULL assignments of local variables in filelist processing
authorPanu Matilainen <pmatilai@redhat.com>
Fri, 27 May 2011 10:45:58 +0000 (13:45 +0300)
committerPanu Matilainen <pmatilai@redhat.com>
Fri, 27 May 2011 12:23:28 +0000 (15:23 +0300)
build/files.c

index e98567c..be63790 100644 (file)
@@ -1967,7 +1967,7 @@ rpmRC processSourceFiles(rpmSpec spec, rpmBuildPkgFlags pkgFlags)
        char * sfn = rpmGetPath( ((srcPtr->flags & RPMBUILD_ISNO) ? "!" : ""),
                "%{_sourcedir}/", srcPtr->source, NULL);
        argvAdd(&files, sfn);
-       sfn = _free(sfn);
+       free(sfn);
     }
 
     for (pkg = spec->packages; pkg != NULL; pkg = pkg->next) {
@@ -1976,7 +1976,7 @@ rpmRC processSourceFiles(rpmSpec spec, rpmBuildPkgFlags pkgFlags)
            sfn = rpmGetPath( ((srcPtr->flags & RPMBUILD_ISNO) ? "!" : ""),
                "%{_sourcedir}/", srcPtr->source, NULL);
            argvAdd(&files, sfn);
-           sfn = _free(sfn);
+           free(sfn);
        }
     }
 
@@ -2104,8 +2104,8 @@ static int checkFiles(const char *buildRoot, StringBuf fileList)
     }
     
 exit:
-    sb_stdout = freeStringBuf(sb_stdout);
-    s = _free(s);
+    freeStringBuf(sb_stdout);
+    free(s);
     return rc;
 }