diff options
| author | Dan McGee <dan@archlinux.org> | 2011-05-02 16:25:47 -0500 |
|---|---|---|
| committer | Dan McGee <dan@archlinux.org> | 2011-05-05 12:10:51 -0500 |
| commit | b14c5477e5e4483352d304a1e97de5922948b934 (patch) | |
| tree | 83b9d854c9c214d697a0f816f006fa459f8b04de /lib/libalpm/be_sync.c | |
| parent | 8fd9037cfd8836db7dd35ba0e8825ba86c4e4688 (diff) | |
| download | pacman-b14c5477e5e4483352d304a1e97de5922948b934.tar.xz | |
Ensure populate error return codes are consistent
It must be -1 to differentiate it from a number of packages loaded
count.
Signed-off-by: Dan McGee <dan@archlinux.org>
Diffstat (limited to 'lib/libalpm/be_sync.c')
| -rw-r--r-- | lib/libalpm/be_sync.c | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/lib/libalpm/be_sync.c b/lib/libalpm/be_sync.c index c2c62aa2..c440cd6b 100644 --- a/lib/libalpm/be_sync.c +++ b/lib/libalpm/be_sync.c @@ -223,8 +223,9 @@ static int sync_db_populate(pmdb_t *db) ASSERT(db != NULL, RET_ERR(PM_ERR_DB_NULL, -1)); - if((archive = archive_read_new()) == NULL) - RET_ERR(PM_ERR_LIBARCHIVE, 1); + if((archive = archive_read_new()) == NULL) { + RET_ERR(PM_ERR_LIBARCHIVE, -1); + } archive_read_support_compression_all(archive); archive_read_support_format_all(archive); @@ -232,7 +233,7 @@ static int sync_db_populate(pmdb_t *db) dbpath = _alpm_db_path(db); if(!dbpath) { /* pm_errno set in _alpm_db_path() */ - return 1; + return -1; } _alpm_log(PM_LOG_DEBUG, "opening database archive %s\n", dbpath); @@ -242,10 +243,10 @@ static int sync_db_populate(pmdb_t *db) _alpm_log(PM_LOG_ERROR, _("could not open file %s: %s\n"), dbpath, archive_error_string(archive)); archive_read_finish(archive); - RET_ERR(PM_ERR_DB_OPEN, 1); + RET_ERR(PM_ERR_DB_OPEN, -1); } if(stat(dbpath, &buf) != 0) { - RET_ERR(PM_ERR_DB_OPEN, 1); + RET_ERR(PM_ERR_DB_OPEN, -1); } est_count = estimate_package_count(&buf, archive); |