From fe76c353af6d91ce9f225dc2b0195a689f6790da Mon Sep 17 00:00:00 2001 From: Dan McGee Date: Mon, 24 Jan 2011 21:46:42 -0600 Subject: Fix memory leak and error code in DB reading We were returning a package error code rather than a DB one, and we would leak the archive memory if the database file didn't exist. Signed-off-by: Dan McGee --- lib/libalpm/be_sync.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) (limited to 'lib') diff --git a/lib/libalpm/be_sync.c b/lib/libalpm/be_sync.c index 4676e8a6..8c67bb19 100644 --- a/lib/libalpm/be_sync.c +++ b/lib/libalpm/be_sync.c @@ -166,7 +166,8 @@ static int sync_db_populate(pmdb_t *db) ARCHIVE_DEFAULT_BYTES_PER_BLOCK) != ARCHIVE_OK) { _alpm_log(PM_LOG_ERROR, _("could not open %s: %s\n"), _alpm_db_path(db), archive_error_string(archive)); - RET_ERR(PM_ERR_PKG_OPEN, 1); + archive_read_finish(archive); + RET_ERR(PM_ERR_DB_OPEN, 1); } while(archive_read_next_header(archive, &entry) == ARCHIVE_OK) { @@ -180,7 +181,7 @@ static int sync_db_populate(pmdb_t *db) pkg = _alpm_pkg_new(); if(pkg == NULL) { archive_read_finish(archive); - return(-1); + RET_ERR(PM_ERR_MEMORY, -1); } name = archive_entry_pathname(entry); -- cgit v1.2.3-54-g00ecf