Mercurial > pidgin.yaz
comparison libgaim/dnsquery.c @ 14510:1d73e11d8787
[gaim-migrate @ 17230]
Evan, see if this fixes the MSN get info crashes for you. Basically
canceled DNS requests weren't being removed from the queue, then Gaim
would pop them off the queue, but since the request had been canceled
the memory was free'd and is now invalid.
committer: Tailor Script <tailor@pidgin.im>
| author | Mark Doliner <mark@kingant.net> |
|---|---|
| date | Sun, 10 Sep 2006 22:33:28 +0000 |
| parents | 5a09940b88f7 |
| children | 4ee6553079b5 |
comparison
equal
deleted
inserted
replaced
| 14509:ebbcd0ad2473 | 14510:1d73e11d8787 |
|---|---|
| 849 | 849 |
| 850 void | 850 void |
| 851 gaim_dnsquery_destroy(GaimDnsQueryData *query_data) | 851 gaim_dnsquery_destroy(GaimDnsQueryData *query_data) |
| 852 { | 852 { |
| 853 #if defined(__unix__) || defined(__APPLE__) | 853 #if defined(__unix__) || defined(__APPLE__) |
| 854 g_queue_remove(queued_requests, query_data); | |
| 855 | |
| 854 if (query_data->resolver != NULL) | 856 if (query_data->resolver != NULL) |
| 855 /* | 857 /* |
| 856 * Ideally we would tell our resolver child to stop resolving | 858 * Ideally we would tell our resolver child to stop resolving |
| 857 * shit and then we would add it back to the free_dns_children | 859 * shit and then we would add it back to the free_dns_children |
| 858 * linked list. However, it's hard to tell children stuff, | 860 * linked list. However, it's hard to tell children stuff, |