* [nfs-utils PATCH] exportfs: Restore the EAI_NONAME check in host_pton()
@ 2015-10-28 17:19 Scott Mayhew
2015-11-04 21:51 ` Steve Dickson
0 siblings, 1 reply; 2+ messages in thread
From: Scott Mayhew @ 2015-10-28 17:19 UTC (permalink / raw)
To: steved; +Cc: linux-nfs
Commit d89e3fc7 removed the EAI_NONAME check altogether instead of just
moving the NULL check. This causes exportfs -u to incorrectly exit
with 1 whenever there's more than one MCL_FQDN export in the exportlist.
Signed-off-by: Scott Mayhew <smayhew@redhat.com>
---
support/export/hostname.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/support/export/hostname.c b/support/export/hostname.c
index 169baa5..7a44d42 100644
--- a/support/export/hostname.c
+++ b/support/export/hostname.c
@@ -134,6 +134,8 @@ host_pton(const char *paddr)
break;
}
return ai;
+ case EAI_NONAME:
+ break;
case EAI_SYSTEM:
xlog(D_GENERAL, "%s: failed to convert %s: (%d) %m",
__func__, paddr, errno);
--
2.4.3
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [nfs-utils PATCH] exportfs: Restore the EAI_NONAME check in host_pton()
2015-10-28 17:19 [nfs-utils PATCH] exportfs: Restore the EAI_NONAME check in host_pton() Scott Mayhew
@ 2015-11-04 21:51 ` Steve Dickson
0 siblings, 0 replies; 2+ messages in thread
From: Steve Dickson @ 2015-11-04 21:51 UTC (permalink / raw)
To: Scott Mayhew; +Cc: linux-nfs
On 10/28/2015 01:19 PM, Scott Mayhew wrote:
> Commit d89e3fc7 removed the EAI_NONAME check altogether instead of just
> moving the NULL check. This causes exportfs -u to incorrectly exit
> with 1 whenever there's more than one MCL_FQDN export in the exportlist.
>
> Signed-off-by: Scott Mayhew <smayhew@redhat.com>
Committed...
steved.
> ---
> support/export/hostname.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/support/export/hostname.c b/support/export/hostname.c
> index 169baa5..7a44d42 100644
> --- a/support/export/hostname.c
> +++ b/support/export/hostname.c
> @@ -134,6 +134,8 @@ host_pton(const char *paddr)
> break;
> }
> return ai;
> + case EAI_NONAME:
> + break;
> case EAI_SYSTEM:
> xlog(D_GENERAL, "%s: failed to convert %s: (%d) %m",
> __func__, paddr, errno);
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2015-11-04 21:51 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-10-28 17:19 [nfs-utils PATCH] exportfs: Restore the EAI_NONAME check in host_pton() Scott Mayhew
2015-11-04 21:51 ` Steve Dickson
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox