From 5aa51428150b880e694d039b171ee6f8a597dbd8 Mon Sep 17 00:00:00 2001 From: Aurelien Foret Date: Tue, 7 Feb 2006 22:01:50 +0000 Subject: error codes cleanup --- lib/libalpm/alpm.c | 16 +++++++++------- 1 file changed, 9 insertions(+), 7 deletions(-) (limited to 'lib/libalpm/alpm.c') diff --git a/lib/libalpm/alpm.c b/lib/libalpm/alpm.c index d3eceefa..788e504c 100644 --- a/lib/libalpm/alpm.c +++ b/lib/libalpm/alpm.c @@ -312,7 +312,7 @@ int alpm_db_update(PM_DB *db, char *archive, char *ts) _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); + RET_ERR(PM_ERR_DB_REMOVE, -1); } } @@ -325,12 +325,12 @@ int alpm_db_update(PM_DB *db, char *archive, char *ts) db_write each entry (see sync_load_dbarchive to get archive content) */ _alpm_log(PM_LOG_FLOW2, "unpacking %s", archive); if(_alpm_unpack(archive, db->path, NULL)) { - RET_ERR(PM_ERR_XXX, -1); + RET_ERR(PM_ERR_SYSTEM, -1); } if(ts && strlen(ts) != 0) { if(db_setlastupdate(db, ts) == -1) { - RET_ERR(PM_ERR_XXX, -1); + RET_ERR(PM_ERR_SYSTEM, -1); } } @@ -527,9 +527,11 @@ int alpm_pkg_load(char *filename, pmpkg_t **pkg) int alpm_pkg_free(pmpkg_t *pkg) { ASSERT(pkg != NULL, RET_ERR(PM_ERR_WRONG_ARGS, -1)); - ASSERT(pkg->origin != PKG_FROM_CACHE, RET_ERR(PM_ERR_XXX, -1)); - pkg_free(pkg); + /* Only free packages loaded in user space */ + if(pkg->origin != PKG_FROM_CACHE) { + pkg_free(pkg); + } return(0); } @@ -545,7 +547,7 @@ int alpm_pkg_checkmd5sum(pmpkg_t *pkg) int retval = 0; ASSERT(pkg != NULL, RET_ERR(PM_ERR_WRONG_ARGS, -1)); - ASSERT(pkg->md5sum[0] != 0, RET_ERR(PM_ERR_XXX, -1)); + ASSERT(pkg->md5sum[0] != 0, RET_ERR(PM_ERR_WRONG_ARGS, -1)); asprintf(&path, "%s%s/%s-%s" PM_EXT_PKG, handle->root, handle->cachedir, @@ -719,7 +721,7 @@ int alpm_trans_sysupgrade() trans = handle->trans; ASSERT(trans != NULL, RET_ERR(PM_ERR_TRANS_NULL, -1)); ASSERT(trans->state == STATE_INITIALIZED, RET_ERR(PM_ERR_TRANS_NOT_INITIALIZED, -1)); - ASSERT(trans->type == PM_TRANS_TYPE_SYNC, RET_ERR(PM_ERR_XXX, -1)); + ASSERT(trans->type == PM_TRANS_TYPE_SYNC, RET_ERR(PM_ERR_TRANS_TYPE, -1)); return(trans_sysupgrade(trans)); } -- cgit v1.2.3-24-g4f1b