From c2cce1f46a754c04c3623088a9e43922a6c10e2d Mon Sep 17 00:00:00 2001 From: Xavier Chantry Date: Sun, 31 Oct 2010 21:39:31 +0100 Subject: Fix a few problems reported by clang-analyzer One missing NULL-check and 3 dead assignments. Signed-off-by: Xavier Chantry Signed-off-by: Dan McGee --- lib/libalpm/be_package.c | 3 +-- lib/libalpm/be_sync.c | 12 +++++------- 2 files changed, 6 insertions(+), 9 deletions(-) (limited to 'lib') diff --git a/lib/libalpm/be_package.c b/lib/libalpm/be_package.c index 254f830b..a0a657db 100644 --- a/lib/libalpm/be_package.c +++ b/lib/libalpm/be_package.c @@ -54,7 +54,6 @@ void *_package_changelog_open(pmpkg_t *pkg) struct archive *archive = NULL; struct archive_entry *entry; const char *pkgfile = pkg->origin_data.file; - int ret = ARCHIVE_OK; if((archive = archive_read_new()) == NULL) { RET_ERR(PM_ERR_LIBARCHIVE, NULL); @@ -68,7 +67,7 @@ void *_package_changelog_open(pmpkg_t *pkg) RET_ERR(PM_ERR_PKG_OPEN, NULL); } - while((ret = archive_read_next_header(archive, &entry)) == ARCHIVE_OK) { + while(archive_read_next_header(archive, &entry) == ARCHIVE_OK) { const char *entry_name = archive_entry_pathname(entry); if(strcmp(entry_name, ".CHANGELOG") == 0) { diff --git a/lib/libalpm/be_sync.c b/lib/libalpm/be_sync.c index e1c76638..851d7b83 100644 --- a/lib/libalpm/be_sync.c +++ b/lib/libalpm/be_sync.c @@ -129,7 +129,6 @@ int _alpm_sync_db_populate(pmdb_t *db) int count = 0; struct archive *archive; struct archive_entry *entry; - const char * archive_path; ALPM_LOG_FUNC; @@ -156,8 +155,6 @@ int _alpm_sync_db_populate(pmdb_t *db) st = archive_entry_stat(entry); if(S_ISDIR(st->st_mode)) { - archive_path = archive_entry_pathname(entry); - pkg = _alpm_pkg_new(); if(pkg == NULL) { archive_read_finish(archive); @@ -204,7 +201,7 @@ int _alpm_sync_db_populate(pmdb_t *db) int _alpm_sync_db_read(pmdb_t *db, struct archive *archive, struct archive_entry *entry) { char line[1024]; - const char *entryname; + const char *entryname = NULL; char *filename, *pkgname, *p, *q; pmpkg_t *pkg; @@ -214,13 +211,14 @@ int _alpm_sync_db_read(pmdb_t *db, struct archive *archive, struct archive_entry RET_ERR(PM_ERR_DB_NULL, -1); } - if(entry == NULL) { + if(entry != NULL) { + entryname = archive_entry_pathname(entry); + } + if(entryname == NULL) { _alpm_log(PM_LOG_DEBUG, "invalid archive entry provided to _alpm_sync_db_read, skipping\n"); return(-1); } - entryname = archive_entry_pathname(entry); - _alpm_log(PM_LOG_FUNCTION, "loading package data from archive entry %s\n", entryname); -- cgit v1.2.3-70-g09d2