From f1409fa7c61e880c7b88665c3109985f7bd63b7f Mon Sep 17 00:00:00 2001 From: Jiri Denemark Date: Thu, 13 Oct 2011 16:19:37 +0200 Subject: [PATCH] util: Make getaddrinfo failure nonfatal in virGetHostname Setting a hostname that cannot be resolved is not the best configuration but since virGetHostname only calls getaddrinfo to get host's canonical name and we do not fail if the returned canonical name is NULL or "localhost", there is no reason why we should fail if getaddrinfo itself fails. --- src/util/util.c | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/src/util/util.c b/src/util/util.c index 1ff287d008..fd4d7faed9 100644 --- a/src/util/util.c +++ b/src/util/util.c @@ -1858,10 +1858,10 @@ char *virIndexToDiskName(int idx, const char *prefix) * try to resolve this to a fully-qualified name. Therefore we pass it * to getaddrinfo(). There are two possible responses: * a) getaddrinfo() resolves to a FQDN - return the FQDN - * b) getaddrinfo() resolves to localhost - in this case, the data we got - * from gethostname() is actually more useful than what we got from - * getaddrinfo(). Return the value from gethostname() and hope for - * the best. + * b) getaddrinfo() files or resolves to localhost - in this case, the + * data we got from gethostname() is actually more useful than what + * we got from getaddrinfo(). Return the value from gethostname() + * and hope for the best. */ char *virGetHostname(virConnectPtr conn ATTRIBUTE_UNUSED) { @@ -1897,10 +1897,10 @@ char *virGetHostname(virConnectPtr conn ATTRIBUTE_UNUSED) hints.ai_family = AF_UNSPEC; r = getaddrinfo(hostname, NULL, &hints, &info); if (r != 0) { - virUtilError(VIR_ERR_INTERNAL_ERROR, - _("getaddrinfo failed for '%s': %s"), - hostname, gai_strerror(r)); - return NULL; + VIR_WARN("getaddrinfo failed for '%s': %s", + hostname, gai_strerror(r)); + result = strdup(hostname); + goto check_and_return; } /* Tell static analyzers about getaddrinfo semantics. */ -- GitLab