Re: [PATCH 2/2] exportfs: do not warn if /etc/exports.d/ does not exist

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



I've tried this issue in different way:)
Steve, could you see this bug?

      https://bugzilla.redhat.com/show_bug.cgi?id=697006

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


[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux