Mercurial > pidgin
changeset 8770:7df86465ec82
[gaim-migrate @ 9531]
using a pointer before checking it isn't exactly a good thing. nosnilmot
caught this.
committer: Tailor Script <tailor@pidgin.im>
| author | Luke Schierer <lschiere@pidgin.im> |
|---|---|
| date | Fri, 23 Apr 2004 15:02:39 +0000 |
| parents | 0bde91a2ea4d |
| children | 521de634de78 |
| files | src/protocols/novell/nmcontact.c |
| diffstat | 1 files changed, 3 insertions(+), 1 deletions(-) [+] |
line wrap: on
line diff
--- a/src/protocols/novell/nmcontact.c Fri Apr 23 06:14:15 2004 +0000 +++ b/src/protocols/novell/nmcontact.c Fri Apr 23 15:02:39 2004 +0000 @@ -701,11 +701,13 @@ void nm_folder_remove_contact(NMFolder * folder, NMContact * contact) { - GSList *node = folder->contacts; + GSList *node; if (folder == NULL || contact == NULL) return; + node = folder->contacts; + while (node) { if (contact->id == ((NMContact *) (node->data))->id) { folder->contacts = g_slist_remove(folder->contacts, node->data);