summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAurelien Foret <aurelien@archlinux.org>2005-04-24 21:27:35 +0000
committerAurelien Foret <aurelien@archlinux.org>2005-04-24 21:27:35 +0000
commit49257b49f9dce71f8c5970d20e3a41e4541b70e8 (patch)
tree6c53877378bf62b5ccb54bfcca319c740e947830
parent7c04fe95ac822f435790aafeeeb9a91dd82f87be (diff)
downloadpacman-49257b49f9dce71f8c5970d20e3a41e4541b70e8.tar.xz
avoided to load twice the same target in remove_loadtarget()
-rw-r--r--lib/libalpm/remove.c7
1 files changed, 7 insertions, 0 deletions
diff --git a/lib/libalpm/remove.c b/lib/libalpm/remove.c
index 279e2cb9..e42035ee 100644
--- a/lib/libalpm/remove.c
+++ b/lib/libalpm/remove.c
@@ -56,10 +56,17 @@ int remove_loadtarget(pmtrans_t *trans, pmdb_t *db, char *name)
/* ORE
we should better find the package in the cache, and then perform a
db_read(INFRQ_FILES) to add files information to it. */
+ _alpm_log(PM_LOG_FLOW2, "loading target %s", name);
if((info = db_scan(db, name, INFRQ_ALL)) == NULL) {
_alpm_log(PM_LOG_ERROR, "could not find %s in database", name);
RET_ERR(PM_ERR_PKG_NOT_FOUND, -1);
}
+
+ if(pkg_isin(info, trans->packages)) {
+ FREEPKG(info);
+ RET_ERR(PM_ERR_TRANS_DUP_TARGET, -1);
+ }
+
trans->packages = pm_list_add(trans->packages, info);
return(0);