|
 |
306fa1 |
autofs-5.1.4 - improve hostname lookup error logging
|
|
|
306fa1 |
|
|
|
306fa1 |
From: Ian Kent <raven@themaw.net>
|
|
|
306fa1 |
|
|
|
306fa1 |
There's not enough information in name lookup log messages
|
|
|
306fa1 |
to determine if the failed lookup is in fact a problem or
|
|
|
306fa1 |
is expected.
|
|
|
306fa1 |
|
|
|
306fa1 |
Signed-off-by: Ian Kent <raven@themaw.net>
|
|
|
306fa1 |
---
|
|
|
306fa1 |
CHANGELOG | 1 +
|
|
|
306fa1 |
lib/parse_subs.c | 3 ++-
|
|
|
306fa1 |
lib/rpc_subs.c | 3 ++-
|
|
|
306fa1 |
modules/dclist.c | 4 +++-
|
|
|
306fa1 |
modules/parse_amd.c | 9 ++++++---
|
|
|
306fa1 |
modules/replicated.c | 5 +++--
|
|
|
306fa1 |
6 files changed, 17 insertions(+), 8 deletions(-)
|
|
|
306fa1 |
|
|
|
306fa1 |
--- autofs-5.0.7.orig/CHANGELOG
|
|
|
306fa1 |
+++ autofs-5.0.7/CHANGELOG
|
|
|
306fa1 |
@@ -325,6 +325,7 @@
|
|
|
306fa1 |
- document ghost option in auto.master man page.
|
|
|
306fa1 |
- make expire remaining log level debug.
|
|
|
306fa1 |
- workaround getaddrinfo(3) ai_canonname bug
|
|
|
306fa1 |
+- improve hostname lookup error logging.
|
|
|
306fa1 |
|
|
|
306fa1 |
25/07/2012 autofs-5.0.7
|
|
|
306fa1 |
=======================
|
|
|
306fa1 |
--- autofs-5.0.7.orig/lib/parse_subs.c
|
|
|
306fa1 |
+++ autofs-5.0.7/lib/parse_subs.c
|
|
|
306fa1 |
@@ -476,7 +476,8 @@ unsigned int get_network_proximity(const
|
|
|
306fa1 |
|
|
|
306fa1 |
ret = getaddrinfo(name_or_num, NULL, &hints, &ni);
|
|
|
306fa1 |
if (ret) {
|
|
|
306fa1 |
- logerr("getaddrinfo: %s", gai_strerror(ret));
|
|
|
306fa1 |
+ logerr("hostname lookup for %s failed: %s",
|
|
|
306fa1 |
+ name_or_num, gai_strerror(ret));
|
|
|
306fa1 |
return PROXIMITY_ERROR;
|
|
|
306fa1 |
}
|
|
|
306fa1 |
|
|
|
306fa1 |
--- autofs-5.0.7.orig/lib/rpc_subs.c
|
|
|
306fa1 |
+++ autofs-5.0.7/lib/rpc_subs.c
|
|
|
306fa1 |
@@ -702,7 +702,8 @@ static int create_client(struct conn_inf
|
|
|
306fa1 |
ret = getaddrinfo(info->host, NULL, &hints, &ai;;
|
|
|
306fa1 |
if (ret) {
|
|
|
306fa1 |
error(LOGOPT_ANY,
|
|
|
306fa1 |
- "hostname lookup failed: %s", gai_strerror(ret));
|
|
|
306fa1 |
+ "hostname lookup for %s failed: %s",
|
|
|
306fa1 |
+ info->host, gai_strerror(ret));
|
|
|
306fa1 |
goto out_close;
|
|
|
306fa1 |
}
|
|
|
306fa1 |
|
|
|
306fa1 |
--- autofs-5.0.7.orig/modules/dclist.c
|
|
|
306fa1 |
+++ autofs-5.0.7/modules/dclist.c
|
|
|
306fa1 |
@@ -357,7 +357,9 @@ static char *getdnsdomainname(unsigned i
|
|
|
306fa1 |
|
|
|
306fa1 |
ret = getaddrinfo(name, NULL, &hints, &ni);
|
|
|
306fa1 |
if (ret) {
|
|
|
306fa1 |
- error(logopt, "hostname lookup failed: %s", gai_strerror(ret));
|
|
|
306fa1 |
+ error(logopt,
|
|
|
306fa1 |
+ "hostname lookup for %s failed: %s",
|
|
|
306fa1 |
+ name, gai_strerror(ret));
|
|
|
306fa1 |
return NULL;
|
|
|
306fa1 |
}
|
|
|
306fa1 |
|
|
|
306fa1 |
--- autofs-5.0.7.orig/modules/parse_amd.c
|
|
|
306fa1 |
+++ autofs-5.0.7/modules/parse_amd.c
|
|
|
306fa1 |
@@ -260,7 +260,8 @@ static int match_my_name(unsigned int lo
|
|
|
306fa1 |
ret = getaddrinfo(v->val, NULL, &hints, &cni);
|
|
|
306fa1 |
if (ret) {
|
|
|
306fa1 |
error(logopt, MODPREFIX
|
|
|
306fa1 |
- "hostname lookup failed: %s\n", gai_strerror(ret));
|
|
|
306fa1 |
+ "hostname lookup for %s failed: %s\n",
|
|
|
306fa1 |
+ v->val, gai_strerror(ret));
|
|
|
306fa1 |
goto out;
|
|
|
306fa1 |
}
|
|
|
306fa1 |
|
|
|
306fa1 |
@@ -270,7 +271,8 @@ static int match_my_name(unsigned int lo
|
|
|
306fa1 |
ret = getaddrinfo(name, NULL, &hints, &ni);
|
|
|
306fa1 |
if (ret) {
|
|
|
306fa1 |
error(logopt, MODPREFIX
|
|
|
306fa1 |
- "hostname lookup failed: %s\n", gai_strerror(ret));
|
|
|
306fa1 |
+ "hostname lookup for %s failed: %s\n",
|
|
|
306fa1 |
+ name, gai_strerror(ret));
|
|
|
306fa1 |
freeaddrinfo(cni);
|
|
|
306fa1 |
goto out;
|
|
|
306fa1 |
}
|
|
|
306fa1 |
@@ -680,7 +682,8 @@ static char *normalize_hostname(unsigned
|
|
|
306fa1 |
ret = getaddrinfo(host, NULL, &hints, &ni);
|
|
|
306fa1 |
if (ret) {
|
|
|
306fa1 |
error(logopt, MODPREFIX
|
|
|
306fa1 |
- "hostname lookup failed: %s", gai_strerror(ret));
|
|
|
306fa1 |
+ "hostname lookup for %s failed: %s",
|
|
|
306fa1 |
+ host, gai_strerror(ret));
|
|
|
306fa1 |
return NULL;
|
|
|
306fa1 |
}
|
|
|
306fa1 |
name = strdup(ni->ai_canonname);
|
|
|
306fa1 |
--- autofs-5.0.7.orig/modules/replicated.c
|
|
|
306fa1 |
+++ autofs-5.0.7/modules/replicated.c
|
|
|
306fa1 |
@@ -945,8 +945,9 @@ try_name:
|
|
|
306fa1 |
|
|
|
306fa1 |
ret = getaddrinfo(name, NULL, &hints, &ni);
|
|
|
306fa1 |
if (ret) {
|
|
|
306fa1 |
- error(LOGOPT_ANY, "hostname lookup failed: %s",
|
|
|
306fa1 |
- gai_strerror(ret));
|
|
|
306fa1 |
+ error(LOGOPT_ANY,
|
|
|
306fa1 |
+ "hostname lookup for %s failed: %s",
|
|
|
306fa1 |
+ name, gai_strerror(ret));
|
|
|
306fa1 |
free(name);
|
|
|
306fa1 |
return 0;
|
|
|
306fa1 |
}
|