On 09/01/2011 08:40 PM, Masatake YAMATO wrote: > I've tried this issue in different way:) > Steve, could you see this bug? > > https://bugzilla.redhat.com/show_bug.cgi?id=697006 Fair enough.... but the patch in that bz is distro specific, means it would have no affect on other distros... With that said, your patch is a good idea so I will added my todo list steved. > > Masatake YAMATO > > > On Thu, 01 Sep 2011 11:35:56 -0400, Steve Dickson <SteveD@xxxxxxxxxx> wrote >> >> >> On 09/01/2011 11:16 AM, Mike Frysinger wrote: >>> It isn't uncommon for /etc/exports.d/ to not exist, and imo, it's not >>> that big of a deal as often times, a simple /etc/exports is sufficient. >>> So silently skip the case where the dir is missing. >>> >>> Signed-off-by: Mike Frysinger <vapier@xxxxxxxxxx> >> Committed... >> >> steved. >> >>> --- >>> utils/exportfs/exportfs.c | 7 +++++-- >>> 1 files changed, 5 insertions(+), 2 deletions(-) >>> >>> diff --git a/utils/exportfs/exportfs.c b/utils/exportfs/exportfs.c >>> index 986a272..7432a65 100644 >>> --- a/utils/exportfs/exportfs.c >>> +++ b/utils/exportfs/exportfs.c >>> @@ -529,9 +529,12 @@ export_d_read(const char *dname) >>> >>> >>> n = scandir(dname, &namelist, NULL, versionsort); >>> - if (n < 0) >>> + if (n < 0) { >>> + if (errno == ENOENT) >>> + /* Silently return */ >>> + return; >>> xlog(L_NOTICE, "scandir %s: %s", dname, strerror(errno)); >>> - else if (n == 0) >>> + } else if (n == 0) >>> return; >>> >>> for (i = 0; i < n; i++) { >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-nfs" in >> the body of a message to majordomo@xxxxxxxxxxxxxxx >> More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
