From a48b72b8b9115ba673643ae2ba5a6d9a2326880d Mon Sep 17 00:00:00 2001
From: jbj <devnull@localhost>
Date: Wed, 13 Jan 1999 18:30:31 +0000
Subject: [PATCH] Fix: seg fault due to multiple close in runScript(). More
 const char * within verify.

CVS patchset: 2696
CVS date: 1999/01/13 18:30:31
---
 lib/uninstall.c | 10 +++-------
 verify.c        | 20 ++++++++++----------
 verify.h        |  2 +-
 3 files changed, 14 insertions(+), 18 deletions(-)

diff --git a/lib/uninstall.c b/lib/uninstall.c
index 6cddfb1..9b9f0d3 100644
--- a/lib/uninstall.c
+++ b/lib/uninstall.c
@@ -223,12 +223,11 @@ static int runScript(Header h, const char * root, int progArgc, const char ** pr
 
     if (errfd != NULL) {
 	if (rpmIsVerbose()) {
-	    out = errfd;
+	    out = fdDup(fdFileno(errfd));
 	} else {
 	    out = fdOpen("/dev/null", O_WRONLY, 0);
-	    if (fdFileno(out) < 0) {
-		out = errfd;
-	    }
+	    if (fdFileno(out) < 0)
+		out = fdDup(fdFileno(errfd));
 	}
     } else {
 	out = fdDup(STDOUT_FILENO);
@@ -281,9 +280,6 @@ static int runScript(Header h, const char * root, int progArgc, const char ** pr
     if (freePrefixes) free(prefixes);
 
     fdClose(out);	/* XXX dup'd STDOUT_FILENO */
-    if (errfd != NULL) {
-	if (fdFileno(errfd) > STDERR_FILENO) fdClose(errfd);
-    }
     
     if (script) {
 	if (!rpmIsDebug()) unlink(fn);
diff --git a/verify.c b/verify.c
index b00666e..85517d5 100644
--- a/verify.c
+++ b/verify.c
@@ -6,13 +6,13 @@
 #include "url.h"
 #include "verify.h"
 
-static int verifyHeader(char * prefix, Header h, int verifyFlags);
-static int verifyMatches(char * prefix, rpmdb db, dbiIndexSet matches,
+static int verifyHeader(const char * prefix, Header h, int verifyFlags);
+static int verifyMatches(const char * prefix, rpmdb db, dbiIndexSet matches,
 			  int verifyFlags);
 static int verifyDependencies(rpmdb db, Header h);
 
-static int verifyHeader(char * prefix, Header h, int verifyFlags) {
-    char ** fileList;
+static int verifyHeader(const char * prefix, Header h, int verifyFlags) {
+    const char ** fileList;
     int count, type;
     int verifyResult;
     int i, ec, rc;
@@ -77,7 +77,7 @@ static int verifyDependencies(rpmdb db, Header h) {
     rpmTransactionSet rpmdep;
     struct rpmDependencyConflict * conflicts;
     int numConflicts;
-    char * name, * version, * release;
+    const char * name, * version, * release;
     int type, count, i;
 
     rpmdep = rpmtransCreateSet(db, NULL);
@@ -107,7 +107,7 @@ static int verifyDependencies(rpmdb db, Header h) {
     return 0;
 }
 
-static int verifyPackage(char * root, rpmdb db, Header h, int verifyFlags) {
+static int verifyPackage(const char * root, rpmdb db, Header h, int verifyFlags) {
     int ec, rc;
     FD_t fdo;
     ec = 0;
@@ -125,7 +125,7 @@ static int verifyPackage(char * root, rpmdb db, Header h, int verifyFlags) {
     return ec;
 }
 
-static int verifyMatches(char * prefix, rpmdb db, dbiIndexSet matches,
+static int verifyMatches(const char * prefix, rpmdb db, dbiIndexSet matches,
 			  int verifyFlags) {
     int ec, rc;
     int i;
@@ -152,7 +152,7 @@ static int verifyMatches(char * prefix, rpmdb db, dbiIndexSet matches,
     return ec;
 }
 
-int doVerify(char * prefix, enum verifysources source, char ** argv,
+int doVerify(const char * prefix, enum verifysources source, const char ** argv,
 	      int verifyFlags) {
     Header h;
     int offset;
@@ -160,7 +160,6 @@ int doVerify(char * prefix, enum verifysources source, char ** argv,
     int isSource;
     rpmdb db;
     dbiIndexSet matches;
-    char * arg;
 
     ec = 0;
     if (source == VERIFY_RPM && !(verifyFlags & VERIFY_DEPS)) {
@@ -180,13 +179,14 @@ int doVerify(char * prefix, enum verifysources source, char ** argv,
 		    fprintf(stderr, _("could not read database record!\n"));
 		    return 1;	/* XXX was exit(EXIT_FAILURE) */
 		}
+		
 		if ((rc = verifyPackage(prefix, db, h, verifyFlags)) != 0)
 		    ec = rc;
 		headerFree(h);
 	}
     } else {
 	while (*argv) {
-	    arg = *argv++;
+	    const char *arg = *argv++;
 
 	    rc = 0;
 	    switch (source) {
diff --git a/verify.h b/verify.h
index 193ec24..9ded9ac 100644
--- a/verify.h
+++ b/verify.h
@@ -11,7 +11,7 @@
 enum verifysources { VERIFY_PATH, VERIFY_PACKAGE, VERIFY_EVERY, VERIFY_RPM, 
 			VERIFY_GRP, };
 
-int doVerify(char * prefix, enum verifysources source, char ** argv,
+int doVerify(const char * prefix, enum verifysources source, const char ** argv,
 	      int verifyFlags);
 
 #endif
-- 
2.7.4