From: jbj <devnull@localhost>
Date: Fri, 31 Mar 2000 09:33:17 +0000 (+0000)
Subject: fix: duplicate free in db3.
X-Git-Tag: tznext/4.11.0.1.tizen20130304~8507
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=920d26a5150cbfb9ed9c8e547470d4b02c9a267b;p=tools%2Flibrpm-tizen.git

fix: duplicate free in db3.
Use Berkeley "home" db directory with relative rather than absolute paths.

CVS patchset: 3646
CVS date: 2000/03/31 09:33:17
---

diff --git a/lib/db2.c b/lib/db2.c
index 3adc4c5..1ed4bec 100644
--- a/lib/db2.c
+++ b/lib/db2.c
@@ -20,12 +20,12 @@ static FILE * db_errfile = NULL;
 static const char * db_errpfx = "rpmdb";
 static int db_verbose = 1;
 
-static int dbmp_mmapsize = 16 * 1024 * 1024;	/* XXX db2 default: 10 Mb */
-static int dbmp_size = 16 * 1024 * 1024;	/* XXX db2 default: 128 Kb */
+static int dbmp_mmapsize = 10 * 1024 * 1024;	/* XXX db2 default: 10 Mb */
+static int dbmp_size = 1 * 1024 * 1024;		/* XXX db2 default: 128 Kb */
 
 /* XXX dbinfo parameters */
 static int db_cachesize = 0;
-static int db_pagesize = 16 * 1024;	/* 512 - 64K, 0 is fs blksize */
+static int db_pagesize = 0;			/* 512 - 64K, 0 is fs blksize */
 static void * (*db_malloc) (size_t nbytes) = NULL;
 
 static u_int32_t (*dbh_hash) (const void *, u_int32_t) = NULL;
@@ -33,8 +33,8 @@ static u_int32_t dbh_ffactor = 0;	/* db1 default: 8 */
 static u_int32_t dbh_nelem = 0;		/* db1 default: 1 */
 static u_int32_t dbh_flags = 0;
 
-#if defined(INIT_CDB)
-static int dbopenflags = DB_INIT_CDB;
+#if defined(__USE_DB3)
+static int dbopenflags = DB_INIT_MPOOL|DB_PRIVATE;
 #else
 static int dbopenflags = DB_INIT_MPOOL;
 #endif
@@ -264,13 +264,14 @@ static int db2close(dbiIndex dbi, unsigned int flags)
 #else
 	xx = db_appexit(dbenv);
 	xx = cvtdberr(dbi, "db_appexit", xx, _debug);
-#endif
 	free(dbi->dbi_dbenv);
+#endif
 	dbi->dbi_dbenv = NULL;
     }
 #else
     rc = db->close(db);
 #endif
+    dbi->dbi_db = NULL;
 
     return rc;
 }
@@ -418,11 +419,20 @@ static int db2open(dbiIndex dbi)
 
 #if defined(__USE_DB2) || defined(__USE_DB3)
     char * dbhome = NULL;
+    char * dbfile = NULL;
     DB * db = NULL;
     DB_ENV * dbenv = NULL;
     void * dbinfo = NULL;
     u_int32_t dbflags;
 
+    dbhome = alloca(strlen(dbi->dbi_file) + 1);
+    strcpy(dbhome, dbi->dbi_file);
+    dbfile = strrchr(dbhome, '/');
+    if (dbfile)
+	*dbfile++ = '\0';
+    else
+	dbfile = dbhome;
+
     dbflags = (	!(dbi->dbi_flags & O_RDWR) ? DB_RDONLY :
 		((dbi->dbi_flags & O_CREAT) ? DB_CREATE : 0));
 
