Thu, 02 Jul 2009 04:26:00 +0000
Increase the logging level of some debugging messages that seemed to be a
bit more important than "informational". From a patch by Mauro but with
additional changes recommended by rekkanoryo.
Fixes #8751.
committer: Elliott Sales de Andrade <qulogic@pidgin.im>
| 14243 | 1 | /** |
| 2 | * @file dnsquery.c DNS query API | |
| 3 | * @ingroup core | |
|
20147
66f05a854eee
applied changes from 8a731bbd0197fbcc91a705c2d8f528154216defa
Richard Laager <rlaager@pidgin.im>
parents:
19859
diff
changeset
|
4 | */ |
|
66f05a854eee
applied changes from 8a731bbd0197fbcc91a705c2d8f528154216defa
Richard Laager <rlaager@pidgin.im>
parents:
19859
diff
changeset
|
5 | |
|
66f05a854eee
applied changes from 8a731bbd0197fbcc91a705c2d8f528154216defa
Richard Laager <rlaager@pidgin.im>
parents:
19859
diff
changeset
|
6 | /* purple |
| 14243 | 7 | * |
| 15884 | 8 | * Purple is the legal property of its developers, whose names are too numerous |
| 14243 | 9 | * to list here. Please refer to the COPYRIGHT file distributed with this |
| 10 | * source distribution. | |
| 11 | * | |
| 12 | * This program is free software; you can redistribute it and/or modify | |
| 13 | * it under the terms of the GNU General Public License as published by | |
| 14 | * the Free Software Foundation; either version 2 of the License, or | |
| 15 | * (at your option) any later version. | |
| 16 | * | |
| 17 | * This program is distributed in the hope that it will be useful, | |
| 18 | * but WITHOUT ANY WARRANTY; without even the implied warranty of | |
| 19 | * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the | |
| 20 | * GNU General Public License for more details. | |
| 21 | * | |
| 22 | * You should have received a copy of the GNU General Public License | |
| 23 | * along with this program; if not, write to the Free Software | |
|
19859
71d37b57eff2
The FSF changed its address a while ago; our files were out of date.
John Bailey <rekkanoryo@rekkanoryo.org>
parents:
16701
diff
changeset
|
24 | * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02111-1301 USA |
| 14243 | 25 | * |
| 26 | */ | |
| 27 | ||
| 28 | #include "internal.h" | |
| 29 | #include "debug.h" | |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
30 | #include "dnsquery.h" |
| 14243 | 31 | #include "notify.h" |
| 32 | #include "prefs.h" | |
| 33 | #include "util.h" | |
| 34 | ||
|
27157
609f187d0ac7
Call res_init() when DNS queries fail in case the failure indicates a timeout
Paul Aurich <darkrain42@pidgin.im>
parents:
27147
diff
changeset
|
35 | #ifndef _WIN32 |
|
609f187d0ac7
Call res_init() when DNS queries fail in case the failure indicates a timeout
Paul Aurich <darkrain42@pidgin.im>
parents:
27147
diff
changeset
|
36 | #include <resolv.h> |
|
609f187d0ac7
Call res_init() when DNS queries fail in case the failure indicates a timeout
Paul Aurich <darkrain42@pidgin.im>
parents:
27147
diff
changeset
|
37 | #endif |
|
609f187d0ac7
Call res_init() when DNS queries fail in case the failure indicates a timeout
Paul Aurich <darkrain42@pidgin.im>
parents:
27147
diff
changeset
|
38 | |
|
23352
9714223ca7a7
A patch from Peter O'Gorman at The Written Word, Inc. to fix various
Peter O'Gorman <pogma@thewrittenword.com>
parents:
22622
diff
changeset
|
39 | #if (defined(__APPLE__) || defined (__unix__)) && !defined(__osf__) |
|
9714223ca7a7
A patch from Peter O'Gorman at The Written Word, Inc. to fix various
Peter O'Gorman <pogma@thewrittenword.com>
parents:
22622
diff
changeset
|
40 | #define PURPLE_DNSQUERY_USE_FORK |
|
9714223ca7a7
A patch from Peter O'Gorman at The Written Word, Inc. to fix various
Peter O'Gorman <pogma@thewrittenword.com>
parents:
22622
diff
changeset
|
41 | #endif |
| 14243 | 42 | /************************************************************************** |
| 43 | * DNS query API | |
| 44 | **************************************************************************/ | |
| 45 | ||
| 15884 | 46 | static PurpleDnsQueryUiOps *dns_query_ui_ops = NULL; |
|
15763
2b529e4350bc
The UI can now use the GaimDnsQueryUiOps to handle DNS resolving itself; if this is done, the platform-specific DNS query code won't be used. This will be used in Adium to use an efficient threads-based DNS resolve (since Adium is already multithreaded, this is significantly cheaper than fork()).
Evan Schoenberg <evands@pidgin.im>
parents:
15707
diff
changeset
|
47 | |
| 15884 | 48 | typedef struct _PurpleDnsQueryResolverProcess PurpleDnsQueryResolverProcess; |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
49 | |
| 15884 | 50 | struct _PurpleDnsQueryData { |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
51 | char *hostname; |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
52 | int port; |
| 15884 | 53 | PurpleDnsQueryConnectFunction callback; |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
54 | gpointer data; |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
55 | guint timeout; |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
56 | |
|
23352
9714223ca7a7
A patch from Peter O'Gorman at The Written Word, Inc. to fix various
Peter O'Gorman <pogma@thewrittenword.com>
parents:
22622
diff
changeset
|
57 | #if defined(PURPLE_DNSQUERY_USE_FORK) |
| 15884 | 58 | PurpleDnsQueryResolverProcess *resolver; |
|
23352
9714223ca7a7
A patch from Peter O'Gorman at The Written Word, Inc. to fix various
Peter O'Gorman <pogma@thewrittenword.com>
parents:
22622
diff
changeset
|
59 | #elif defined _WIN32 /* end PURPLE_DNSQUERY_USE_FORK */ |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
60 | GThread *resolver; |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
61 | GSList *hosts; |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
62 | gchar *error_message; |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
63 | #endif |
|
14245
8a215625c6cb
[gaim-migrate @ 16835]
Mark Doliner <markdoliner@pidgin.im>
parents:
14244
diff
changeset
|
64 | }; |
|
8a215625c6cb
[gaim-migrate @ 16835]
Mark Doliner <markdoliner@pidgin.im>
parents:
14244
diff
changeset
|
65 | |
|
23352
9714223ca7a7
A patch from Peter O'Gorman at The Written Word, Inc. to fix various
Peter O'Gorman <pogma@thewrittenword.com>
parents:
22622
diff
changeset
|
66 | #if defined(PURPLE_DNSQUERY_USE_FORK) |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
67 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
68 | #define MAX_DNS_CHILDREN 4 |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
69 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
70 | /* |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
71 | * This structure keeps a reference to a child resolver process. |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
72 | */ |
| 15884 | 73 | struct _PurpleDnsQueryResolverProcess { |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
74 | guint inpa; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
75 | int fd_in, fd_out; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
76 | pid_t dns_pid; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
77 | }; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
78 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
79 | static GSList *free_dns_children = NULL; |
|
14711
404c5a931bb2
[gaim-migrate @ 17395]
Mark Doliner <markdoliner@pidgin.im>
parents:
14572
diff
changeset
|
80 | static GSList *queued_requests = NULL; |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
81 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
82 | static int number_of_dns_children = 0; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
83 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
84 | /* |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
85 | * This is a convenience struct used to pass data to |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
86 | * the child resolver process. |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
87 | */ |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
88 | typedef struct { |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
89 | char hostname[512]; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
90 | int port; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
91 | } dns_params_t; |
|
24321
0d6d024049eb
Improve a few comments
Mark Doliner <markdoliner@pidgin.im>
parents:
24318
diff
changeset
|
92 | #endif /* end PURPLE_DNSQUERY_USE_FORK */ |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
93 | |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
94 | static void |
| 15884 | 95 | purple_dnsquery_resolved(PurpleDnsQueryData *query_data, GSList *hosts) |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
96 | { |
| 15884 | 97 | purple_debug_info("dnsquery", "IP resolved for %s\n", query_data->hostname); |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
98 | if (query_data->callback != NULL) |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
99 | query_data->callback(hosts, query_data->data, NULL); |
|
14420
3c78e468b45a
[gaim-migrate @ 17064]
Mark Doliner <markdoliner@pidgin.im>
parents:
14349
diff
changeset
|
100 | else |
|
3c78e468b45a
[gaim-migrate @ 17064]
Mark Doliner <markdoliner@pidgin.im>
parents:
14349
diff
changeset
|
101 | { |
|
3c78e468b45a
[gaim-migrate @ 17064]
Mark Doliner <markdoliner@pidgin.im>
parents:
14349
diff
changeset
|
102 | /* |
|
3c78e468b45a
[gaim-migrate @ 17064]
Mark Doliner <markdoliner@pidgin.im>
parents:
14349
diff
changeset
|
103 | * Callback is a required parameter, but it can get set to |
|
3c78e468b45a
[gaim-migrate @ 17064]
Mark Doliner <markdoliner@pidgin.im>
parents:
14349
diff
changeset
|
104 | * NULL if we cancel a thread-based DNS lookup. So we need |
|
3c78e468b45a
[gaim-migrate @ 17064]
Mark Doliner <markdoliner@pidgin.im>
parents:
14349
diff
changeset
|
105 | * to free hosts. |
|
3c78e468b45a
[gaim-migrate @ 17064]
Mark Doliner <markdoliner@pidgin.im>
parents:
14349
diff
changeset
|
106 | */ |
|
3c78e468b45a
[gaim-migrate @ 17064]
Mark Doliner <markdoliner@pidgin.im>
parents:
14349
diff
changeset
|
107 | while (hosts != NULL) |
|
3c78e468b45a
[gaim-migrate @ 17064]
Mark Doliner <markdoliner@pidgin.im>
parents:
14349
diff
changeset
|
108 | { |
|
3c78e468b45a
[gaim-migrate @ 17064]
Mark Doliner <markdoliner@pidgin.im>
parents:
14349
diff
changeset
|
109 | hosts = g_slist_remove(hosts, hosts->data); |
|
3c78e468b45a
[gaim-migrate @ 17064]
Mark Doliner <markdoliner@pidgin.im>
parents:
14349
diff
changeset
|
110 | g_free(hosts->data); |
|
3c78e468b45a
[gaim-migrate @ 17064]
Mark Doliner <markdoliner@pidgin.im>
parents:
14349
diff
changeset
|
111 | hosts = g_slist_remove(hosts, hosts->data); |
|
3c78e468b45a
[gaim-migrate @ 17064]
Mark Doliner <markdoliner@pidgin.im>
parents:
14349
diff
changeset
|
112 | } |
|
3c78e468b45a
[gaim-migrate @ 17064]
Mark Doliner <markdoliner@pidgin.im>
parents:
14349
diff
changeset
|
113 | } |
|
3c78e468b45a
[gaim-migrate @ 17064]
Mark Doliner <markdoliner@pidgin.im>
parents:
14349
diff
changeset
|
114 | |
|
24325
9b63613e82ab
Fix DNS resolution. I think I broke this this morning in my revision
Mark Doliner <markdoliner@pidgin.im>
parents:
24324
diff
changeset
|
115 | #ifdef PURPLE_DNSQUERY_USE_FORK |
|
24318
b2b1a42a7816
Revert my revision 3777b91f70ab86e67c1e039d08fb6d87e06f20d6, which
Mark Doliner <markdoliner@pidgin.im>
parents:
24287
diff
changeset
|
116 | /* |
|
24325
9b63613e82ab
Fix DNS resolution. I think I broke this this morning in my revision
Mark Doliner <markdoliner@pidgin.im>
parents:
24324
diff
changeset
|
117 | * Add the resolver to the list of available resolvers, and set it |
|
9b63613e82ab
Fix DNS resolution. I think I broke this this morning in my revision
Mark Doliner <markdoliner@pidgin.im>
parents:
24324
diff
changeset
|
118 | * to NULL so that it doesn't get destroyed along with the query_data |
|
24318
b2b1a42a7816
Revert my revision 3777b91f70ab86e67c1e039d08fb6d87e06f20d6, which
Mark Doliner <markdoliner@pidgin.im>
parents:
24287
diff
changeset
|
119 | */ |
|
24325
9b63613e82ab
Fix DNS resolution. I think I broke this this morning in my revision
Mark Doliner <markdoliner@pidgin.im>
parents:
24324
diff
changeset
|
120 | free_dns_children = g_slist_prepend(free_dns_children, query_data->resolver); |
|
24318
b2b1a42a7816
Revert my revision 3777b91f70ab86e67c1e039d08fb6d87e06f20d6, which
Mark Doliner <markdoliner@pidgin.im>
parents:
24287
diff
changeset
|
121 | query_data->resolver = NULL; |
|
24325
9b63613e82ab
Fix DNS resolution. I think I broke this this morning in my revision
Mark Doliner <markdoliner@pidgin.im>
parents:
24324
diff
changeset
|
122 | #endif /* PURPLE_DNSQUERY_USE_FORK */ |
|
24318
b2b1a42a7816
Revert my revision 3777b91f70ab86e67c1e039d08fb6d87e06f20d6, which
Mark Doliner <markdoliner@pidgin.im>
parents:
24287
diff
changeset
|
123 | |
| 15884 | 124 | purple_dnsquery_destroy(query_data); |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
125 | } |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
126 | |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
127 | static void |
| 15884 | 128 | purple_dnsquery_failed(PurpleDnsQueryData *query_data, const gchar *error_message) |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
129 | { |
|
27497
2dc5a0e0c4c2
Increase the logging level of some debugging messages that seemed to be a
Mauro Brasil <mauro.brasil@tqi.com.br>
parents:
27157
diff
changeset
|
130 | purple_debug_error("dnsquery", "%s\n", error_message); |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
131 | if (query_data->callback != NULL) |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
132 | query_data->callback(NULL, query_data->data, error_message); |
| 15884 | 133 | purple_dnsquery_destroy(query_data); |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
134 | } |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
135 | |
|
15763
2b529e4350bc
The UI can now use the GaimDnsQueryUiOps to handle DNS resolving itself; if this is done, the platform-specific DNS query code won't be used. This will be used in Adium to use an efficient threads-based DNS resolve (since Adium is already multithreaded, this is significantly cheaper than fork()).
Evan Schoenberg <evands@pidgin.im>
parents:
15707
diff
changeset
|
136 | static gboolean |
| 15884 | 137 | purple_dnsquery_ui_resolve(PurpleDnsQueryData *query_data) |
|
15763
2b529e4350bc
The UI can now use the GaimDnsQueryUiOps to handle DNS resolving itself; if this is done, the platform-specific DNS query code won't be used. This will be used in Adium to use an efficient threads-based DNS resolve (since Adium is already multithreaded, this is significantly cheaper than fork()).
Evan Schoenberg <evands@pidgin.im>
parents:
15707
diff
changeset
|
138 | { |
| 15884 | 139 | PurpleDnsQueryUiOps *ops = purple_dnsquery_get_ui_ops(); |
|
15763
2b529e4350bc
The UI can now use the GaimDnsQueryUiOps to handle DNS resolving itself; if this is done, the platform-specific DNS query code won't be used. This will be used in Adium to use an efficient threads-based DNS resolve (since Adium is already multithreaded, this is significantly cheaper than fork()).
Evan Schoenberg <evands@pidgin.im>
parents:
15707
diff
changeset
|
140 | |
|
15775
a1aed6d0c33b
No changes; fixed a bunch of whitespace problems made by a text editor with 'use tabs' disabled. Sorry for the noise.
Evan Schoenberg <evands@pidgin.im>
parents:
15771
diff
changeset
|
141 | if (ops && ops->resolve_host) |
|
24323
fe88d45063e8
Simplify two bits of code a little
Mark Doliner <markdoliner@pidgin.im>
parents:
24322
diff
changeset
|
142 | return ops->resolve_host(query_data, purple_dnsquery_resolved, purple_dnsquery_failed); |
|
15763
2b529e4350bc
The UI can now use the GaimDnsQueryUiOps to handle DNS resolving itself; if this is done, the platform-specific DNS query code won't be used. This will be used in Adium to use an efficient threads-based DNS resolve (since Adium is already multithreaded, this is significantly cheaper than fork()).
Evan Schoenberg <evands@pidgin.im>
parents:
15707
diff
changeset
|
143 | |
|
2b529e4350bc
The UI can now use the GaimDnsQueryUiOps to handle DNS resolving itself; if this is done, the platform-specific DNS query code won't be used. This will be used in Adium to use an efficient threads-based DNS resolve (since Adium is already multithreaded, this is significantly cheaper than fork()).
Evan Schoenberg <evands@pidgin.im>
parents:
15707
diff
changeset
|
144 | return FALSE; |
|
2b529e4350bc
The UI can now use the GaimDnsQueryUiOps to handle DNS resolving itself; if this is done, the platform-specific DNS query code won't be used. This will be used in Adium to use an efficient threads-based DNS resolve (since Adium is already multithreaded, this is significantly cheaper than fork()).
Evan Schoenberg <evands@pidgin.im>
parents:
15707
diff
changeset
|
145 | } |
|
2b529e4350bc
The UI can now use the GaimDnsQueryUiOps to handle DNS resolving itself; if this is done, the platform-specific DNS query code won't be used. This will be used in Adium to use an efficient threads-based DNS resolve (since Adium is already multithreaded, this is significantly cheaper than fork()).
Evan Schoenberg <evands@pidgin.im>
parents:
15707
diff
changeset
|
146 | |
|
23352
9714223ca7a7
A patch from Peter O'Gorman at The Written Word, Inc. to fix various
Peter O'Gorman <pogma@thewrittenword.com>
parents:
22622
diff
changeset
|
147 | #if defined(PURPLE_DNSQUERY_USE_FORK) |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
148 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
149 | /* |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
150 | * Unix! |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
151 | */ |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
152 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
153 | /* |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
154 | * Begin the DNS resolver child process functions. |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
155 | */ |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
156 | #ifdef HAVE_SIGNAL_H |
|
16701
0b41f23396cc
Mark two functions as "noreturn"
Mark Doliner <markdoliner@pidgin.im>
parents:
15890
diff
changeset
|
157 | G_GNUC_NORETURN static void |
|
22108
cb9819851163
Squash some compiler warnings, some from my -Wstrict-prototypes fixing.
Richard Laager <rlaager@pidgin.im>
parents:
22104
diff
changeset
|
158 | trap_gdb_bug(int sig) |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
159 | { |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
160 | const char *message = |
| 15884 | 161 | "Purple's DNS child got a SIGTRAP signal.\n" |
| 162 | "This can be caused by trying to run purple inside gdb.\n" | |
| 163 | "There is a known gdb bug which prevents this. Supposedly purple\n" | |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
164 | "should have detected you were using gdb and used an ugly hack,\n" |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
165 | "check cope_with_gdb_brokenness() in dnsquery.c.\n\n" |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
166 | "For more info about this bug, see http://sources.redhat.com/ml/gdb/2001-07/msg00349.html\n"; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
167 | fputs("\n* * *\n",stderr); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
168 | fputs(message,stderr); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
169 | fputs("* * *\n\n",stderr); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
170 | execlp("xmessage","xmessage","-center", message, NULL); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
171 | _exit(1); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
172 | } |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
173 | #endif |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
174 | |
|
22234
4d54e1c5c696
More compiler warning fixes from o_sukhodolsky (with additions by me).
Mark Doliner <markdoliner@pidgin.im>
parents:
22108
diff
changeset
|
175 | static void |
|
4d54e1c5c696
More compiler warning fixes from o_sukhodolsky (with additions by me).
Mark Doliner <markdoliner@pidgin.im>
parents:
22108
diff
changeset
|
176 | write_to_parent(int fd, const void *buf, size_t count) |
|
4d54e1c5c696
More compiler warning fixes from o_sukhodolsky (with additions by me).
Mark Doliner <markdoliner@pidgin.im>
parents:
22108
diff
changeset
|
177 | { |
|
4d54e1c5c696
More compiler warning fixes from o_sukhodolsky (with additions by me).
Mark Doliner <markdoliner@pidgin.im>
parents:
22108
diff
changeset
|
178 | ssize_t written; |
|
4d54e1c5c696
More compiler warning fixes from o_sukhodolsky (with additions by me).
Mark Doliner <markdoliner@pidgin.im>
parents:
22108
diff
changeset
|
179 | |
|
4d54e1c5c696
More compiler warning fixes from o_sukhodolsky (with additions by me).
Mark Doliner <markdoliner@pidgin.im>
parents:
22108
diff
changeset
|
180 | written = write(fd, buf, count); |
|
4d54e1c5c696
More compiler warning fixes from o_sukhodolsky (with additions by me).
Mark Doliner <markdoliner@pidgin.im>
parents:
22108
diff
changeset
|
181 | if (written != count) { |
|
4d54e1c5c696
More compiler warning fixes from o_sukhodolsky (with additions by me).
Mark Doliner <markdoliner@pidgin.im>
parents:
22108
diff
changeset
|
182 | if (written < 0) |
|
4d54e1c5c696
More compiler warning fixes from o_sukhodolsky (with additions by me).
Mark Doliner <markdoliner@pidgin.im>
parents:
22108
diff
changeset
|
183 | fprintf(stderr, "dns[%d]: Error writing data to " |
|
4d54e1c5c696
More compiler warning fixes from o_sukhodolsky (with additions by me).
Mark Doliner <markdoliner@pidgin.im>
parents:
22108
diff
changeset
|
184 | "parent: %s\n", getpid(), strerror(errno)); |
|
4d54e1c5c696
More compiler warning fixes from o_sukhodolsky (with additions by me).
Mark Doliner <markdoliner@pidgin.im>
parents:
22108
diff
changeset
|
185 | else |
|
4d54e1c5c696
More compiler warning fixes from o_sukhodolsky (with additions by me).
Mark Doliner <markdoliner@pidgin.im>
parents:
22108
diff
changeset
|
186 | fprintf(stderr, "dns[%d]: Error: Tried to write %" |
|
4d54e1c5c696
More compiler warning fixes from o_sukhodolsky (with additions by me).
Mark Doliner <markdoliner@pidgin.im>
parents:
22108
diff
changeset
|
187 | G_GSIZE_FORMAT " bytes to parent but instead " |
|
4d54e1c5c696
More compiler warning fixes from o_sukhodolsky (with additions by me).
Mark Doliner <markdoliner@pidgin.im>
parents:
22108
diff
changeset
|
188 | "wrote %" G_GSIZE_FORMAT " bytes\n", |
|
4d54e1c5c696
More compiler warning fixes from o_sukhodolsky (with additions by me).
Mark Doliner <markdoliner@pidgin.im>
parents:
22108
diff
changeset
|
189 | getpid(), count, written); |
|
4d54e1c5c696
More compiler warning fixes from o_sukhodolsky (with additions by me).
Mark Doliner <markdoliner@pidgin.im>
parents:
22108
diff
changeset
|
190 | } |
|
4d54e1c5c696
More compiler warning fixes from o_sukhodolsky (with additions by me).
Mark Doliner <markdoliner@pidgin.im>
parents:
22108
diff
changeset
|
191 | } |
|
4d54e1c5c696
More compiler warning fixes from o_sukhodolsky (with additions by me).
Mark Doliner <markdoliner@pidgin.im>
parents:
22108
diff
changeset
|
192 | |
|
16701
0b41f23396cc
Mark two functions as "noreturn"
Mark Doliner <markdoliner@pidgin.im>
parents:
15890
diff
changeset
|
193 | G_GNUC_NORETURN static void |
| 15884 | 194 | purple_dnsquery_resolver_run(int child_out, int child_in, gboolean show_debug) |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
195 | { |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
196 | dns_params_t dns_params; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
197 | const size_t zero = 0; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
198 | int rc; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
199 | #ifdef HAVE_GETADDRINFO |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
200 | struct addrinfo hints, *res, *tmp; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
201 | char servname[20]; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
202 | #else |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
203 | struct sockaddr_in sin; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
204 | const size_t addrlen = sizeof(sin); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
205 | #endif |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
206 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
207 | #ifdef HAVE_SIGNAL_H |
| 15884 | 208 | purple_restore_default_signal_handlers(); |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
209 | signal(SIGTRAP, trap_gdb_bug); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
210 | #endif |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
211 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
212 | /* |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
213 | * We resolve 1 host name for each iteration of this |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
214 | * while loop. |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
215 | * |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
216 | * The top half of this reads in the hostname and port |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
217 | * number from the socket with our parent. The bottom |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
218 | * half of this resolves the IP (blocking) and sends |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
219 | * the result back to our parent, when finished. |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
220 | */ |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
221 | while (1) { |
|
24318
b2b1a42a7816
Revert my revision 3777b91f70ab86e67c1e039d08fb6d87e06f20d6, which
Mark Doliner <markdoliner@pidgin.im>
parents:
24287
diff
changeset
|
222 | fd_set fds; |
|
b2b1a42a7816
Revert my revision 3777b91f70ab86e67c1e039d08fb6d87e06f20d6, which
Mark Doliner <markdoliner@pidgin.im>
parents:
24287
diff
changeset
|
223 | struct timeval tv = { .tv_sec = 20, .tv_usec = 0 }; |
|
b2b1a42a7816
Revert my revision 3777b91f70ab86e67c1e039d08fb6d87e06f20d6, which
Mark Doliner <markdoliner@pidgin.im>
parents:
24287
diff
changeset
|
224 | FD_ZERO(&fds); |
|
b2b1a42a7816
Revert my revision 3777b91f70ab86e67c1e039d08fb6d87e06f20d6, which
Mark Doliner <markdoliner@pidgin.im>
parents:
24287
diff
changeset
|
225 | FD_SET(child_in, &fds); |
|
b2b1a42a7816
Revert my revision 3777b91f70ab86e67c1e039d08fb6d87e06f20d6, which
Mark Doliner <markdoliner@pidgin.im>
parents:
24287
diff
changeset
|
226 | rc = select(child_in + 1, &fds, NULL, NULL, &tv); |
|
b2b1a42a7816
Revert my revision 3777b91f70ab86e67c1e039d08fb6d87e06f20d6, which
Mark Doliner <markdoliner@pidgin.im>
parents:
24287
diff
changeset
|
227 | if (!rc) { |
|
b2b1a42a7816
Revert my revision 3777b91f70ab86e67c1e039d08fb6d87e06f20d6, which
Mark Doliner <markdoliner@pidgin.im>
parents:
24287
diff
changeset
|
228 | if (show_debug) |
|
b2b1a42a7816
Revert my revision 3777b91f70ab86e67c1e039d08fb6d87e06f20d6, which
Mark Doliner <markdoliner@pidgin.im>
parents:
24287
diff
changeset
|
229 | printf("dns[%d]: nobody needs me... =(\n", getpid()); |
|
b2b1a42a7816
Revert my revision 3777b91f70ab86e67c1e039d08fb6d87e06f20d6, which
Mark Doliner <markdoliner@pidgin.im>
parents:
24287
diff
changeset
|
230 | break; |
|
b2b1a42a7816
Revert my revision 3777b91f70ab86e67c1e039d08fb6d87e06f20d6, which
Mark Doliner <markdoliner@pidgin.im>
parents:
24287
diff
changeset
|
231 | } |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
232 | rc = read(child_in, &dns_params, sizeof(dns_params_t)); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
233 | if (rc < 0) { |
|
22234
4d54e1c5c696
More compiler warning fixes from o_sukhodolsky (with additions by me).
Mark Doliner <markdoliner@pidgin.im>
parents:
22108
diff
changeset
|
234 | fprintf(stderr, "dns[%d]: Error: Could not read dns_params: " |
|
4d54e1c5c696
More compiler warning fixes from o_sukhodolsky (with additions by me).
Mark Doliner <markdoliner@pidgin.im>
parents:
22108
diff
changeset
|
235 | "%s\n", getpid(), strerror(errno)); |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
236 | break; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
237 | } |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
238 | if (rc == 0) { |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
239 | if (show_debug) |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
240 | printf("dns[%d]: Oops, father has gone, wait for me, wait...!\n", getpid()); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
241 | _exit(0); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
242 | } |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
243 | if (dns_params.hostname[0] == '\0') { |
|
22234
4d54e1c5c696
More compiler warning fixes from o_sukhodolsky (with additions by me).
Mark Doliner <markdoliner@pidgin.im>
parents:
22108
diff
changeset
|
244 | fprintf(stderr, "dns[%d]: Error: Parent requested resolution " |
|
4d54e1c5c696
More compiler warning fixes from o_sukhodolsky (with additions by me).
Mark Doliner <markdoliner@pidgin.im>
parents:
22108
diff
changeset
|
245 | "of an empty hostname (port = %d)!!!\n", getpid(), |
|
4d54e1c5c696
More compiler warning fixes from o_sukhodolsky (with additions by me).
Mark Doliner <markdoliner@pidgin.im>
parents:
22108
diff
changeset
|
246 | dns_params.port); |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
247 | _exit(1); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
248 | } |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
249 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
250 | /* We have the hostname and port, now resolve the IP */ |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
251 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
252 | #ifdef HAVE_GETADDRINFO |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
253 | g_snprintf(servname, sizeof(servname), "%d", dns_params.port); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
254 | memset(&hints, 0, sizeof(hints)); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
255 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
256 | /* This is only used to convert a service |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
257 | * name to a port number. As we know we are |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
258 | * passing a number already, we know this |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
259 | * value will not be really used by the C |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
260 | * library. |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
261 | */ |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
262 | hints.ai_socktype = SOCK_STREAM; |
|
27147
7f09d9398273
Don't do IPv6 AAAA lookups if there is no IPv6 address configured on
Mark Doliner <markdoliner@pidgin.im>
parents:
27116
diff
changeset
|
263 | #ifdef AI_ADDRCONFIG |
|
7f09d9398273
Don't do IPv6 AAAA lookups if there is no IPv6 address configured on
Mark Doliner <markdoliner@pidgin.im>
parents:
27116
diff
changeset
|
264 | hints.ai_flags |= AI_ADDRCONFIG; |
|
7f09d9398273
Don't do IPv6 AAAA lookups if there is no IPv6 address configured on
Mark Doliner <markdoliner@pidgin.im>
parents:
27116
diff
changeset
|
265 | #endif /* AI_ADDRCONFIG */ |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
266 | rc = getaddrinfo(dns_params.hostname, servname, &hints, &res); |
|
22234
4d54e1c5c696
More compiler warning fixes from o_sukhodolsky (with additions by me).
Mark Doliner <markdoliner@pidgin.im>
parents:
22108
diff
changeset
|
267 | write_to_parent(child_out, &rc, sizeof(rc)); |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
268 | if (rc != 0) { |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
269 | close(child_out); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
270 | if (show_debug) |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
271 | printf("dns[%d] Error: getaddrinfo returned %d\n", |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
272 | getpid(), rc); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
273 | dns_params.hostname[0] = '\0'; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
274 | continue; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
275 | } |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
276 | tmp = res; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
277 | while (res) { |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
278 | size_t ai_addrlen = res->ai_addrlen; |
|
22234
4d54e1c5c696
More compiler warning fixes from o_sukhodolsky (with additions by me).
Mark Doliner <markdoliner@pidgin.im>
parents:
22108
diff
changeset
|
279 | write_to_parent(child_out, &ai_addrlen, sizeof(ai_addrlen)); |
|
4d54e1c5c696
More compiler warning fixes from o_sukhodolsky (with additions by me).
Mark Doliner <markdoliner@pidgin.im>
parents:
22108
diff
changeset
|
280 | write_to_parent(child_out, res->ai_addr, res->ai_addrlen); |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
281 | res = res->ai_next; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
282 | } |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
283 | freeaddrinfo(tmp); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
284 | #else |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
285 | if (!inet_aton(dns_params.hostname, &sin.sin_addr)) { |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
286 | struct hostent *hp; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
287 | if (!(hp = gethostbyname(dns_params.hostname))) { |
|
22234
4d54e1c5c696
More compiler warning fixes from o_sukhodolsky (with additions by me).
Mark Doliner <markdoliner@pidgin.im>
parents:
22108
diff
changeset
|
288 | write_to_parent(child_out, &h_errno, sizeof(int)); |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
289 | close(child_out); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
290 | if (show_debug) |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
291 | printf("DNS Error: %d\n", h_errno); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
292 | _exit(0); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
293 | } |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
294 | memset(&sin, 0, sizeof(struct sockaddr_in)); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
295 | memcpy(&sin.sin_addr.s_addr, hp->h_addr, hp->h_length); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
296 | sin.sin_family = hp->h_addrtype; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
297 | } else |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
298 | sin.sin_family = AF_INET; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
299 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
300 | sin.sin_port = htons(dns_params.port); |
|
25360
4315bf325ab4
Ludovico Cavedon noticed that on 64-bit platforms, when HAVE_GETADDRINFO isn't
Daniel Atallah <datallah@pidgin.im>
parents:
24701
diff
changeset
|
301 | rc = 0; |
|
4315bf325ab4
Ludovico Cavedon noticed that on 64-bit platforms, when HAVE_GETADDRINFO isn't
Daniel Atallah <datallah@pidgin.im>
parents:
24701
diff
changeset
|
302 | write_to_parent(child_out, &rc, sizeof(rc)); |
|
22234
4d54e1c5c696
More compiler warning fixes from o_sukhodolsky (with additions by me).
Mark Doliner <markdoliner@pidgin.im>
parents:
22108
diff
changeset
|
303 | write_to_parent(child_out, &addrlen, sizeof(addrlen)); |
|
4d54e1c5c696
More compiler warning fixes from o_sukhodolsky (with additions by me).
Mark Doliner <markdoliner@pidgin.im>
parents:
22108
diff
changeset
|
304 | write_to_parent(child_out, &sin, addrlen); |
|
25360
4315bf325ab4
Ludovico Cavedon noticed that on 64-bit platforms, when HAVE_GETADDRINFO isn't
Daniel Atallah <datallah@pidgin.im>
parents:
24701
diff
changeset
|
305 | #endif |
|
22234
4d54e1c5c696
More compiler warning fixes from o_sukhodolsky (with additions by me).
Mark Doliner <markdoliner@pidgin.im>
parents:
22108
diff
changeset
|
306 | write_to_parent(child_out, &zero, sizeof(zero)); |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
307 | dns_params.hostname[0] = '\0'; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
308 | } |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
309 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
310 | close(child_out); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
311 | close(child_in); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
312 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
313 | _exit(0); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
314 | } |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
315 | /* |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
316 | * End the DNS resolver child process functions. |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
317 | */ |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
318 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
319 | /* |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
320 | * Begin the functions for dealing with the DNS child processes. |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
321 | */ |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
322 | static void |
|
22104
56970903b8e9
Probe for -Wstrict-prototypes to get some more warnings. I then cleaned up
Richard Laager <rlaager@pidgin.im>
parents:
21977
diff
changeset
|
323 | cope_with_gdb_brokenness(void) |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
324 | { |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
325 | #ifdef __linux__ |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
326 | static gboolean already_done = FALSE; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
327 | char s[256], e[512]; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
328 | int n; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
329 | pid_t ppid; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
330 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
331 | if(already_done) |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
332 | return; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
333 | already_done = TRUE; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
334 | ppid = getppid(); |
|
27116
19dff7b3ed0f
A patch from Scott Wolchok to replace snprintf() with g_snprintf() and
Richard Laager <rlaager@pidgin.im>
parents:
25360
diff
changeset
|
335 | g_snprintf(s, sizeof(s), "/proc/%d/exe", ppid); |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
336 | n = readlink(s, e, sizeof(e)); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
337 | if(n < 0) |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
338 | return; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
339 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
340 | e[MIN(n,sizeof(e)-1)] = '\0'; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
341 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
342 | if(strstr(e,"gdb")) { |
| 15884 | 343 | purple_debug_info("dns", |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
344 | "Debugger detected, performing useless query...\n"); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
345 | gethostbyname("x.x.x.x.x"); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
346 | } |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
347 | #endif |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
348 | } |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
349 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
350 | static void |
| 15884 | 351 | purple_dnsquery_resolver_destroy(PurpleDnsQueryResolverProcess *resolver) |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
352 | { |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
353 | g_return_if_fail(resolver != NULL); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
354 | |
|
24701
777f269be1b0
Fix a crash Jacky Lam noted on devel@pidgin.im:
Richard Laager <rlaager@pidgin.im>
parents:
24390
diff
changeset
|
355 | /* Keep this before the kill() call below. */ |
|
777f269be1b0
Fix a crash Jacky Lam noted on devel@pidgin.im:
Richard Laager <rlaager@pidgin.im>
parents:
24390
diff
changeset
|
356 | if (resolver->inpa != 0) { |
|
777f269be1b0
Fix a crash Jacky Lam noted on devel@pidgin.im:
Richard Laager <rlaager@pidgin.im>
parents:
24390
diff
changeset
|
357 | purple_input_remove(resolver->inpa); |
|
777f269be1b0
Fix a crash Jacky Lam noted on devel@pidgin.im:
Richard Laager <rlaager@pidgin.im>
parents:
24390
diff
changeset
|
358 | resolver->inpa = 0; |
|
777f269be1b0
Fix a crash Jacky Lam noted on devel@pidgin.im:
Richard Laager <rlaager@pidgin.im>
parents:
24390
diff
changeset
|
359 | } |
|
777f269be1b0
Fix a crash Jacky Lam noted on devel@pidgin.im:
Richard Laager <rlaager@pidgin.im>
parents:
24390
diff
changeset
|
360 | |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
361 | /* |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
362 | * We might as well attempt to kill our child process. It really |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
363 | * doesn't matter if this fails, because children will expire on |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
364 | * their own after a few seconds. |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
365 | */ |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
366 | if (resolver->dns_pid > 0) |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
367 | kill(resolver->dns_pid, SIGKILL); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
368 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
369 | close(resolver->fd_in); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
370 | close(resolver->fd_out); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
371 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
372 | g_free(resolver); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
373 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
374 | number_of_dns_children--; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
375 | } |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
376 | |
| 15884 | 377 | static PurpleDnsQueryResolverProcess * |
| 378 | purple_dnsquery_resolver_new(gboolean show_debug) | |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
379 | { |
| 15884 | 380 | PurpleDnsQueryResolverProcess *resolver; |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
381 | int child_out[2], child_in[2]; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
382 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
383 | /* Create pipes for communicating with the child process */ |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
384 | if (pipe(child_out) || pipe(child_in)) { |
| 15884 | 385 | purple_debug_error("dns", |
|
21389
e1dd8142bb87
replace most calls to strerror with calls to g_strerror. strerror will return
Nathan Walp <nwalp@pidgin.im>
parents:
20212
diff
changeset
|
386 | "Could not create pipes: %s\n", g_strerror(errno)); |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
387 | return NULL; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
388 | } |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
389 | |
| 15884 | 390 | resolver = g_new(PurpleDnsQueryResolverProcess, 1); |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
391 | resolver->inpa = 0; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
392 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
393 | cope_with_gdb_brokenness(); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
394 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
395 | /* "Go fork and multiply." --Tommy Caldwell (Emily's dad, not the climber) */ |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
396 | resolver->dns_pid = fork(); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
397 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
398 | /* If we are the child process... */ |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
399 | if (resolver->dns_pid == 0) { |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
400 | /* We should not access the parent's side of the pipes, so close them */ |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
401 | close(child_out[0]); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
402 | close(child_in[1]); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
403 | |
| 15884 | 404 | purple_dnsquery_resolver_run(child_out[1], child_in[0], show_debug); |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
405 | /* The thread calls _exit() rather than returning, so we never get here */ |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
406 | } |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
407 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
408 | /* We should not access the child's side of the pipes, so close them */ |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
409 | close(child_out[1]); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
410 | close(child_in[0]); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
411 | if (resolver->dns_pid == -1) { |
| 15884 | 412 | purple_debug_error("dns", |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
413 | "Could not create child process for DNS: %s\n", |
|
21389
e1dd8142bb87
replace most calls to strerror with calls to g_strerror. strerror will return
Nathan Walp <nwalp@pidgin.im>
parents:
20212
diff
changeset
|
414 | g_strerror(errno)); |
| 15884 | 415 | purple_dnsquery_resolver_destroy(resolver); |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
416 | return NULL; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
417 | } |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
418 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
419 | resolver->fd_out = child_out[0]; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
420 | resolver->fd_in = child_in[1]; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
421 | number_of_dns_children++; |
| 15884 | 422 | purple_debug_info("dns", |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
423 | "Created new DNS child %d, there are now %d children.\n", |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
424 | resolver->dns_pid, number_of_dns_children); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
425 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
426 | return resolver; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
427 | } |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
428 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
429 | /** |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
430 | * @return TRUE if the request was sent succesfully. FALSE |
|
24321
0d6d024049eb
Improve a few comments
Mark Doliner <markdoliner@pidgin.im>
parents:
24318
diff
changeset
|
431 | * if the request could not be sent. This isn't |
|
0d6d024049eb
Improve a few comments
Mark Doliner <markdoliner@pidgin.im>
parents:
24318
diff
changeset
|
432 | * necessarily an error. If the child has expired, |
|
0d6d024049eb
Improve a few comments
Mark Doliner <markdoliner@pidgin.im>
parents:
24318
diff
changeset
|
433 | * for example, we won't be able to send the message. |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
434 | */ |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
435 | static gboolean |
| 15884 | 436 | send_dns_request_to_child(PurpleDnsQueryData *query_data, |
| 437 | PurpleDnsQueryResolverProcess *resolver) | |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
438 | { |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
439 | pid_t pid; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
440 | dns_params_t dns_params; |
|
24287
a2e8d72efd34
Fix this g_return_val_if_fail. It should have been returning TRUE or
Mark Doliner <markdoliner@pidgin.im>
parents:
24286
diff
changeset
|
441 | ssize_t rc; |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
442 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
443 | /* This waitpid might return the child's PID if it has recently |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
444 | * exited, or it might return an error if it exited "long |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
445 | * enough" ago that it has already been reaped; in either |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
446 | * instance, we can't use it. */ |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
447 | pid = waitpid(resolver->dns_pid, NULL, WNOHANG); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
448 | if (pid > 0) { |
| 15884 | 449 | purple_debug_warning("dns", "DNS child %d no longer exists\n", |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
450 | resolver->dns_pid); |
| 15884 | 451 | purple_dnsquery_resolver_destroy(resolver); |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
452 | return FALSE; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
453 | } else if (pid < 0) { |
| 15884 | 454 | purple_debug_warning("dns", "Wait for DNS child %d failed: %s\n", |
|
21389
e1dd8142bb87
replace most calls to strerror with calls to g_strerror. strerror will return
Nathan Walp <nwalp@pidgin.im>
parents:
20212
diff
changeset
|
455 | resolver->dns_pid, g_strerror(errno)); |
| 15884 | 456 | purple_dnsquery_resolver_destroy(resolver); |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
457 | return FALSE; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
458 | } |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
459 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
460 | /* Copy the hostname and port into a single data structure */ |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
461 | strncpy(dns_params.hostname, query_data->hostname, sizeof(dns_params.hostname) - 1); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
462 | dns_params.hostname[sizeof(dns_params.hostname) - 1] = '\0'; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
463 | dns_params.port = query_data->port; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
464 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
465 | /* Send the data structure to the child */ |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
466 | rc = write(resolver->fd_in, &dns_params, sizeof(dns_params)); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
467 | if (rc < 0) { |
|
22622
1ecb840b5101
Fix a bunch of compiler warnings caused by my addition of G_GNUC_PRINTF()
Mark Doliner <markdoliner@pidgin.im>
parents:
22234
diff
changeset
|
468 | purple_debug_error("dns", "Unable to write to DNS child %d: %s\n", |
|
21389
e1dd8142bb87
replace most calls to strerror with calls to g_strerror. strerror will return
Nathan Walp <nwalp@pidgin.im>
parents:
20212
diff
changeset
|
469 | resolver->dns_pid, g_strerror(errno)); |
| 15884 | 470 | purple_dnsquery_resolver_destroy(resolver); |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
471 | return FALSE; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
472 | } |
|
24287
a2e8d72efd34
Fix this g_return_val_if_fail. It should have been returning TRUE or
Mark Doliner <markdoliner@pidgin.im>
parents:
24286
diff
changeset
|
473 | if (rc < sizeof(dns_params)) { |
|
24390
24755e1e9f11
Fix this debug message. Thanks to fledermaus who pointed it out.
Mark Doliner <markdoliner@pidgin.im>
parents:
24325
diff
changeset
|
474 | purple_debug_error("dns", "Tried to write %" G_GSSIZE_FORMAT |
|
24755e1e9f11
Fix this debug message. Thanks to fledermaus who pointed it out.
Mark Doliner <markdoliner@pidgin.im>
parents:
24325
diff
changeset
|
475 | " bytes to child but only wrote %" G_GSSIZE_FORMAT "\n", |
|
24287
a2e8d72efd34
Fix this g_return_val_if_fail. It should have been returning TRUE or
Mark Doliner <markdoliner@pidgin.im>
parents:
24286
diff
changeset
|
476 | sizeof(dns_params), rc); |
|
a2e8d72efd34
Fix this g_return_val_if_fail. It should have been returning TRUE or
Mark Doliner <markdoliner@pidgin.im>
parents:
24286
diff
changeset
|
477 | purple_dnsquery_resolver_destroy(resolver); |
|
a2e8d72efd34
Fix this g_return_val_if_fail. It should have been returning TRUE or
Mark Doliner <markdoliner@pidgin.im>
parents:
24286
diff
changeset
|
478 | return FALSE; |
|
a2e8d72efd34
Fix this g_return_val_if_fail. It should have been returning TRUE or
Mark Doliner <markdoliner@pidgin.im>
parents:
24286
diff
changeset
|
479 | } |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
480 | |
| 15884 | 481 | purple_debug_info("dns", |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
482 | "Successfully sent DNS request to child %d\n", |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
483 | resolver->dns_pid); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
484 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
485 | query_data->resolver = resolver; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
486 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
487 | return TRUE; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
488 | } |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
489 | |
| 15884 | 490 | static void host_resolved(gpointer data, gint source, PurpleInputCondition cond); |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
491 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
492 | static void |
|
22104
56970903b8e9
Probe for -Wstrict-prototypes to get some more warnings. I then cleaned up
Richard Laager <rlaager@pidgin.im>
parents:
21977
diff
changeset
|
493 | handle_next_queued_request(void) |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
494 | { |
| 15884 | 495 | PurpleDnsQueryData *query_data; |
| 496 | PurpleDnsQueryResolverProcess *resolver; | |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
497 | |
|
14711
404c5a931bb2
[gaim-migrate @ 17395]
Mark Doliner <markdoliner@pidgin.im>
parents:
14572
diff
changeset
|
498 | if (queued_requests == NULL) |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
499 | /* No more DNS queries, yay! */ |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
500 | return; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
501 | |
|
14711
404c5a931bb2
[gaim-migrate @ 17395]
Mark Doliner <markdoliner@pidgin.im>
parents:
14572
diff
changeset
|
502 | query_data = queued_requests->data; |
|
404c5a931bb2
[gaim-migrate @ 17395]
Mark Doliner <markdoliner@pidgin.im>
parents:
14572
diff
changeset
|
503 | queued_requests = g_slist_delete_link(queued_requests, queued_requests); |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
504 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
505 | /* |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
506 | * If we have any children, attempt to have them perform the DNS |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
507 | * query. If we're able to send the query then resolver will be |
| 15884 | 508 | * set to the PurpleDnsQueryResolverProcess. Otherwise, resolver |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
509 | * will be NULL and we'll need to create a new DNS request child. |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
510 | */ |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
511 | while (free_dns_children != NULL) |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
512 | { |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
513 | resolver = free_dns_children->data; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
514 | free_dns_children = g_slist_remove(free_dns_children, resolver); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
515 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
516 | if (send_dns_request_to_child(query_data, resolver)) |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
517 | /* We found an acceptable child, yay */ |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
518 | break; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
519 | } |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
520 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
521 | /* We need to create a new DNS request child */ |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
522 | if (query_data->resolver == NULL) |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
523 | { |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
524 | if (number_of_dns_children >= MAX_DNS_CHILDREN) |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
525 | { |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
526 | /* Apparently all our children are busy */ |
|
14711
404c5a931bb2
[gaim-migrate @ 17395]
Mark Doliner <markdoliner@pidgin.im>
parents:
14572
diff
changeset
|
527 | queued_requests = g_slist_prepend(queued_requests, query_data); |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
528 | return; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
529 | } |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
530 | |
| 15884 | 531 | resolver = purple_dnsquery_resolver_new(purple_debug_is_enabled()); |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
532 | if (resolver == NULL) |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
533 | { |
| 15884 | 534 | purple_dnsquery_failed(query_data, _("Unable to create new resolver process\n")); |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
535 | return; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
536 | } |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
537 | if (!send_dns_request_to_child(query_data, resolver)) |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
538 | { |
| 15884 | 539 | purple_dnsquery_failed(query_data, _("Unable to send request to resolver process\n")); |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
540 | return; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
541 | } |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
542 | } |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
543 | |
| 15884 | 544 | query_data->resolver->inpa = purple_input_add(query_data->resolver->fd_out, |
| 545 | PURPLE_INPUT_READ, host_resolved, query_data); | |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
546 | } |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
547 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
548 | /* |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
549 | * End the functions for dealing with the DNS child processes. |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
550 | */ |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
551 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
552 | static void |
| 15884 | 553 | host_resolved(gpointer data, gint source, PurpleInputCondition cond) |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
554 | { |
| 15884 | 555 | PurpleDnsQueryData *query_data; |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
556 | int rc, err; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
557 | GSList *hosts = NULL; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
558 | struct sockaddr *addr = NULL; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
559 | size_t addrlen; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
560 | char message[1024]; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
561 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
562 | query_data = data; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
563 | |
| 15884 | 564 | purple_debug_info("dns", "Got response for '%s'\n", query_data->hostname); |
| 565 | purple_input_remove(query_data->resolver->inpa); | |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
566 | query_data->resolver->inpa = 0; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
567 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
568 | rc = read(query_data->resolver->fd_out, &err, sizeof(err)); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
569 | if ((rc == 4) && (err != 0)) |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
570 | { |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
571 | #ifdef HAVE_GETADDRINFO |
|
14454
df290cda81a3
[gaim-migrate @ 17099]
Mark Doliner <markdoliner@pidgin.im>
parents:
14420
diff
changeset
|
572 | g_snprintf(message, sizeof(message), _("Error resolving %s:\n%s"), |
|
21977
c87889698afb
Create a purple_gai_strerror() function similar to g_str_error() which
Mark Doliner <markdoliner@pidgin.im>
parents:
21389
diff
changeset
|
573 | query_data->hostname, purple_gai_strerror(err)); |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
574 | #else |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
575 | g_snprintf(message, sizeof(message), _("Error resolving %s: %d"), |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
576 | query_data->hostname, err); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
577 | #endif |
|
27157
609f187d0ac7
Call res_init() when DNS queries fail in case the failure indicates a timeout
Paul Aurich <darkrain42@pidgin.im>
parents:
27147
diff
changeset
|
578 | /* Re-read resolv.conf and friends in case DNS servers have changed */ |
|
609f187d0ac7
Call res_init() when DNS queries fail in case the failure indicates a timeout
Paul Aurich <darkrain42@pidgin.im>
parents:
27147
diff
changeset
|
579 | res_init(); |
|
609f187d0ac7
Call res_init() when DNS queries fail in case the failure indicates a timeout
Paul Aurich <darkrain42@pidgin.im>
parents:
27147
diff
changeset
|
580 | |
| 15884 | 581 | purple_dnsquery_failed(query_data, message); |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
582 | } else if (rc > 0) { |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
583 | /* Success! */ |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
584 | while (rc > 0) { |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
585 | rc = read(query_data->resolver->fd_out, &addrlen, sizeof(addrlen)); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
586 | if (rc > 0 && addrlen > 0) { |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
587 | addr = g_malloc(addrlen); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
588 | rc = read(query_data->resolver->fd_out, addr, addrlen); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
589 | hosts = g_slist_append(hosts, GINT_TO_POINTER(addrlen)); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
590 | hosts = g_slist_append(hosts, addr); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
591 | } else { |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
592 | break; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
593 | } |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
594 | } |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
595 | /* wait4(resolver->dns_pid, NULL, WNOHANG, NULL); */ |
| 15884 | 596 | purple_dnsquery_resolved(query_data, hosts); |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
597 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
598 | } else if (rc == -1) { |
|
21389
e1dd8142bb87
replace most calls to strerror with calls to g_strerror. strerror will return
Nathan Walp <nwalp@pidgin.im>
parents:
20212
diff
changeset
|
599 | g_snprintf(message, sizeof(message), _("Error reading from resolver process:\n%s"), g_strerror(errno)); |
| 15884 | 600 | purple_dnsquery_failed(query_data, message); |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
601 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
602 | } else if (rc == 0) { |
|
24322
05dd0a880a44
Change a string to be more user friendly (people shouldn't have any idea
Mark Doliner <markdoliner@pidgin.im>
parents:
24321
diff
changeset
|
603 | g_snprintf(message, sizeof(message), _("Resolver process exited without answering our request")); |
| 15884 | 604 | purple_dnsquery_failed(query_data, message); |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
605 | } |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
606 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
607 | handle_next_queued_request(); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
608 | } |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
609 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
610 | static gboolean |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
611 | resolve_host(gpointer data) |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
612 | { |
| 15884 | 613 | PurpleDnsQueryData *query_data; |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
614 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
615 | query_data = data; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
616 | query_data->timeout = 0; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
617 | |
|
24324
a36c5d08a43c
There was a little weirdness with the code that called the UI function
Mark Doliner <markdoliner@pidgin.im>
parents:
24323
diff
changeset
|
618 | if (purple_dnsquery_ui_resolve(query_data)) |
|
a36c5d08a43c
There was a little weirdness with the code that called the UI function
Mark Doliner <markdoliner@pidgin.im>
parents:
24323
diff
changeset
|
619 | { |
|
a36c5d08a43c
There was a little weirdness with the code that called the UI function
Mark Doliner <markdoliner@pidgin.im>
parents:
24323
diff
changeset
|
620 | /* The UI is handling the resolve; we're done */ |
|
a36c5d08a43c
There was a little weirdness with the code that called the UI function
Mark Doliner <markdoliner@pidgin.im>
parents:
24323
diff
changeset
|
621 | return FALSE; |
|
a36c5d08a43c
There was a little weirdness with the code that called the UI function
Mark Doliner <markdoliner@pidgin.im>
parents:
24323
diff
changeset
|
622 | } |
|
a36c5d08a43c
There was a little weirdness with the code that called the UI function
Mark Doliner <markdoliner@pidgin.im>
parents:
24323
diff
changeset
|
623 | |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
624 | handle_next_queued_request(); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
625 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
626 | return FALSE; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
627 | } |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
628 | |
| 15884 | 629 | PurpleDnsQueryData * |
| 630 | purple_dnsquery_a(const char *hostname, int port, | |
| 631 | PurpleDnsQueryConnectFunction callback, gpointer data) | |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
632 | { |
| 15884 | 633 | PurpleDnsQueryData *query_data; |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
634 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
635 | g_return_val_if_fail(hostname != NULL, NULL); |
|
15775
a1aed6d0c33b
No changes; fixed a bunch of whitespace problems made by a text editor with 'use tabs' disabled. Sorry for the noise.
Evan Schoenberg <evands@pidgin.im>
parents:
15771
diff
changeset
|
636 | g_return_val_if_fail(port != 0, NULL); |
|
14420
3c78e468b45a
[gaim-migrate @ 17064]
Mark Doliner <markdoliner@pidgin.im>
parents:
14349
diff
changeset
|
637 | g_return_val_if_fail(callback != NULL, NULL); |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
638 | |
| 15884 | 639 | query_data = g_new(PurpleDnsQueryData, 1); |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
640 | query_data->hostname = g_strdup(hostname); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
641 | g_strstrip(query_data->hostname); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
642 | query_data->port = port; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
643 | query_data->callback = callback; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
644 | query_data->data = data; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
645 | query_data->resolver = NULL; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
646 | |
|
24323
fe88d45063e8
Simplify two bits of code a little
Mark Doliner <markdoliner@pidgin.im>
parents:
24322
diff
changeset
|
647 | if (*query_data->hostname == '\0') |
|
14528
b8e4715b2953
[gaim-migrate @ 17180]
Mark Doliner <markdoliner@pidgin.im>
parents:
14454
diff
changeset
|
648 | { |
| 15884 | 649 | purple_dnsquery_destroy(query_data); |
|
14528
b8e4715b2953
[gaim-migrate @ 17180]
Mark Doliner <markdoliner@pidgin.im>
parents:
14454
diff
changeset
|
650 | g_return_val_if_reached(NULL); |
|
b8e4715b2953
[gaim-migrate @ 17180]
Mark Doliner <markdoliner@pidgin.im>
parents:
14454
diff
changeset
|
651 | } |
|
b8e4715b2953
[gaim-migrate @ 17180]
Mark Doliner <markdoliner@pidgin.im>
parents:
14454
diff
changeset
|
652 | |
|
14711
404c5a931bb2
[gaim-migrate @ 17395]
Mark Doliner <markdoliner@pidgin.im>
parents:
14572
diff
changeset
|
653 | queued_requests = g_slist_append(queued_requests, query_data); |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
654 | |
| 15884 | 655 | purple_debug_info("dns", "DNS query for '%s' queued\n", query_data->hostname); |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
656 | |
| 15884 | 657 | query_data->timeout = purple_timeout_add(0, resolve_host, query_data); |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
658 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
659 | return query_data; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
660 | } |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
661 | |
|
23352
9714223ca7a7
A patch from Peter O'Gorman at The Written Word, Inc. to fix various
Peter O'Gorman <pogma@thewrittenword.com>
parents:
22622
diff
changeset
|
662 | #elif defined _WIN32 /* end PURPLE_DNSQUERY_USE_FORK */ |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
663 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
664 | /* |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
665 | * Windows! |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
666 | */ |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
667 | |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
668 | static gboolean |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
669 | dns_main_thread_cb(gpointer data) |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
670 | { |
|
20212
9bf576eeaac0
applied changes from 77376fe079e753b612b3b8c901060e689f3dee11
Luke Schierer <lschiere@pidgin.im>
parents:
20147
diff
changeset
|
671 | PurpleDnsQueryData *query_data = data; |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
672 | |
|
20212
9bf576eeaac0
applied changes from 77376fe079e753b612b3b8c901060e689f3dee11
Luke Schierer <lschiere@pidgin.im>
parents:
20147
diff
changeset
|
673 | /* We're done, so purple_dnsquery_destroy() shouldn't think it is canceling an in-progress lookup */ |
|
9bf576eeaac0
applied changes from 77376fe079e753b612b3b8c901060e689f3dee11
Luke Schierer <lschiere@pidgin.im>
parents:
20147
diff
changeset
|
674 | query_data->resolver = NULL; |
| 14243 | 675 | |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
676 | if (query_data->error_message != NULL) |
| 15884 | 677 | purple_dnsquery_failed(query_data, query_data->error_message); |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
678 | else |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
679 | { |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
680 | GSList *hosts; |
|
14306
7c3f2162a897
[gaim-migrate @ 16926]
Mark Doliner <markdoliner@pidgin.im>
parents:
14300
diff
changeset
|
681 | |
| 15884 | 682 | /* We don't want purple_dns_query_resolved() to free(hosts) */ |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
683 | hosts = query_data->hosts; |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
684 | query_data->hosts = NULL; |
| 15884 | 685 | purple_dnsquery_resolved(query_data, hosts); |
| 14243 | 686 | } |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
687 | |
| 14243 | 688 | return FALSE; |
| 689 | } | |
| 690 | ||
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
691 | static gpointer |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
692 | dns_thread(gpointer data) |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
693 | { |
| 15884 | 694 | PurpleDnsQueryData *query_data; |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
695 | #ifdef HAVE_GETADDRINFO |
| 14243 | 696 | int rc; |
| 697 | struct addrinfo hints, *res, *tmp; | |
| 698 | char servname[20]; | |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
699 | #else |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
700 | struct sockaddr_in sin; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
701 | struct hostent *hp; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
702 | #endif |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
703 | |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
704 | query_data = data; |
| 14243 | 705 | |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
706 | #ifdef HAVE_GETADDRINFO |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
707 | g_snprintf(servname, sizeof(servname), "%d", query_data->port); |
| 14243 | 708 | memset(&hints,0,sizeof(hints)); |
| 709 | ||
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
710 | /* |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
711 | * This is only used to convert a service |
| 14243 | 712 | * name to a port number. As we know we are |
| 713 | * passing a number already, we know this | |
| 714 | * value will not be really used by the C | |
| 715 | * library. | |
| 716 | */ | |
| 717 | hints.ai_socktype = SOCK_STREAM; | |
|
27147
7f09d9398273
Don't do IPv6 AAAA lookups if there is no IPv6 address configured on
Mark Doliner <markdoliner@pidgin.im>
parents:
27116
diff
changeset
|
718 | #ifdef AI_ADDRCONFIG |
|
7f09d9398273
Don't do IPv6 AAAA lookups if there is no IPv6 address configured on
Mark Doliner <markdoliner@pidgin.im>
parents:
27116
diff
changeset
|
719 | hints.ai_flags |= AI_ADDRCONFIG; |
|
7f09d9398273
Don't do IPv6 AAAA lookups if there is no IPv6 address configured on
Mark Doliner <markdoliner@pidgin.im>
parents:
27116
diff
changeset
|
720 | #endif /* AI_ADDRCONFIG */ |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
721 | if ((rc = getaddrinfo(query_data->hostname, servname, &hints, &res)) == 0) { |
| 14243 | 722 | tmp = res; |
| 723 | while(res) { | |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
724 | query_data->hosts = g_slist_append(query_data->hosts, |
| 14243 | 725 | GSIZE_TO_POINTER(res->ai_addrlen)); |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
726 | query_data->hosts = g_slist_append(query_data->hosts, |
| 14243 | 727 | g_memdup(res->ai_addr, res->ai_addrlen)); |
| 728 | res = res->ai_next; | |
| 729 | } | |
| 730 | freeaddrinfo(tmp); | |
| 731 | } else { | |
|
21977
c87889698afb
Create a purple_gai_strerror() function similar to g_str_error() which
Mark Doliner <markdoliner@pidgin.im>
parents:
21389
diff
changeset
|
732 | query_data->error_message = g_strdup_printf(_("Error resolving %s:\n%s"), query_data->hostname, purple_gai_strerror(rc)); |
| 14243 | 733 | } |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
734 | #else |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
735 | if ((hp = gethostbyname(query_data->hostname))) { |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
736 | memset(&sin, 0, sizeof(struct sockaddr_in)); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
737 | memcpy(&sin.sin_addr.s_addr, hp->h_addr, hp->h_length); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
738 | sin.sin_family = hp->h_addrtype; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
739 | sin.sin_port = htons(query_data->port); |
| 14243 | 740 | |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
741 | query_data->hosts = g_slist_append(query_data->hosts, |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
742 | GSIZE_TO_POINTER(sizeof(sin))); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
743 | query_data->hosts = g_slist_append(query_data->hosts, |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
744 | g_memdup(&sin, sizeof(sin))); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
745 | } else { |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
746 | query_data->error_message = g_strdup_printf(_("Error resolving %s: %d"), query_data->hostname, h_errno); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
747 | } |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
748 | #endif |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
749 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
750 | /* back to main thread */ |
|
20212
9bf576eeaac0
applied changes from 77376fe079e753b612b3b8c901060e689f3dee11
Luke Schierer <lschiere@pidgin.im>
parents:
20147
diff
changeset
|
751 | purple_timeout_add(0, dns_main_thread_cb, query_data); |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
752 | |
| 14243 | 753 | return 0; |
| 754 | } | |
| 755 | ||
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
756 | static gboolean |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
757 | resolve_host(gpointer data) |
| 14243 | 758 | { |
| 15884 | 759 | PurpleDnsQueryData *query_data; |
| 14243 | 760 | struct sockaddr_in sin; |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
761 | GError *err = NULL; |
| 14243 | 762 | |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
763 | query_data = data; |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
764 | query_data->timeout = 0; |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
765 | |
| 15884 | 766 | if (purple_dnsquery_ui_resolve(query_data)) |
|
15775
a1aed6d0c33b
No changes; fixed a bunch of whitespace problems made by a text editor with 'use tabs' disabled. Sorry for the noise.
Evan Schoenberg <evands@pidgin.im>
parents:
15771
diff
changeset
|
767 | { |
|
a1aed6d0c33b
No changes; fixed a bunch of whitespace problems made by a text editor with 'use tabs' disabled. Sorry for the noise.
Evan Schoenberg <evands@pidgin.im>
parents:
15771
diff
changeset
|
768 | /* The UI is handling the resolve; we're done */ |
|
a1aed6d0c33b
No changes; fixed a bunch of whitespace problems made by a text editor with 'use tabs' disabled. Sorry for the noise.
Evan Schoenberg <evands@pidgin.im>
parents:
15771
diff
changeset
|
769 | return FALSE; |
|
a1aed6d0c33b
No changes; fixed a bunch of whitespace problems made by a text editor with 'use tabs' disabled. Sorry for the noise.
Evan Schoenberg <evands@pidgin.im>
parents:
15771
diff
changeset
|
770 | } |
|
15763
2b529e4350bc
The UI can now use the GaimDnsQueryUiOps to handle DNS resolving itself; if this is done, the platform-specific DNS query code won't be used. This will be used in Adium to use an efficient threads-based DNS resolve (since Adium is already multithreaded, this is significantly cheaper than fork()).
Evan Schoenberg <evands@pidgin.im>
parents:
15707
diff
changeset
|
771 | |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
772 | if (inet_aton(query_data->hostname, &sin.sin_addr)) |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
773 | { |
|
14306
7c3f2162a897
[gaim-migrate @ 16926]
Mark Doliner <markdoliner@pidgin.im>
parents:
14300
diff
changeset
|
774 | /* |
|
7c3f2162a897
[gaim-migrate @ 16926]
Mark Doliner <markdoliner@pidgin.im>
parents:
14300
diff
changeset
|
775 | * The given "hostname" is actually an IP address, so we |
|
7c3f2162a897
[gaim-migrate @ 16926]
Mark Doliner <markdoliner@pidgin.im>
parents:
14300
diff
changeset
|
776 | * don't need to do anything. |
|
7c3f2162a897
[gaim-migrate @ 16926]
Mark Doliner <markdoliner@pidgin.im>
parents:
14300
diff
changeset
|
777 | */ |
| 14243 | 778 | GSList *hosts = NULL; |
| 779 | sin.sin_family = AF_INET; | |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
780 | sin.sin_port = htons(query_data->port); |
| 14243 | 781 | hosts = g_slist_append(hosts, GINT_TO_POINTER(sizeof(sin))); |
| 782 | hosts = g_slist_append(hosts, g_memdup(&sin, sizeof(sin))); | |
| 15884 | 783 | purple_dnsquery_resolved(query_data, hosts); |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
784 | } |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
785 | else |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
786 | { |
|
14306
7c3f2162a897
[gaim-migrate @ 16926]
Mark Doliner <markdoliner@pidgin.im>
parents:
14300
diff
changeset
|
787 | /* |
|
7c3f2162a897
[gaim-migrate @ 16926]
Mark Doliner <markdoliner@pidgin.im>
parents:
14300
diff
changeset
|
788 | * Spin off a separate thread to perform the DNS lookup so |
|
7c3f2162a897
[gaim-migrate @ 16926]
Mark Doliner <markdoliner@pidgin.im>
parents:
14300
diff
changeset
|
789 | * that we don't block the UI. |
|
7c3f2162a897
[gaim-migrate @ 16926]
Mark Doliner <markdoliner@pidgin.im>
parents:
14300
diff
changeset
|
790 | */ |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
791 | query_data->resolver = g_thread_create(dns_thread, |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
792 | query_data, FALSE, &err); |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
793 | if (query_data->resolver == NULL) |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
794 | { |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
795 | char message[1024]; |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
796 | g_snprintf(message, sizeof(message), _("Thread creation failure: %s"), |
| 15859 | 797 | (err && err->message) ? err->message : _("Unknown reason")); |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
798 | g_error_free(err); |
| 15884 | 799 | purple_dnsquery_failed(query_data, message); |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
800 | } |
| 14243 | 801 | } |
| 802 | ||
| 803 | return FALSE; | |
| 804 | } | |
| 805 | ||
| 15884 | 806 | PurpleDnsQueryData * |
| 807 | purple_dnsquery_a(const char *hostname, int port, | |
| 808 | PurpleDnsQueryConnectFunction callback, gpointer data) | |
| 14243 | 809 | { |
| 15884 | 810 | PurpleDnsQueryData *query_data; |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
811 | |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
812 | g_return_val_if_fail(hostname != NULL, NULL); |
|
15775
a1aed6d0c33b
No changes; fixed a bunch of whitespace problems made by a text editor with 'use tabs' disabled. Sorry for the noise.
Evan Schoenberg <evands@pidgin.im>
parents:
15771
diff
changeset
|
813 | g_return_val_if_fail(port != 0, NULL); |
|
14420
3c78e468b45a
[gaim-migrate @ 17064]
Mark Doliner <markdoliner@pidgin.im>
parents:
14349
diff
changeset
|
814 | g_return_val_if_fail(callback != NULL, NULL); |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
815 | |
| 15884 | 816 | purple_debug_info("dnsquery", "Performing DNS lookup for %s\n", hostname); |
|
14306
7c3f2162a897
[gaim-migrate @ 16926]
Mark Doliner <markdoliner@pidgin.im>
parents:
14300
diff
changeset
|
817 | |
|
20212
9bf576eeaac0
applied changes from 77376fe079e753b612b3b8c901060e689f3dee11
Luke Schierer <lschiere@pidgin.im>
parents:
20147
diff
changeset
|
818 | query_data = g_new0(PurpleDnsQueryData, 1); |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
819 | query_data->hostname = g_strdup(hostname); |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
820 | g_strstrip(query_data->hostname); |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
821 | query_data->port = port; |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
822 | query_data->callback = callback; |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
823 | query_data->data = data; |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
824 | |
|
14528
b8e4715b2953
[gaim-migrate @ 17180]
Mark Doliner <markdoliner@pidgin.im>
parents:
14454
diff
changeset
|
825 | if (strlen(query_data->hostname) == 0) |
|
b8e4715b2953
[gaim-migrate @ 17180]
Mark Doliner <markdoliner@pidgin.im>
parents:
14454
diff
changeset
|
826 | { |
| 15884 | 827 | purple_dnsquery_destroy(query_data); |
|
14528
b8e4715b2953
[gaim-migrate @ 17180]
Mark Doliner <markdoliner@pidgin.im>
parents:
14454
diff
changeset
|
828 | g_return_val_if_reached(NULL); |
|
b8e4715b2953
[gaim-migrate @ 17180]
Mark Doliner <markdoliner@pidgin.im>
parents:
14454
diff
changeset
|
829 | } |
|
b8e4715b2953
[gaim-migrate @ 17180]
Mark Doliner <markdoliner@pidgin.im>
parents:
14454
diff
changeset
|
830 | |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
831 | /* Don't call the callback before returning */ |
| 15884 | 832 | query_data->timeout = purple_timeout_add(0, resolve_host, query_data); |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
833 | |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
834 | return query_data; |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
835 | } |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
836 | |
|
23352
9714223ca7a7
A patch from Peter O'Gorman at The Written Word, Inc. to fix various
Peter O'Gorman <pogma@thewrittenword.com>
parents:
22622
diff
changeset
|
837 | #else /* not PURPLE_DNSQUERY_USE_FORK or _WIN32 */ |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
838 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
839 | /* |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
840 | * We weren't able to do anything fancier above, so use the |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
841 | * fail-safe name resolution code, which is blocking. |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
842 | */ |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
843 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
844 | static gboolean |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
845 | resolve_host(gpointer data) |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
846 | { |
| 15884 | 847 | PurpleDnsQueryData *query_data; |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
848 | struct sockaddr_in sin; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
849 | GSList *hosts = NULL; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
850 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
851 | query_data = data; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
852 | query_data->timeout = 0; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
853 | |
| 15884 | 854 | if (purple_dnsquery_ui_resolve(query_data)) |
|
15775
a1aed6d0c33b
No changes; fixed a bunch of whitespace problems made by a text editor with 'use tabs' disabled. Sorry for the noise.
Evan Schoenberg <evands@pidgin.im>
parents:
15771
diff
changeset
|
855 | { |
|
a1aed6d0c33b
No changes; fixed a bunch of whitespace problems made by a text editor with 'use tabs' disabled. Sorry for the noise.
Evan Schoenberg <evands@pidgin.im>
parents:
15771
diff
changeset
|
856 | /* The UI is handling the resolve; we're done */ |
|
a1aed6d0c33b
No changes; fixed a bunch of whitespace problems made by a text editor with 'use tabs' disabled. Sorry for the noise.
Evan Schoenberg <evands@pidgin.im>
parents:
15771
diff
changeset
|
857 | return FALSE; |
|
a1aed6d0c33b
No changes; fixed a bunch of whitespace problems made by a text editor with 'use tabs' disabled. Sorry for the noise.
Evan Schoenberg <evands@pidgin.im>
parents:
15771
diff
changeset
|
858 | } |
|
15763
2b529e4350bc
The UI can now use the GaimDnsQueryUiOps to handle DNS resolving itself; if this is done, the platform-specific DNS query code won't be used. This will be used in Adium to use an efficient threads-based DNS resolve (since Adium is already multithreaded, this is significantly cheaper than fork()).
Evan Schoenberg <evands@pidgin.im>
parents:
15707
diff
changeset
|
859 | |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
860 | if (!inet_aton(query_data->hostname, &sin.sin_addr)) { |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
861 | struct hostent *hp; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
862 | if(!(hp = gethostbyname(query_data->hostname))) { |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
863 | char message[1024]; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
864 | g_snprintf(message, sizeof(message), _("Error resolving %s: %d"), |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
865 | query_data->hostname, h_errno); |
| 15884 | 866 | purple_dnsquery_failed(query_data, message); |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
867 | return FALSE; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
868 | } |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
869 | memset(&sin, 0, sizeof(struct sockaddr_in)); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
870 | memcpy(&sin.sin_addr.s_addr, hp->h_addr, hp->h_length); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
871 | sin.sin_family = hp->h_addrtype; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
872 | } else |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
873 | sin.sin_family = AF_INET; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
874 | sin.sin_port = htons(query_data->port); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
875 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
876 | hosts = g_slist_append(hosts, GINT_TO_POINTER(sizeof(sin))); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
877 | hosts = g_slist_append(hosts, g_memdup(&sin, sizeof(sin))); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
878 | |
| 15884 | 879 | purple_dnsquery_resolved(query_data, hosts); |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
880 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
881 | return FALSE; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
882 | } |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
883 | |
| 15884 | 884 | PurpleDnsQueryData * |
| 885 | purple_dnsquery_a(const char *hostname, int port, | |
| 886 | PurpleDnsQueryConnectFunction callback, gpointer data) | |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
887 | { |
| 15884 | 888 | PurpleDnsQueryData *query_data; |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
889 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
890 | g_return_val_if_fail(hostname != NULL, NULL); |
|
15775
a1aed6d0c33b
No changes; fixed a bunch of whitespace problems made by a text editor with 'use tabs' disabled. Sorry for the noise.
Evan Schoenberg <evands@pidgin.im>
parents:
15771
diff
changeset
|
891 | g_return_val_if_fail(port != 0, NULL); |
|
14528
b8e4715b2953
[gaim-migrate @ 17180]
Mark Doliner <markdoliner@pidgin.im>
parents:
14454
diff
changeset
|
892 | g_return_val_if_fail(callback != NULL, NULL); |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
893 | |
| 15884 | 894 | query_data = g_new(PurpleDnsQueryData, 1); |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
895 | query_data->hostname = g_strdup(hostname); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
896 | g_strstrip(query_data->hostname); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
897 | query_data->port = port; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
898 | query_data->callback = callback; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
899 | query_data->data = data; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
900 | |
|
14528
b8e4715b2953
[gaim-migrate @ 17180]
Mark Doliner <markdoliner@pidgin.im>
parents:
14454
diff
changeset
|
901 | if (strlen(query_data->hostname) == 0) |
|
b8e4715b2953
[gaim-migrate @ 17180]
Mark Doliner <markdoliner@pidgin.im>
parents:
14454
diff
changeset
|
902 | { |
| 15884 | 903 | purple_dnsquery_destroy(query_data); |
|
14528
b8e4715b2953
[gaim-migrate @ 17180]
Mark Doliner <markdoliner@pidgin.im>
parents:
14454
diff
changeset
|
904 | g_return_val_if_reached(NULL); |
|
b8e4715b2953
[gaim-migrate @ 17180]
Mark Doliner <markdoliner@pidgin.im>
parents:
14454
diff
changeset
|
905 | } |
|
b8e4715b2953
[gaim-migrate @ 17180]
Mark Doliner <markdoliner@pidgin.im>
parents:
14454
diff
changeset
|
906 | |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
907 | /* Don't call the callback before returning */ |
| 15884 | 908 | query_data->timeout = purple_timeout_add(0, resolve_host, query_data); |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
909 | |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
910 | return query_data; |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
911 | } |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
912 | |
|
23352
9714223ca7a7
A patch from Peter O'Gorman at The Written Word, Inc. to fix various
Peter O'Gorman <pogma@thewrittenword.com>
parents:
22622
diff
changeset
|
913 | #endif /* not PURPLE_DNSQUERY_USE_FORK or _WIN32 */ |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
914 | |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
915 | void |
| 15884 | 916 | purple_dnsquery_destroy(PurpleDnsQueryData *query_data) |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
917 | { |
| 15884 | 918 | PurpleDnsQueryUiOps *ops = purple_dnsquery_get_ui_ops(); |
|
15763
2b529e4350bc
The UI can now use the GaimDnsQueryUiOps to handle DNS resolving itself; if this is done, the platform-specific DNS query code won't be used. This will be used in Adium to use an efficient threads-based DNS resolve (since Adium is already multithreaded, this is significantly cheaper than fork()).
Evan Schoenberg <evands@pidgin.im>
parents:
15707
diff
changeset
|
919 | |
|
15775
a1aed6d0c33b
No changes; fixed a bunch of whitespace problems made by a text editor with 'use tabs' disabled. Sorry for the noise.
Evan Schoenberg <evands@pidgin.im>
parents:
15771
diff
changeset
|
920 | if (ops && ops->destroy) |
|
a1aed6d0c33b
No changes; fixed a bunch of whitespace problems made by a text editor with 'use tabs' disabled. Sorry for the noise.
Evan Schoenberg <evands@pidgin.im>
parents:
15771
diff
changeset
|
921 | ops->destroy(query_data); |
|
15763
2b529e4350bc
The UI can now use the GaimDnsQueryUiOps to handle DNS resolving itself; if this is done, the platform-specific DNS query code won't be used. This will be used in Adium to use an efficient threads-based DNS resolve (since Adium is already multithreaded, this is significantly cheaper than fork()).
Evan Schoenberg <evands@pidgin.im>
parents:
15707
diff
changeset
|
922 | |
|
23352
9714223ca7a7
A patch from Peter O'Gorman at The Written Word, Inc. to fix various
Peter O'Gorman <pogma@thewrittenword.com>
parents:
22622
diff
changeset
|
923 | #if defined(PURPLE_DNSQUERY_USE_FORK) |
|
14711
404c5a931bb2
[gaim-migrate @ 17395]
Mark Doliner <markdoliner@pidgin.im>
parents:
14572
diff
changeset
|
924 | queued_requests = g_slist_remove(queued_requests, query_data); |
|
14572
80e79a4926b4
[gaim-migrate @ 17230]
Mark Doliner <markdoliner@pidgin.im>
parents:
14528
diff
changeset
|
925 | |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
926 | if (query_data->resolver != NULL) |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
927 | /* |
|
24321
0d6d024049eb
Improve a few comments
Mark Doliner <markdoliner@pidgin.im>
parents:
24318
diff
changeset
|
928 | * This is only non-NULL when we're cancelling an in-progress |
|
0d6d024049eb
Improve a few comments
Mark Doliner <markdoliner@pidgin.im>
parents:
24318
diff
changeset
|
929 | * query. Ideally we would tell our resolver child to stop |
|
0d6d024049eb
Improve a few comments
Mark Doliner <markdoliner@pidgin.im>
parents:
24318
diff
changeset
|
930 | * resolving shit and then we would add it back to the |
|
0d6d024049eb
Improve a few comments
Mark Doliner <markdoliner@pidgin.im>
parents:
24318
diff
changeset
|
931 | * free_dns_children linked list. However, it's hard to tell |
|
0d6d024049eb
Improve a few comments
Mark Doliner <markdoliner@pidgin.im>
parents:
24318
diff
changeset
|
932 | * children stuff, they just don't listen. So we'll just |
|
0d6d024049eb
Improve a few comments
Mark Doliner <markdoliner@pidgin.im>
parents:
24318
diff
changeset
|
933 | * kill the process and allow a new child to be started if we |
|
0d6d024049eb
Improve a few comments
Mark Doliner <markdoliner@pidgin.im>
parents:
24318
diff
changeset
|
934 | * have more stuff to resolve. |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
935 | */ |
| 15884 | 936 | purple_dnsquery_resolver_destroy(query_data->resolver); |
|
23352
9714223ca7a7
A patch from Peter O'Gorman at The Written Word, Inc. to fix various
Peter O'Gorman <pogma@thewrittenword.com>
parents:
22622
diff
changeset
|
937 | #elif defined _WIN32 /* end PURPLE_DNSQUERY_USE_FORK */ |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
938 | if (query_data->resolver != NULL) |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
939 | { |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
940 | /* |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
941 | * It's not really possible to kill a thread. So instead we |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
942 | * just set the callback to NULL and let the DNS lookup |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
943 | * finish. |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
944 | */ |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
945 | query_data->callback = NULL; |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
946 | return; |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
947 | } |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
948 | |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
949 | while (query_data->hosts != NULL) |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
950 | { |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
951 | /* Discard the length... */ |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
952 | query_data->hosts = g_slist_remove(query_data->hosts, query_data->hosts->data); |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
953 | /* Free the address... */ |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
954 | g_free(query_data->hosts->data); |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
955 | query_data->hosts = g_slist_remove(query_data->hosts, query_data->hosts->data); |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
956 | } |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
957 | g_free(query_data->error_message); |
|
24321
0d6d024049eb
Improve a few comments
Mark Doliner <markdoliner@pidgin.im>
parents:
24318
diff
changeset
|
958 | #endif /* end _WIN32 */ |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
959 | |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
960 | if (query_data->timeout > 0) |
| 15884 | 961 | purple_timeout_remove(query_data->timeout); |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
962 | |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
963 | g_free(query_data->hostname); |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
964 | g_free(query_data); |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
965 | } |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
966 | |
|
15763
2b529e4350bc
The UI can now use the GaimDnsQueryUiOps to handle DNS resolving itself; if this is done, the platform-specific DNS query code won't be used. This will be used in Adium to use an efficient threads-based DNS resolve (since Adium is already multithreaded, this is significantly cheaper than fork()).
Evan Schoenberg <evands@pidgin.im>
parents:
15707
diff
changeset
|
967 | char * |
| 15884 | 968 | purple_dnsquery_get_host(PurpleDnsQueryData *query_data) |
|
15763
2b529e4350bc
The UI can now use the GaimDnsQueryUiOps to handle DNS resolving itself; if this is done, the platform-specific DNS query code won't be used. This will be used in Adium to use an efficient threads-based DNS resolve (since Adium is already multithreaded, this is significantly cheaper than fork()).
Evan Schoenberg <evands@pidgin.im>
parents:
15707
diff
changeset
|
969 | { |
|
2b529e4350bc
The UI can now use the GaimDnsQueryUiOps to handle DNS resolving itself; if this is done, the platform-specific DNS query code won't be used. This will be used in Adium to use an efficient threads-based DNS resolve (since Adium is already multithreaded, this is significantly cheaper than fork()).
Evan Schoenberg <evands@pidgin.im>
parents:
15707
diff
changeset
|
970 | g_return_val_if_fail(query_data != NULL, NULL); |
|
2b529e4350bc
The UI can now use the GaimDnsQueryUiOps to handle DNS resolving itself; if this is done, the platform-specific DNS query code won't be used. This will be used in Adium to use an efficient threads-based DNS resolve (since Adium is already multithreaded, this is significantly cheaper than fork()).
Evan Schoenberg <evands@pidgin.im>
parents:
15707
diff
changeset
|
971 | |
|
2b529e4350bc
The UI can now use the GaimDnsQueryUiOps to handle DNS resolving itself; if this is done, the platform-specific DNS query code won't be used. This will be used in Adium to use an efficient threads-based DNS resolve (since Adium is already multithreaded, this is significantly cheaper than fork()).
Evan Schoenberg <evands@pidgin.im>
parents:
15707
diff
changeset
|
972 | return query_data->hostname; |
|
2b529e4350bc
The UI can now use the GaimDnsQueryUiOps to handle DNS resolving itself; if this is done, the platform-specific DNS query code won't be used. This will be used in Adium to use an efficient threads-based DNS resolve (since Adium is already multithreaded, this is significantly cheaper than fork()).
Evan Schoenberg <evands@pidgin.im>
parents:
15707
diff
changeset
|
973 | } |
|
2b529e4350bc
The UI can now use the GaimDnsQueryUiOps to handle DNS resolving itself; if this is done, the platform-specific DNS query code won't be used. This will be used in Adium to use an efficient threads-based DNS resolve (since Adium is already multithreaded, this is significantly cheaper than fork()).
Evan Schoenberg <evands@pidgin.im>
parents:
15707
diff
changeset
|
974 | |
|
15771
848371e40d86
Remove a g_return_val_if_fail() which isn't needed, made gaim_dnsquery_get_port() return an unsigned short and return 0 if query_data is NULL, and fixed a whitespace mistake
Evan Schoenberg <evands@pidgin.im>
parents:
15764
diff
changeset
|
975 | unsigned short |
| 15884 | 976 | purple_dnsquery_get_port(PurpleDnsQueryData *query_data) |
|
15763
2b529e4350bc
The UI can now use the GaimDnsQueryUiOps to handle DNS resolving itself; if this is done, the platform-specific DNS query code won't be used. This will be used in Adium to use an efficient threads-based DNS resolve (since Adium is already multithreaded, this is significantly cheaper than fork()).
Evan Schoenberg <evands@pidgin.im>
parents:
15707
diff
changeset
|
977 | { |
|
15771
848371e40d86
Remove a g_return_val_if_fail() which isn't needed, made gaim_dnsquery_get_port() return an unsigned short and return 0 if query_data is NULL, and fixed a whitespace mistake
Evan Schoenberg <evands@pidgin.im>
parents:
15764
diff
changeset
|
978 | g_return_val_if_fail(query_data != NULL, 0); |
|
15763
2b529e4350bc
The UI can now use the GaimDnsQueryUiOps to handle DNS resolving itself; if this is done, the platform-specific DNS query code won't be used. This will be used in Adium to use an efficient threads-based DNS resolve (since Adium is already multithreaded, this is significantly cheaper than fork()).
Evan Schoenberg <evands@pidgin.im>
parents:
15707
diff
changeset
|
979 | |
|
15775
a1aed6d0c33b
No changes; fixed a bunch of whitespace problems made by a text editor with 'use tabs' disabled. Sorry for the noise.
Evan Schoenberg <evands@pidgin.im>
parents:
15771
diff
changeset
|
980 | return query_data->port; |
|
15763
2b529e4350bc
The UI can now use the GaimDnsQueryUiOps to handle DNS resolving itself; if this is done, the platform-specific DNS query code won't be used. This will be used in Adium to use an efficient threads-based DNS resolve (since Adium is already multithreaded, this is significantly cheaper than fork()).
Evan Schoenberg <evands@pidgin.im>
parents:
15707
diff
changeset
|
981 | } |
|
2b529e4350bc
The UI can now use the GaimDnsQueryUiOps to handle DNS resolving itself; if this is done, the platform-specific DNS query code won't be used. This will be used in Adium to use an efficient threads-based DNS resolve (since Adium is already multithreaded, this is significantly cheaper than fork()).
Evan Schoenberg <evands@pidgin.im>
parents:
15707
diff
changeset
|
982 | |
|
2b529e4350bc
The UI can now use the GaimDnsQueryUiOps to handle DNS resolving itself; if this is done, the platform-specific DNS query code won't be used. This will be used in Adium to use an efficient threads-based DNS resolve (since Adium is already multithreaded, this is significantly cheaper than fork()).
Evan Schoenberg <evands@pidgin.im>
parents:
15707
diff
changeset
|
983 | void |
| 15884 | 984 | purple_dnsquery_set_ui_ops(PurpleDnsQueryUiOps *ops) |
|
15763
2b529e4350bc
The UI can now use the GaimDnsQueryUiOps to handle DNS resolving itself; if this is done, the platform-specific DNS query code won't be used. This will be used in Adium to use an efficient threads-based DNS resolve (since Adium is already multithreaded, this is significantly cheaper than fork()).
Evan Schoenberg <evands@pidgin.im>
parents:
15707
diff
changeset
|
985 | { |
|
2b529e4350bc
The UI can now use the GaimDnsQueryUiOps to handle DNS resolving itself; if this is done, the platform-specific DNS query code won't be used. This will be used in Adium to use an efficient threads-based DNS resolve (since Adium is already multithreaded, this is significantly cheaper than fork()).
Evan Schoenberg <evands@pidgin.im>
parents:
15707
diff
changeset
|
986 | dns_query_ui_ops = ops; |
|
2b529e4350bc
The UI can now use the GaimDnsQueryUiOps to handle DNS resolving itself; if this is done, the platform-specific DNS query code won't be used. This will be used in Adium to use an efficient threads-based DNS resolve (since Adium is already multithreaded, this is significantly cheaper than fork()).
Evan Schoenberg <evands@pidgin.im>
parents:
15707
diff
changeset
|
987 | } |
|
2b529e4350bc
The UI can now use the GaimDnsQueryUiOps to handle DNS resolving itself; if this is done, the platform-specific DNS query code won't be used. This will be used in Adium to use an efficient threads-based DNS resolve (since Adium is already multithreaded, this is significantly cheaper than fork()).
Evan Schoenberg <evands@pidgin.im>
parents:
15707
diff
changeset
|
988 | |
| 15884 | 989 | PurpleDnsQueryUiOps * |
| 990 | purple_dnsquery_get_ui_ops(void) | |
|
15763
2b529e4350bc
The UI can now use the GaimDnsQueryUiOps to handle DNS resolving itself; if this is done, the platform-specific DNS query code won't be used. This will be used in Adium to use an efficient threads-based DNS resolve (since Adium is already multithreaded, this is significantly cheaper than fork()).
Evan Schoenberg <evands@pidgin.im>
parents:
15707
diff
changeset
|
991 | { |
|
15771
848371e40d86
Remove a g_return_val_if_fail() which isn't needed, made gaim_dnsquery_get_port() return an unsigned short and return 0 if query_data is NULL, and fixed a whitespace mistake
Evan Schoenberg <evands@pidgin.im>
parents:
15764
diff
changeset
|
992 | /* It is perfectly acceptable for dns_query_ui_ops to be NULL; this just |
|
848371e40d86
Remove a g_return_val_if_fail() which isn't needed, made gaim_dnsquery_get_port() return an unsigned short and return 0 if query_data is NULL, and fixed a whitespace mistake
Evan Schoenberg <evands@pidgin.im>
parents:
15764
diff
changeset
|
993 | * means that the default platform-specific implementation will be used. |
|
848371e40d86
Remove a g_return_val_if_fail() which isn't needed, made gaim_dnsquery_get_port() return an unsigned short and return 0 if query_data is NULL, and fixed a whitespace mistake
Evan Schoenberg <evands@pidgin.im>
parents:
15764
diff
changeset
|
994 | */ |
|
15763
2b529e4350bc
The UI can now use the GaimDnsQueryUiOps to handle DNS resolving itself; if this is done, the platform-specific DNS query code won't be used. This will be used in Adium to use an efficient threads-based DNS resolve (since Adium is already multithreaded, this is significantly cheaper than fork()).
Evan Schoenberg <evands@pidgin.im>
parents:
15707
diff
changeset
|
995 | return dns_query_ui_ops; |
|
2b529e4350bc
The UI can now use the GaimDnsQueryUiOps to handle DNS resolving itself; if this is done, the platform-specific DNS query code won't be used. This will be used in Adium to use an efficient threads-based DNS resolve (since Adium is already multithreaded, this is significantly cheaper than fork()).
Evan Schoenberg <evands@pidgin.im>
parents:
15707
diff
changeset
|
996 | } |
|
2b529e4350bc
The UI can now use the GaimDnsQueryUiOps to handle DNS resolving itself; if this is done, the platform-specific DNS query code won't be used. This will be used in Adium to use an efficient threads-based DNS resolve (since Adium is already multithreaded, this is significantly cheaper than fork()).
Evan Schoenberg <evands@pidgin.im>
parents:
15707
diff
changeset
|
997 | |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
998 | void |
| 15884 | 999 | purple_dnsquery_init(void) |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
1000 | { |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
1001 | } |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
1002 | |
|
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
1003 | void |
| 15884 | 1004 | purple_dnsquery_uninit(void) |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
1005 | { |
|
23352
9714223ca7a7
A patch from Peter O'Gorman at The Written Word, Inc. to fix various
Peter O'Gorman <pogma@thewrittenword.com>
parents:
22622
diff
changeset
|
1006 | #if defined(PURPLE_DNSQUERY_USE_FORK) |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
1007 | while (free_dns_children != NULL) |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
1008 | { |
| 15884 | 1009 | purple_dnsquery_resolver_destroy(free_dns_children->data); |
|
14349
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
1010 | free_dns_children = g_slist_remove(free_dns_children, free_dns_children->data); |
|
6b7a67b5f56d
[gaim-migrate @ 16974]
Mark Doliner <markdoliner@pidgin.im>
parents:
14325
diff
changeset
|
1011 | } |
|
24321
0d6d024049eb
Improve a few comments
Mark Doliner <markdoliner@pidgin.im>
parents:
24318
diff
changeset
|
1012 | #endif /* end PURPLE_DNSQUERY_USE_FORK */ |
|
14300
68a0aa63f3b8
[gaim-migrate @ 16920]
Mark Doliner <markdoliner@pidgin.im>
parents:
14254
diff
changeset
|
1013 | } |