From fb8437b588356f2efd9dd093a304edc3a1859a1d Mon Sep 17 00:00:00 2001 From: Sören Brinkmann Date: Tue, 11 Mar 2014 19:29:22 -0700 Subject: pacman/upgrade: Fix memory leaks MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Make sure allocated memory is freed before returning. Signed-off-by: Sören Brinkmann Signed-off-by: Allan McRae --- src/pacman/upgrade.c | 19 +++++++++++++------ 1 file changed, 13 insertions(+), 6 deletions(-) diff --git a/src/pacman/upgrade.c b/src/pacman/upgrade.c index cbc605d6..a6be0e18 100644 --- a/src/pacman/upgrade.c +++ b/src/pacman/upgrade.c @@ -75,12 +75,13 @@ int pacman_upgrade(alpm_list_t *targets) } if(retval) { - return retval; + goto fail_free; } /* Step 1: create a new transaction */ if(trans_init(config->flags, 1) == -1) { - return 1; + retval = 1; + goto fail_free; } printf(_("loading packages...\n")); @@ -112,15 +113,21 @@ int pacman_upgrade(alpm_list_t *targets) config->explicit_adds = alpm_list_add(config->explicit_adds, pkg); } - free(file_is_remote); - if(retval) { - trans_release(); - return retval; + goto fail_release; } + free(file_is_remote); + /* now that targets are resolved, we can hand it all off to the sync code */ return sync_prepare_execute(); + +fail_release: + trans_release(); +fail_free: + free(file_is_remote); + + return retval; } /* vim: set noet: */ -- cgit v1.2.3-24-g4f1b