moparisthebest
/
pacman
mirror of https://github.com/moparisthebest/pacman
1
0
Fork 0
Code Issues Releases Wiki Activity

Fix a memleak in _alpm_sync_free. 

An alpm_list_free call was missing.
Also make use of alpm_list_free_inner in both _alpm_sync_free and
_alpm_trans_free.

Signed-off-by: Chantry Xavier <shiningxc@gmail.com>
Signed-off-by: Dan McGee <dan@archlinux.org>
This commit is contained in:
Chantry Xavier 2007-11-21 17:10:20 +01:00 committed by Dan McGee
parent f5d2150e9d
commit 8bee526d28
2 changed files with 4 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
lib/libalpm/sync.c
View File

@ -74,11 +74,8 @@ void _alpm_sync_free(pmsyncpkg_t *sync)
/* TODO wow this is ugly */
if(sync->type == PM_SYNC_TYPE_REPLACE) {
alpm_list_t *tmp;
for(tmp = sync->data; tmp; tmp = alpm_list_next(tmp)) {
_alpm_pkg_free(tmp->data);
tmp->data = NULL;
}
alpm_list_free_inner(sync->data, (alpm_list_fn_free)_alpm_pkg_free);
alpm_list_free(sync->data);
sync->data = NULL;
} else {
_alpm_pkg_free(sync->data);

10
lib/libalpm/trans.c
View File

@ -246,8 +246,6 @@ pmtrans_t *_alpm_trans_new()
void _alpm_trans_free(pmtrans_t *trans)
{
alpm_list_t *i;
ALPM_LOG_FUNC;
if(trans == NULL) {
@ -256,13 +254,9 @@ void _alpm_trans_free(pmtrans_t *trans)
FREELIST(trans->targets);
if(trans->type == PM_TRANS_TYPE_SYNC) {
for(i = trans->packages; i; i = alpm_list_next(i)) {
_alpm_sync_free(i->data);
}
alpm_list_free_inner(trans->packages, (alpm_list_fn_free)_alpm_sync_free);
} else {
for(i = trans->packages; i; i = alpm_list_next(i)) {
_alpm_pkg_free(i->data);
}
alpm_list_free_inner(trans->packages, (alpm_list_fn_free)_alpm_pkg_free);
}
alpm_list_free(trans->packages);