# [BUG] deamon.c does not compile when NO_IPV6 is used

4 messages from 2015-05-05 to 2015-05-18. Participants: Eduardo Espejo, Junio C Hamano, René Scharfe.
Thread: https://gitlist.dev/t/39254

## Eduardo Espejo, 2015-05-05 14:20

Subject: [BUG] deamon.c does not compile when NO_IPV6 is used
Message-ID: <CAK4aQQJ9yT8+Q9jPQ78cauhyPj3j5WmpgrQJ5=cA-ECjaWn8BQ@mail.gmail.com>
URL: https://gitlist.dev/e/CAK4aQQJ9yT8%2BQ9jPQ78cauhyPj3j5WmpgrQJ5%3DcA-ECjaWn8BQ%40mail.gmail.com

```
Tested in 2.4 commit 3d4a3ffe64:

daemon.c:636:24: error: use of undeclared identifier 'hostname'
                hent = gethostbyname(hostname.buf);

--
Eduardo Espejo

```

## Junio C Hamano, 2015-05-05 17:55

Subject: Re: [BUG] deamon.c does not compile when NO_IPV6 is used
Message-ID: <xmqqioc6ewnw.fsf@gitster.dls.corp.google.com>
URL: https://gitlist.dev/e/xmqqioc6ewnw.fsf%40gitster.dls.corp.google.com
In-Reply-To: <CAK4aQQJ9yT8+Q9jPQ78cauhyPj3j5WmpgrQJ5=cA-ECjaWn8BQ@mail.gmail.com>

```
Eduardo Espejo <eduardo.espejo@gmail.com> writes:

> Tested in 2.4 commit 3d4a3ffe64:
>
> daemon.c:636:24: error: use of undeclared identifier 'hostname'
>                 hent = gethostbyname(hostname.buf);

Thanks.  It seems nobody actively interested in developing Git
builds with NO_IPV6 these days?

Does this help?

 daemon.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/daemon.c b/daemon.c
index ac2bc85..d3d3e43 100644
--- a/daemon.c
+++ b/daemon.c
@@ -633,7 +633,7 @@ static void lookup_hostname(struct hostinfo *hi)
 		char **ap;
 		static char addrbuf[HOST_NAME_MAX + 1];
 
-		hent = gethostbyname(hostname.buf);
+		hent = gethostbyname(hi->hostname.buf);
 		if (hent) {
 			ap = hent->h_addr_list;
 			memset(&sa, 0, sizeof sa);

```

## Junio C Hamano, 2015-05-05 18:07

Subject: [PATCH] daemon: unbreak NO_IPV6 build regression
Message-ID: <xmqqd22eew4v.fsf_-_@gitster.dls.corp.google.com>
URL: https://gitlist.dev/e/xmqqd22eew4v.fsf_-_%40gitster.dls.corp.google.com
In-Reply-To: <xmqqioc6ewnw.fsf@gitster.dls.corp.google.com>

```
When 01cec54e (daemon: deglobalize hostname information, 2015-03-07)
wrapped the global variables such as hostname inside a struct, it
forgot to convert one location that spelled "hostname" that needs to
be updated to "hi->hostname".

This was inside NO_IPV6 block, and was not caught by anybody.

Reported by: Eduardo Espejo
Signed-off-by: Junio C Hamano <gitster@pobox.com>
---
 daemon.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/daemon.c b/daemon.c
index 9ee2187..4be1091 100644
--- a/daemon.c
+++ b/daemon.c
@@ -633,7 +633,7 @@ static void lookup_hostname(struct hostinfo *hi)
 		char **ap;
 		static char addrbuf[HOST_NAME_MAX + 1];
 
-		hent = gethostbyname(hostname.buf);
+		hent = gethostbyname(hi->hostname.buf);
 		if (hent) {
 			ap = hent->h_addr_list;
 			memset(&sa, 0, sizeof sa);
-- 
2.4.0-311-gf1d9b8d

```

## René Scharfe, 2015-05-18 22:29

Subject: Re: [PATCH] daemon: unbreak NO_IPV6 build regression
Message-ID: <555A67D4.9040204@web.de>
URL: https://gitlist.dev/e/555A67D4.9040204%40web.de
In-Reply-To: <xmqqd22eew4v.fsf_-_@gitster.dls.corp.google.com>

```
Am 05.05.2015 um 20:07 schrieb Junio C Hamano:
> When 01cec54e (daemon: deglobalize hostname information, 2015-03-07)
> wrapped the global variables such as hostname inside a struct, it
> forgot to convert one location that spelled "hostname" that needs to
> be updated to "hi->hostname".
>
> This was inside NO_IPV6 block, and was not caught by anybody.

Sorry for that, and thank you for cleaning up after me. :-/

@Eduardo: Out of interest, on which platform (CPU, operating system, 
compiler) did you notice the bug?

Thanks,
René

```