From: Panu Matilainen <pmatilai@redhat.com>
Date: Wed, 16 Sep 2009 09:41:44 +0000 (+0300)
Subject: Minor cleanup to newRpmdb()
X-Git-Tag: rpm-4.8.0-beta1~278
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=4cfcdd661d398691163f27c0ad9744b33d2dfae9;p=platform%2Fupstream%2Frpm.git

Minor cleanup to newRpmdb()
- no need to free what hasn't been allocated yet
---

diff --git a/lib/rpmdb.c b/lib/rpmdb.c
index c405fa5..890b674 100644
--- a/lib/rpmdb.c
+++ b/lib/rpmdb.c
@@ -907,19 +907,14 @@ rpmdb newRpmdb(const char * root,
     if (perms >= 0)	db->db_perms = perms;
     if (flags >= 0)	db->db_flags = flags;
 
-    if (root && *root) {
-	db->db_root = rpmGetPath(root, NULL);
-    } else
-	db->db_root = rpmGetPath(_DB_ROOT, NULL);
-
     db->db_home = rpmGetPath( (home && *home ? home : _DB_HOME), NULL);
     if (!(db->db_home && db->db_home[0] != '%')) {
 	rpmlog(RPMLOG_ERR, _("no dbpath has been set\n"));
-	db->db_root = _free(db->db_root);
 	db->db_home = _free(db->db_home);
 	db = _free(db);
 	return NULL;
     }
+    db->db_root = rpmGetPath((root && *root) ? root : _DB_ROOT, NULL);
     db->db_errpfx = rpmExpand( (epfx && *epfx ? epfx : _DB_ERRPFX), NULL);
     db->db_remove_env = 0;
     db->db_filter_dups = _db_filter_dups;