diff options
| author | Dan McGee <dan@archlinux.org> | 2011-06-03 18:37:15 +0200 |
|---|---|---|
| committer | Dan McGee <dan@archlinux.org> | 2011-06-14 02:41:16 +0200 |
| commit | 29ea0fa09fbde915d8abfdda2ee98493cd041239 (patch) | |
| tree | 2ff5c7066d81ecc545f2d32b6c9d056ca1aeb20e | |
| parent | 4d9278f87fffc1d1fe7e6a2f805a2a8df296bb0d (diff) | |
| download | pacman-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.c | 6 |
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) { |