Since commit 2ee7a8d8, there is no cleanup needed in this function. Just
return instead of jumping to the cleanup label.
Signed-off-by: Allan McRae
---
lib/libalpm/add.c | 16 +++++-----------
1 file changed, 5 insertions(+), 11 deletions(-)
diff --git a/lib/libalpm/add.c b/lib/libalpm/add.c
index 8ea89407..db0dba68 100644
--- a/lib/libalpm/add.c
+++ b/lib/libalpm/add.c
@@ -483,8 +483,7 @@ static int commit_single_pkg(alpm_handle_t *handle, alpm_pkg_t *newpkg,
/* set up fake remove transaction */
if(_alpm_remove_single_package(handle, oldpkg, newpkg, 0, 0) == -1) {
handle->pm_errno = ALPM_ERR_TRANS_ABORT;
- ret = -1;
- goto cleanup;
+ return -1;
}
}
@@ -495,15 +494,13 @@ static int commit_single_pkg(alpm_handle_t *handle, alpm_pkg_t *newpkg,
"error: could not create database entry %s-%s\n",
newpkg->name, newpkg->version);
handle->pm_errno = ALPM_ERR_DB_WRITE;
- ret = -1;
- goto cleanup;
+ return -1;
}
fd = _alpm_open_archive(db->handle, pkgfile, &buf,
&archive, ALPM_ERR_PKG_OPEN);
if(fd < 0) {
- ret = -1;
- goto cleanup;
+ return -1;
}
/* save the cwd so we can restore it later */
@@ -521,8 +518,7 @@ static int commit_single_pkg(alpm_handle_t *handle, alpm_pkg_t *newpkg,
close(cwdfd);
}
close(fd);
- ret = -1;
- goto cleanup;
+ return -1;
}
if(trans->flags & ALPM_TRANS_FLAG_DBONLY) {
@@ -606,8 +602,7 @@ static int commit_single_pkg(alpm_handle_t *handle, alpm_pkg_t *newpkg,
"error: could not update database entry %s-%s\n",
newpkg->name, newpkg->version);
handle->pm_errno = ALPM_ERR_DB_WRITE;
- ret = -1;
- goto cleanup;
+ return -1;
}
if(_alpm_db_add_pkgincache(db, newpkg) == -1) {
@@ -653,7 +648,6 @@ static int commit_single_pkg(alpm_handle_t *handle, alpm_pkg_t *newpkg,
event.type = ALPM_EVENT_PACKAGE_OPERATION_DONE;
EVENT(handle, &event);
-cleanup:
return ret;
}
--
2.25.0