autofs-5.1.3 - add missing MODPREFIX to logging in amd parser From: Ian Kent Some of the logging in the amd parser is missing the log identifier MODPREFIX so add it. Signed-off-by: Ian Kent --- CHANGELOG | 1 + modules/parse_amd.c | 25 ++++++++++++++----------- 2 files changed, 15 insertions(+), 11 deletions(-) --- autofs-5.0.7.orig/CHANGELOG +++ autofs-5.0.7/CHANGELOG @@ -269,6 +269,7 @@ - improve debug logging of lookup key. - fix cachefs parse message not being logged. - fix typo in amd_parse.c. +- add missing MODPREFIX to logging in amd parser. 25/07/2012 autofs-5.0.7 ======================= --- autofs-5.0.7.orig/modules/parse_amd.c +++ autofs-5.0.7/modules/parse_amd.c @@ -245,7 +245,7 @@ static int match_my_name(unsigned int lo } if (!v || !v->val) { - error(logopt, "error: ${host} not set"); + error(logopt, MODPREFIX "error: ${host} not set"); goto out; } @@ -259,7 +259,7 @@ static int match_my_name(unsigned int lo /* Get host canonical name */ ret = getaddrinfo(v->val, NULL, &hints, &cni); if (ret) { - error(logopt, + error(logopt, MODPREFIX "hostname lookup failed: %s\n", gai_strerror(ret)); goto out; } @@ -269,7 +269,7 @@ static int match_my_name(unsigned int lo /* Resolve comparison name to its names and compare */ ret = getaddrinfo(name, NULL, &hints, &ni); if (ret) { - error(logopt, + error(logopt, MODPREFIX "hostname lookup failed: %s\n", gai_strerror(ret)); freeaddrinfo(cni); goto out; @@ -282,7 +282,7 @@ static int match_my_name(unsigned int lo numeric, sizeof(numeric), NULL, 0, NI_NUMERICHOST); if (ret) { - error(logopt, + error(logopt, MODPREFIX "host address info lookup failed: %s\n", gai_strerror(ret)); freeaddrinfo(cni); @@ -293,7 +293,7 @@ static int match_my_name(unsigned int lo ret = getnameinfo(haddr->ai_addr, haddr->ai_addrlen, host, NI_MAXHOST, NULL, 0, 0); if (ret) { - error(logopt, + error(logopt, MODPREFIX "host address info lookup failed: %s\n", gai_strerror(ret)); freeaddrinfo(cni); @@ -330,7 +330,8 @@ static int eval_selector(unsigned int lo case SEL_FLAG_MACRO: v = macro_findvar(sv, s->sel->name, strlen(s->sel->name)); if (!v) { - error(logopt, "failed to get selector %s", s->sel->name); + error(logopt, MODPREFIX + "failed to get selector %s", s->sel->name); return 0; } @@ -499,7 +500,7 @@ static int eval_selector(unsigned int lo else v = macro_findvar(sv, "hostd", 5); if (!v || !*v->val) { - error(logopt, + error(logopt, MODPREFIX "failed to get value of ${host}"); break; } @@ -666,7 +667,8 @@ static char *normalize_hostname(unsigned ret = getaddrinfo(host, NULL, &hints, &ni); if (ret) { - error(logopt, "hostname lookup failed: %s", gai_strerror(ret)); + error(logopt, MODPREFIX + "hostname lookup failed: %s", gai_strerror(ret)); return NULL; } name = strdup(ni->ai_canonname); @@ -1302,7 +1304,7 @@ static unsigned int validate_generic_opt * Can't use entry->type as the mount type to reprot * the error since entry->type == "bind" not "lofs". */ - error(logopt, "lofs: mount device not given"); + error(logopt, MODPREFIX "lofs: mount device not given"); return 0; } else if (!*entry->rfs) return 0; @@ -1575,7 +1577,7 @@ static int match_selectors(unsigned int /* No selectors, always match */ if (!s) { - debug(logopt, "no selectors found in location"); + debug(logopt, MODPREFIX "no selectors found in location"); return 1; } @@ -2000,7 +2002,8 @@ int parse_mount(struct autofs_point *ap, free_amd_entry(cur_defaults); if (rv) - debug(ap->logopt, "no more locations to try, returning fail"); + debug(ap->logopt, MODPREFIX + "no more locations to try, returning fail"); done: free_amd_entry_list(&entries); free_amd_entry(defaults_entry);