On Thu, Feb 24, 2011 at 05:55:05PM -0500, Gurusamy Sarathy wrote:
> Sorry, didn't mean to suggest it is a missing feature--it does look
> like a bug to me. The documented form doesn't work.
>
> Feb 23 10:32:46 vm-rhes5-01 postfix/smtp[1062]: fatal: valid hostname or
> network address required in server description:
> [IPv6:fe80::250:56ff:fe82:54b5]
Yep, indeed host_port() seems to pass "ipv6:..." to valid_hostaddr(),
which calls valid_ipv6_hostaddr(), which is not prepared for the
"ipv6:" prefix.
Perhaps the call to valid_hostaddr() should be replaced by a call to
valid_mailhost_addr(), and the result used to update the "bare" host
address returned from host_port().
Perhaps the simpler patch below will do the job. Please drop the list
a note when you have checked whether this also solves the problem.
Index: src/util/host_port.c
*** src/util/host_port.c 6 Feb 2010 07:35:12 -0000 1.1.1.1
--- src/util/host_port.c 25 Feb 2011 05:57:57 -0000
***************
*** 136,142 ****
* network addresses instead of requiring proper [ipaddress] forms.
*/
if (*host != def_host && !valid_hostname(*host, DONT_GRIPE)
! && !valid_hostaddr(*host, DONT_GRIPE))
return ("valid hostname or network address required");
if (*port != def_service && ISDIGIT(**port) && !alldig(*port))
return ("garbage after numerical service");
--- 136,142 ----
* network addresses instead of requiring proper [ipaddress] forms.
*/
if (*host != def_host && !valid_hostname(*host, DONT_GRIPE)
! && (*host = valid_mailhost_addr(*host, DONT_GRIPE)) == 0)
return ("valid hostname or network address required");
if (*port != def_service && ISDIGIT(**port) && !alldig(*port))
return ("garbage after numerical service");
--
Viktor.
