summaryrefslogtreecommitdiff
path: root/lib/libalpm/alpm.c
diff options
context:
space:
mode:
authorAaron Griffin <aaron@archlinux.org>2006-11-08 06:52:50 +0000
committerAaron Griffin <aaron@archlinux.org>2006-11-08 06:52:50 +0000
commit55f178c629ada663d2a8e5bbed029ec9482f00ea (patch)
treefbd271d7ef2d8c97df1727c02f6ab9b2508813ec /lib/libalpm/alpm.c
parent34931106d7653e73bb474e1d72aee4044f7d5a5d (diff)
* Fixed an error message that should be a debug message
* Fixed "the big segfault" - dbs_sync list was being modified while iterating * over the list, big No-No. Now using a while loop over alpm_list_count.
Diffstat (limited to 'lib/libalpm/alpm.c')
-rw-r--r--lib/libalpm/alpm.c10
1 files changed, 6 insertions, 4 deletions
diff --git a/lib/libalpm/alpm.c b/lib/libalpm/alpm.c
index d360fa65..9f6271f1 100644
--- a/lib/libalpm/alpm.c
+++ b/lib/libalpm/alpm.c
@@ -100,7 +100,7 @@ int alpm_initialize(char *root)
*/
int alpm_release()
{
- pmlist_t *i;
+ int dbs_left = 0;
ASSERT(handle != NULL, RET_ERR(PM_ERR_HANDLE_NULL, -1));
@@ -115,9 +115,11 @@ int alpm_release()
handle->db_local = NULL;
}
/* and also sync ones */
- for(i = handle->dbs_sync; i; i = i->next) {
- alpm_db_unregister(i->data);
- i->data = NULL;
+ while((dbs_left = alpm_list_count(handle->dbs_sync)) > 0) {
+ pmdb_t *db = (pmdb_t *)handle->dbs_sync->data;
+ _alpm_log(PM_LOG_DEBUG, _("removing DB %s, %d remaining..."), db->treename, dbs_left);
+ alpm_db_unregister(db);
+ db = NULL;
}
FREEHANDLE(handle);