bryson/pacman
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix memleak found by add004.py

Browse source 
Signed-off-by: Dan McGee <dan@archlinux.org>
This commit is contained in:
Dan McGee 2008-01-14 00:33:13 -06:00
parent cbcf542ad2
commit 36264a3ab9
1 changed files with 8 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

20
lib/libalpm/add.c
View file

@ -79,21 +79,17 @@ int _alpm_add_loadtarget(pmtrans_t *trans, pmdb_t *db, char *name)
/* check if an older version of said package is already in transaction /* check if an older version of said package is already in transaction
* packages. if so, replace it in the list */ * packages. if so, replace it in the list */
for(i = trans->packages; i; i = i->next) { for(i = trans->packages; i; i = i->next) {
pmpkg_t *pkg = i->data; pmpkg_t *transpkg = i->data;
if(strcmp(pkg->name, pkgname) == 0) { if(strcmp(transpkg->name, pkgname) == 0) {
if(_alpm_versioncmp(pkg->version, pkgver) < 0) { if(_alpm_versioncmp(transpkg->version, pkgver) < 0) {
pmpkg_t *newpkg;
_alpm_log(PM_LOG_WARNING, _("replacing older version %s-%s by %s in target list\n"), _alpm_log(PM_LOG_WARNING, _("replacing older version %s-%s by %s in target list\n"),
pkg->name, pkg->version, pkgver); transpkg->name, transpkg->version, pkgver);
if((newpkg = _alpm_pkg_load(name, 1)) == NULL) {
/* pm_errno is already set by pkg_load() */
goto error;
}
_alpm_pkg_free(i->data); _alpm_pkg_free(i->data);
i->data = newpkg; i->data = pkg;
} else { } else {
_alpm_log(PM_LOG_WARNING, _("newer version %s-%s is in the target list -- skipping\n"), _alpm_log(PM_LOG_WARNING, _("skipping %s-%s because newer version %s is in the target list\n"),
pkg->name, pkg->version); pkgname, pkgver, transpkg->version);
_alpm_pkg_free(pkg);
} }
return(0); return(0);
} }