From 93c17d21b2979f9996e7c10af9a939b87539bfc7 Mon Sep 17 00:00:00 2001 From: Jonathan Nieder Date: Mon, 6 Jun 2011 04:39:29 -0500 Subject: daemon: move locate_host() to tcp.c Keep the different name resolution functions close together so they can learn from each other and perhaps share code in the future. Signed-off-by: Jonathan Nieder --- daemon.c | 68 ++-------------------------------------------------------------- tcp.c | 59 +++++++++++++++++++++++++++++++++++++++++++++++++++++++ tcp.h | 3 +++ 3 files changed, 64 insertions(+), 66 deletions(-) diff --git a/daemon.c b/daemon.c index 66c2545..88c8d2f 100644 --- a/daemon.c +++ b/daemon.c @@ -1,14 +1,11 @@ #include "cache.h" #include "pkt-line.h" +#include "tcp.h" #include "exec_cmd.h" #include "run-command.h" #include "strbuf.h" #include "string-list.h" -#ifndef HOST_NAME_MAX -#define HOST_NAME_MAX 256 -#endif - #ifdef NO_INITGROUPS #define initgroups(x, y) (0) /* nothing */ #endif @@ -504,67 +501,6 @@ static void parse_host_and_port(char *hostport, char **host, } } -#ifndef NO_IPV6 - -static void locate_host(const char *hostname, char **ip_address, - char **canon_hostname) -{ - struct addrinfo hints; - struct addrinfo *ai; - int gai; - static char addrbuf[HOST_NAME_MAX + 1]; - struct sockaddr_in *sin_addr; - - memset(&hints, 0, sizeof(hints)); - hints.ai_flags = AI_CANONNAME; - - gai = getaddrinfo(hostname, NULL, &hints, &ai); - if (gai) - return; - - sin_addr = (void *)ai->ai_addr; - inet_ntop(AF_INET, &sin_addr->sin_addr, addrbuf, sizeof(addrbuf)); - free(*ip_address); - *ip_address = xstrdup(addrbuf); - - free(*canon_hostname); - *canon_hostname = xstrdup(ai->ai_canonname ? - ai->ai_canonname : *ip_address); - - freeaddrinfo(ai); -} - -#else - -static void locate_host(const char *hostname, char **ip_address, - char **canon_hostname) -{ - struct hostent *hent; - struct sockaddr_in sa; - char **ap; - static char addrbuf[HOST_NAME_MAX + 1]; - - hent = gethostbyname(hostname); - if (!hent) - return; - - ap = hent->h_addr_list; - memset(&sa, 0, sizeof sa); - sa.sin_family = hent->h_addrtype; - sa.sin_port = htons(0); - memcpy(&sa.sin_addr, *ap, hent->h_length); - - inet_ntop(hent->h_addrtype, &sa.sin_addr, - addrbuf, sizeof(addrbuf)); - - free(*canon_hostname); - *canon_hostname = xstrdup(hent->h_name); - free(*ip_address); - *ip_address = xstrdup(addrbuf); -} - -#endif - /* * Read the host as supplied by the client connection. */ @@ -604,7 +540,7 @@ static void parse_host_arg(char *extra_args, int buflen) * if possible. */ if (hostname) - locate_host(hostname, &ip_address, &canon_hostname); + git_locate_host(hostname, &ip_address, &canon_hostname); } diff --git a/tcp.c b/tcp.c index dd69502..7e2c3da 100644 --- a/tcp.c +++ b/tcp.c @@ -3,6 +3,10 @@ #include "run-command.h" #include "connect.h" +#ifndef HOST_NAME_MAX +#define HOST_NAME_MAX 256 +#endif + #define STR_(s) # s #define STR(s) STR_(s) @@ -49,6 +53,34 @@ static const char *ai_name(const struct addrinfo *ai) return addr; } +void git_locate_host(const char *hostname, char **ip_address, + char **canon_hostname) +{ + struct addrinfo hints; + struct addrinfo *ai; + int gai; + static char addrbuf[HOST_NAME_MAX + 1]; + struct sockaddr_in *sin_addr; + + memset(&hints, 0, sizeof(hints)); + hints.ai_flags = AI_CANONNAME; + + gai = getaddrinfo(hostname, NULL, &hints, &ai); + if (gai) + return; + + sin_addr = (void *)ai->ai_addr; + inet_ntop(AF_INET, &sin_addr->sin_addr, addrbuf, sizeof(addrbuf)); + free(*ip_address); + *ip_address = xstrdup(addrbuf); + + free(*canon_hostname); + *canon_hostname = xstrdup(ai->ai_canonname ? + ai->ai_canonname : *ip_address); + + freeaddrinfo(ai); +} + /* * Returns a connected socket() fd, or else die()s. */ @@ -113,6 +145,33 @@ static int git_tcp_connect_sock(char *host, int flags) #else /* NO_IPV6 */ +void git_locate_host(const char *hostname, char **ip_address, + char **canon_hostname) +{ + struct hostent *hent; + struct sockaddr_in sa; + char **ap; + static char addrbuf[HOST_NAME_MAX + 1]; + + hent = gethostbyname(hostname); + if (!hent) + return; + + ap = hent->h_addr_list; + memset(&sa, 0, sizeof sa); + sa.sin_family = hent->h_addrtype; + sa.sin_port = htons(0); + memcpy(&sa.sin_addr, *ap, hent->h_length); + + inet_ntop(hent->h_addrtype, &sa.sin_addr, + addrbuf, sizeof(addrbuf)); + + free(*canon_hostname); + *canon_hostname = xstrdup(hent->h_name); + free(*ip_address); + *ip_address = xstrdup(addrbuf); +} + /* * Returns a connected socket() fd, or else die()s. */ diff --git a/tcp.h b/tcp.h index 566afd4..7c5b048 100644 --- a/tcp.h +++ b/tcp.h @@ -1,6 +1,9 @@ #ifndef TCP_H #define TCP_H +extern void git_locate_host(const char *hostname, + char **ip_address, char **canon_hostname); + extern void get_host_and_port(char **host, const char **port); extern int git_use_proxy(const char *host); extern void git_tcp_connect(int fd[2], char *host, int flags); -- 2.1.0.rc2.206.gedb03e5