summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDan McGee <dan@archlinux.org>2011-06-03 18:37:15 +0200
committerDan McGee <dan@archlinux.org>2011-06-14 02:41:16 +0200
commit29ea0fa09fbde915d8abfdda2ee98493cd041239 (patch)
tree2ff5c7066d81ecc545f2d32b6c9d056ca1aeb20e
parent4d9278f87fffc1d1fe7e6a2f805a2a8df296bb0d (diff)
downloadpacman-29ea0fa09fbde915d8abfdda2ee98493cd041239.tar.gz
pacman-29ea0fa09fbde915d8abfdda2ee98493cd041239.tar.xz
Always pass data to trans_commit()
Even though we currently don't use it here in the backend, we might as well pass it in since we used it earlier. Signed-off-by: Dan McGee <dan@archlinux.org>
-rw-r--r--src/pacman/remove.c6
1 files changed, 4 insertions, 2 deletions
diff --git a/src/pacman/remove.c b/src/pacman/remove.c
index 8de538c8..a4e18941 100644
--- a/src/pacman/remove.c
+++ b/src/pacman/remove.c
@@ -121,11 +121,11 @@ int pacman_remove(alpm_list_t *targets)
depstring);
free(depstring);
}
- FREELIST(data);
break;
default:
break;
}
+ FREELIST(data);
retval = 1;
goto cleanup;
}
@@ -165,12 +165,14 @@ int pacman_remove(alpm_list_t *targets)
goto cleanup;
}
- if(alpm_trans_commit(config->handle, NULL) == -1) {
+ if(alpm_trans_commit(config->handle, &data) == -1) {
pm_fprintf(stderr, PM_LOG_ERROR, _("failed to commit transaction (%s)\n"),
alpm_strerror(alpm_errno(config->handle)));
retval = 1;
}
+ FREELIST(data);
+
/* Step 4: release transaction resources */
cleanup:
if(trans_release() == -1) {