diff options
author | Aurelien Foret <aurelien@archlinux.org> | 2005-10-22 08:29:12 +0000 |
---|---|---|
committer | Aurelien Foret <aurelien@archlinux.org> | 2005-10-22 08:29:12 +0000 |
commit | 5d5d7d6339945e0f35e03740f275dbddfd72ae3d (patch) | |
tree | ad0894d5888cb578397140352c06cedcc6748aa7 /lib/libalpm | |
parent | fa627d4776ae56197f2aead259c8b5e19d269586 (diff) |
do not free the cache right before using it!
Diffstat (limited to 'lib/libalpm')
-rw-r--r-- | lib/libalpm/alpm.c | 24 |
1 files changed, 10 insertions, 14 deletions
diff --git a/lib/libalpm/alpm.c b/lib/libalpm/alpm.c index 427c536b..4719c493 100644 --- a/lib/libalpm/alpm.c +++ b/lib/libalpm/alpm.c @@ -282,25 +282,21 @@ int alpm_db_update(PM_DB *db, char *archive, char *ts) } } - if(stat(archive, &buf)) { - /* not found */ - RET_ERR(PM_ERR_NOT_A_FILE, -1); + /* remove the old dir */ + _alpm_log(PM_LOG_FLOW2, "flushing database %s/%s", handle->dbpath, db->treename); + for(lp = db_get_pkgcache(db); lp; lp = lp->next) { + if(db_remove(db, lp->data) == -1) { + if(lp->data) { + _alpm_log(PM_LOG_ERROR, "could not remove database entry %s/%s", db->treename, + ((pmpkg_t *)lp->data)->name); + } + RET_ERR(PM_ERR_XXX, -1); + } } /* Cache needs to be rebuild */ db_free_pkgcache(db); - /* remove the old dir */ - _alpm_log(PM_LOG_FLOW2, "removing database %s/%s", handle->dbpath, db->treename); - for(lp = alpm_db_getpkgcache(db); lp; lp = alpm_list_next(lp)) { - db_remove(db, alpm_list_getdata(lp)); - } - - /* make the new dir */ - if(db_create(handle->root, handle->dbpath, db->treename) != 0) { - RET_ERR(PM_ERR_DB_CREATE, -1); - } - /* uncompress the sync database */ /* ORE we should not simply unpack the archive, but better parse it and |