Skip to content

Commit

Permalink
Merge branch 'ep/ident-with-getaddrinfo' into maint
Browse files Browse the repository at this point in the history
A fix-up for recent topic.

* ep/ident-with-getaddrinfo:
  ident: fix undefined variable when NO_IPV6 is set
  ident.c: add support for IPv6
  • Loading branch information
Junio C Hamano committed Dec 15, 2015
2 parents f97f2e5 + 58d29ec commit de301c5
Showing 1 changed file with 27 additions and 4 deletions.
31 changes: 27 additions & 4 deletions ident.c
Original file line number Diff line number Diff line change
Expand Up @@ -70,10 +70,35 @@ static int add_mailname_host(struct strbuf *buf)
return 0;
}

static int canonical_name(const char *host, struct strbuf *out)
{
int status = -1;

#ifndef NO_IPV6
struct addrinfo hints, *ai;
memset (&hints, '\0', sizeof (hints));
hints.ai_flags = AI_CANONNAME;
if (!getaddrinfo(host, NULL, &hints, &ai)) {
if (ai && strchr(ai->ai_canonname, '.')) {
strbuf_addstr(out, ai->ai_canonname);
status = 0;
}
freeaddrinfo(ai);
}
#else
struct hostent *he = gethostbyname(host);
if (he && strchr(he->h_name, '.')) {
strbuf_addstr(out, he->h_name);
status = 0;
}
#endif /* NO_IPV6 */

return status;
}

static void add_domainname(struct strbuf *out)
{
char buf[1024];
struct hostent *he;

if (gethostname(buf, sizeof(buf))) {
warning("cannot get host name: %s", strerror(errno));
Expand All @@ -82,9 +107,7 @@ static void add_domainname(struct strbuf *out)
}
if (strchr(buf, '.'))
strbuf_addstr(out, buf);
else if ((he = gethostbyname(buf)) && strchr(he->h_name, '.'))
strbuf_addstr(out, he->h_name);
else
else if (canonical_name(buf, out) < 0)
strbuf_addf(out, "%s.(none)", buf);
}

Expand Down

0 comments on commit de301c5

Please sign in to comment.