@@ -433,14 +443,16 @@ static int db2open(dbiIndex dbi)
 	rc = db_create(&db, dbenv, 0);
 	rc = cvtdberr(dbi, "db_create", rc, _debug);
 	if (rc == 0) {
-	    rc = db->set_pagesize(db, db_pagesize);
-	    rc = cvtdberr(dbi, "db->set_pagesize", rc, _debug);
-	    rc = db->open(db, dbi->dbi_file, NULL, dbi_to_dbtype(dbi->dbi_type),
+	    if (db_pagesize) {
+		rc = db->set_pagesize(db, db_pagesize);
+		rc = cvtdberr(dbi, "db->set_pagesize", rc, _debug);
+	    }
+	    rc = db->open(db, dbfile, NULL, dbi_to_dbtype(dbi->dbi_type),
 			dbflags, dbi->dbi_perms);
 	    rc = cvtdberr(dbi, "db->open", rc, _debug);
 	}
 #else
-	rc = db_open(dbi->dbi_file, dbi_to_dbtype(dbi->dbi_type), dbflags,
+	rc = db_open(dbfile, dbi_to_dbtype(dbi->dbi_type), dbflags,
 			dbi->dbi_perms, dbenv, dbinfo, &db);
 	rc = cvtdberr(dbi, "db_open", rc, _debug);
 #endif	/* __USE_DB3 */
@@ -451,13 +463,13 @@ static int db2open(dbiIndex dbi)
     dbi->dbi_dbinfo = dbinfo;
 
 #else
-    dbi->dbi_db = dbopen(dbi->dbi_file, dbi->dbi_flags, dbi->dbi_perms,
+    dbi->dbi_db = dbopen(dbfile, dbi->dbi_flags, dbi->dbi_perms,
 		dbi_to_dbtype(dbi->dbi_type), dbi->dbi_openinfo);
 #endif	/* __USE_DB2 || __USE_DB3 */
 
     if (rc == 0 && dbi->dbi_db != NULL) {
 	rc = 0;
-fprintf(stderr, "*** db%dopen: %s\n", dbi->dbi_major, dbi->dbi_file);
+fprintf(stderr, "*** db%dopen: %s\n", dbi->dbi_major, dbfile);
     } else {
 	rc = 1;
     }
diff --git a/lib/db3.c b/lib/db3.c
index 26c8482..6fa248e 100644
--- a/lib/db3.c
+++ b/lib/db3.c
@@ -21,7 +21,7 @@ static const char * db_errpfx = "rpmdb";
 static int db_verbose = 1;
 
 static int dbmp_mmapsize = 16 * 1024 * 1024;	/* XXX db2 default: 10 Mb */
-static int dbmp_size = 16 * 1024 * 1024;	/* XXX db2 default: 128 Kb */
+static int dbmp_size = 2 * 1024 * 1024;		/* XXX db2 default: 128 Kb */
 
 /* XXX dbinfo parameters */
 static int db_cachesize = 0;
@@ -33,8 +33,8 @@ static u_int32_t dbh_ffactor = 0;	/* db1 default: 8 */
 static u_int32_t dbh_nelem = 0;		/* db1 default: 1 */
 static u_int32_t dbh_flags = 0;
 
-#if defined(__ISE_DB3)
-static int dbopenflags = 0;
+#if defined(__USE_DB3)
+static int dbopenflags = DB_INIT_MPOOL|DB_SYSTEM_MEM;	/* W2DO? DB_PRIVATE */
 #else
 static int dbopenflags = DB_INIT_MPOOL;
 #endif
@@ -264,13 +264,14 @@ static int db3close(dbiIndex dbi, unsigned int flags)
 #else
 	xx = db_appexit(dbenv);
 	xx = cvtdberr(dbi, "db_appexit", xx, _debug);
-#endif
 	free(dbi->dbi_dbenv);
+#endif
 	dbi->dbi_dbenv = NULL;
     }
 #else
     rc = db->close(db);
 #endif
+    dbi->dbi_db = NULL;
 
     return rc;
 }
@@ -418,11 +419,20 @@ static int db3open(dbiIndex dbi)
 
 #if defined(__USE_DB2) || defined(__USE_DB3)
     char * dbhome = NULL;
+    char * dbfile = NULL;
     DB * db = NULL;
     DB_ENV * dbenv = NULL;
     void * dbinfo = NULL;
     u_int32_t dbflags;
 
+    dbhome = alloca(strlen(dbi->dbi_file) + 1);
+    strcpy(dbhome, dbi->dbi_file);
+    dbfile = strrchr(dbhome, '/');
+    if (dbfile)
+	*dbfile++ = '\0';
+    else
+	dbfile = dbhome;
+
     dbflags = (	!(dbi->dbi_flags & O_RDWR) ? DB_RDONLY :
 		((dbi->dbi_flags & O_CREAT) ? DB_CREATE : 0));
 
@@ -437,12 +447,12 @@ static int db3open(dbiIndex dbi)
 		rc = db->set_pagesize(db, db_pagesize);
 		rc = cvtdberr(dbi, "db->set_pagesize", rc, _debug);
 	    }
-	    rc = db->open(db, dbi->dbi_file, NULL, dbi_to_dbtype(dbi->dbi_type),
+	    rc = db->open(db, dbfile, NULL, dbi_to_dbtype(dbi->dbi_type),
 			dbflags, dbi->dbi_perms);
 	    rc = cvtdberr(dbi, "db->open", rc, _debug);
 	}
 #else
-	rc = db_open(dbi->dbi_file, dbi_to_dbtype(dbi->dbi_type), dbflags,
+	rc = db_open(dbfile, dbi_to_dbtype(dbi->dbi_type), dbflags,
 			dbi->dbi_perms, dbenv, dbinfo, &db);
 	rc = cvtdberr(dbi, "db_open", rc, _debug);
 #endif	/* __USE_DB3 */
@@ -453,13 +463,13 @@ static int db3open(dbiIndex dbi)
     dbi->dbi_dbinfo = dbinfo;
 
 #else
-    dbi->dbi_db = dbopen(dbi->dbi_file, dbi->dbi_flags, dbi->dbi_perms,
+    dbi->dbi_db = dbopen(dbfile, dbi->dbi_flags, dbi->dbi_perms,
 		dbi_to_dbtype(dbi->dbi_type), dbi->dbi_openinfo);
 #endif	/* __USE_DB2 || __USE_DB3 */
 
     if (rc == 0 && dbi->dbi_db != NULL) {
 	rc = 0;
-fprintf(stderr, "*** db%dopen: %s\n", dbi->dbi_major, dbi->dbi_file);
+fprintf(stderr, "*** db%dopen: %s\n", dbi->dbi_major, dbfile);
     } else {
 	rc = 1;
     }