Re: [meta-networking][PATCH] postfix.inc: fix start postfix failed while hostname is numeric

[Joe MacDonald <Joe_MacDonald@mentor.com>]

[-- Attachment #1: Type: text/plain, Size: 2257 bytes --]

[[oe] [meta-networking][PATCH] postfix.inc: fix start postfix failed while hostname is numeric] On 15.11.26 (Thu 18:36) Hongxu Jia wrote:

> While hostname is numeric, start postfix failed
> ...
> root@localhost:~# hostname
> 128.224.163.251
> root@localhost:~# postfix start
> postfix: warning: valid_hostname: numeric hostname: 128.224.163.251
> postfix: fatal: unable to use my own hostname
> ...
> 
> The postfix define a macro SLOPPY_VALID_HOSTNAME to allow the
> numeric hostname.

This seems like it's asking for a lot of trouble, if you either assign
your hostname to '128.224.163.251' (as it appears to be in your commit
log, but I don't know off-hand how you'd do that without writing a
program specifically for that purpose, and even then ...) or if you
assign your hostname to be '128' and the FQDN to be '128.224.163.251'.
Certainly the postfix folks think this is a sufficiently unusual and
presumably hazzard-prone to make it not even a runtime option but a
compile-time one.

Are you sure the issue you're trying to solve here won't be resolved by
applying [] throughout your .cf files?  I'm reluctant to take this patch
since it turns on a surprising feature that, my sense is, is not
desirable in common setups.

-J.

> 
> Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com>
> ---
>  meta-networking/recipes-daemons/postfix/postfix.inc | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/meta-networking/recipes-daemons/postfix/postfix.inc b/meta-networking/recipes-daemons/postfix/postfix.inc
> index 6d39570..f8b8e43 100644
> --- a/meta-networking/recipes-daemons/postfix/postfix.inc
> +++ b/meta-networking/recipes-daemons/postfix/postfix.inc
> @@ -67,7 +67,7 @@ export CCARGS-sasl_class-native = ""
>  export AUXLIBS-sasl_class-native = ""
>  
>  # PCRE, TLS support default
> -export CCARGS  = "${CFLAGS} -DHAS_PCRE -DUSE_TLS ${CCARGS-ldap} ${CCARGS-sasl}"
> +export CCARGS  = "${CFLAGS} -DHAS_PCRE -DUSE_TLS -DSLOPPY_VALID_HOSTNAME ${CCARGS-ldap} ${CCARGS-sasl}"
>  export AUXLIBS = "-lpcre -lssl -lcrypto ${AUXLIBS-sasl} ${AUXLIBS-ldap}"
>  export POSTCONF = "${STAGING_DIR_NATIVE}${sbindir_native}/postconf"
>  
> -- 
> 1.9.1
> 
-- 
-Joe MacDonald.
:wq

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 484 bytes --]