On 11/18/2016 03:58 AM, Jianhong.Yin wrote:
> before this commit, run 'exportfs -s' will output some
> options more than one time. for example:
> /path *(rw,no_root_squash,rw,secure,no_root_squash,no_all_squash)
> the options rw,no_root_squash,etc will be print twice.
> the reason is secinfo_show() will show these security options again
> this patch remove these redundant outputs.
>
> Signed-off-by: Jianhong Yin <yin-jianhong@xxxxxxx>
Committed!
steved.
> ---
> utils/exportfs/exportfs.c | 18 ++++++------------
> 1 file changed, 6 insertions(+), 12 deletions(-)
>
> diff --git a/utils/exportfs/exportfs.c b/utils/exportfs/exportfs.c
> index 98368a5..b11f06b 100644
> --- a/utils/exportfs/exportfs.c
> +++ b/utils/exportfs/exportfs.c
> @@ -747,26 +747,20 @@ dump(int verbose, int export_format)
> continue;
> }
> c = '(';
> - if (ep->e_flags & NFSEXP_READONLY)
> - c = dumpopt(c, "ro");
> - else
> - c = dumpopt(c, "rw");
> if (ep->e_flags & NFSEXP_ASYNC)
> c = dumpopt(c, "async");
> + else
> + c = dumpopt(c, "sync");
> if (ep->e_flags & NFSEXP_GATHERED_WRITES)
> c = dumpopt(c, "wdelay");
> + else
> + c = dumpopt(c, "no_wdelay");
> if (ep->e_flags & NFSEXP_NOHIDE)
> c = dumpopt(c, "nohide");
> + else
> + c = dumpopt(c, "hide");
> if (ep->e_flags & NFSEXP_CROSSMOUNT)
> c = dumpopt(c, "crossmnt");
> - if (ep->e_flags & NFSEXP_INSECURE_PORT)
> - c = dumpopt(c, "insecure");
> - if (ep->e_flags & NFSEXP_ROOTSQUASH)
> - c = dumpopt(c, "root_squash");
> - else
> - c = dumpopt(c, "no_root_squash");
> - if (ep->e_flags & NFSEXP_ALLSQUASH)
> - c = dumpopt(c, "all_squash");
> if (ep->e_flags & NFSEXP_NOSUBTREECHECK)
> c = dumpopt(c, "no_subtree_check");
> if (ep->e_flags & NFSEXP_NOAUTHNLM)
>
--
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
