# HG changeset patch # User Luke Schierer # Date 1079110762 0 # Node ID 4bb3d8dc717e626ad0bc0f5cf5c6aa4335bfb5cf # Parent 6bd8bc8f07bb99898c9fc0983d7982d66bc406cb [gaim-migrate @ 9166] " If getaddrinfo() is used, the addrlen and addr returned through that function are written through the pipe to the child Gaim processes. getaddrinfo() sets the addrlen and addr fields through the following structure, defined in : struct addrinfo { int ai_flags; int ai_family; int ai_socktype; int ai_protocol; size_t ai_addrlen; char *ai_canonname; struct sockaddr *ai_addr; struct addrinfo *ai_next; }; This is from FreeBSD/amd64 5.2.1-RELEASE. This structure is defined differently on different systems. Take, for example, this OpenBSD/i386 3.5-beta system: struct addrinfo { int ai_flags; int ai_family; int ai_socktype; int ai_protocol; socklen_t ai_addrlen; struct sockaddr *ai_addr; char *ai_canonname; struct addrinfo *ai_next; }; After being read, the addrlen and addr of each host is written through the descriptor: src/proxy.c: 466 rc = getaddrinfo(dns_params.hostname, servname, &hints, &res); ... 478 while(res) { 479 write(child_out[1], &(res->ai_addrlen), sizeof(res->ai_addrlen)); 480 write(child_out[1], res->ai_addr, res->ai_addrlen); 481 res = res->ai_next; 482 } And later subsequently read: 286 rc=read(req->fd_out, &addrlen, sizeof(addrlen)); 287 if(rc>0 && addrlen > 0) { 288 addr=g_malloc(addrlen); 289 rc=read(req->fd_out, addr, addrlen); So hence, the type of addrlen that is used in host_resolved() must match that of the addrlen used in the addrinfo structure, or they must at least be guarenteed to be the same size." --jarady committer: Tailor Script diff -r 6bd8bc8f07bb -r 4bb3d8dc717e COPYRIGHT --- a/COPYRIGHT Fri Mar 12 16:22:30 2004 +0000 +++ b/COPYRIGHT Fri Mar 12 16:59:22 2004 +0000 @@ -129,5 +129,6 @@ Jason Willis Matt Wilson Ximian +Jared Yanovich Marco Ziech Jaroen Zwartepoorte diff -r 6bd8bc8f07bb -r 4bb3d8dc717e src/proxy.c --- a/src/proxy.c Fri Mar 12 16:22:30 2004 +0000 +++ b/src/proxy.c Fri Mar 12 16:59:22 2004 +0000 @@ -315,7 +315,7 @@ int rc, err; GSList *hosts = NULL; struct sockaddr *addr = NULL; - socklen_t addrlen; + size_t addrlen; gaim_debug(GAIM_DEBUG_INFO, "dns", "Host '%s' resolved\n", req->host); gaim_input_remove(req->inpa);