From owner-freebsd-apache@FreeBSD.ORG Sun Aug 12 23:53:48 2012 Return-Path: Delivered-To: apache@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id 8D94D1065672 for ; Sun, 12 Aug 2012 23:53:48 +0000 (UTC) (envelope-from curtis@occnc.com) Received: from gateway.ipv6.occnc.com (gateway.ipv6.occnc.com [IPv6:2001:470:1f07:1545::1:132]) by mx1.freebsd.org (Postfix) with ESMTP id 2C9198FC0A for ; Sun, 12 Aug 2012 23:53:48 +0000 (UTC) Received: from newharbor.ipv6.occnc.com (newharbor.ipv6.occnc.com [IPv6:2001:470:1f07:1545::1:320]) (authenticated bits=0) by gateway.ipv6.occnc.com (8.14.5/8.14.5) with ESMTP id q7CNrOji075223; Sun, 12 Aug 2012 19:53:25 -0400 (EDT) (envelope-from curtis@occnc.com) Message-Id: <201208122353.q7CNrOji075223@gateway.ipv6.occnc.com> To: "apache@FreeBSD.org" From: Curtis Villamizar In-reply-to: Your message of "Mon, 13 Aug 2012 00:25:28 +0200." <50282D58.5070605@FreeBSD.org> Date: Sun, 12 Aug 2012 19:53:24 -0400 Cc: curtis@occnc.com Subject: Re: patch to apache for IPv6 only httpd X-BeenThere: freebsd-apache@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: curtis@occnc.com List-Id: Support of apache-related ports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 12 Aug 2012 23:53:48 -0000 In message <50282D58.5070605@FreeBSD.org> Olli Hauer writes: > On 2012-08-11 23:22, Curtis Villamizar wrote: > > FYI- > > > > This very simple patch prevents an assert when httpd is run on a > > server that has no IPv4 address. An example is a BSD jail running > > with only an IPv6 address on the interface (no loopback so no > > 127.0.0.1 address to bind to). > > > > Copy the patch below into the files directory as > > /usr/ports/www/apache22/files/patch-server__config.c > > > > The port maintainer is on the Cc. No bug has been reported. > > > > BTW- I noticed this a long time ago. I don't remember if I reported > > it or forgot to. Appologies if this is a duplicate. It is not in the > > httpd-2.2.22.tar.bz2 tarball, in the distinfo dated 23 July 2012. > > > > Curtis > > > > > > --- ./server/config.c.orig 2010-10-07 09:56:54.000000000 -0700 > > +++ ./server/config.c 2011-09-26 21:20:35.000000000 -0700 > > @@ -1979,6 +1979,9 @@ > > /* NOT virtual host; don't match any real network interface */ > > rv = apr_sockaddr_info_get(&s->addrs->host_addr, > > NULL, APR_INET, 0, 0, p); > > + if (rv != APR_SUCCESS) > > + rv = apr_sockaddr_info_get(&s->addrs->host_addr, > > + NULL, APR_UNSPEC, 0, 0, p); > > ap_assert(rv == APR_SUCCESS); /* otherwise: bug or no storage */ > > > > s->addrs->host_port = 0; /* matches any port */ > > Hi Curtis, > > Is this patch also relevant if apache22 was build with > -DWITH_IPV6_V6ONLY ? > > -- > olli No. Apache was not built with -DWITH_IPV6_V6ONLY. It is apache built with dual stack support though I don't think the code is any different if built with WITH_IPV6_V6ONLY. Nothing here is conditionally compiled. I also checked the .h where APR_INET is defined (in srclib/apr/include/apr_network_io.h). It is unconditionally defined to AF_INET. The code compiles but the server fails to run, giving an assert when starting up. Its a simple patch. APR_INET6 could be used instead of APR_UNSPEC in the patch. Curtis