From 1ca2e791eee22895907bceac07f2cbd2ea0193e8 Mon Sep 17 00:00:00 2001 From: "J. Bruce Fields" Date: Tue, 6 Oct 2009 18:53:22 -0500 Subject: [PATCH] exports: common exportent initializer Consolidate duplicated initialization code. Signed-off-by: J. Bruce Fields --- support/nfs/exports.c | 59 ++++++++++++++++++++----------------------- 1 file changed, 27 insertions(+), 32 deletions(-) diff --git a/support/nfs/exports.c b/support/nfs/exports.c index 50e83a8..551fc0d 100644 --- a/support/nfs/exports.c +++ b/support/nfs/exports.c @@ -84,6 +84,31 @@ setexportent(char *fname, char *type) first = 1; } +static void init_exportent (struct exportent *ee, int fromkernel) +{ + ee->e_flags = EXPORT_DEFAULT_FLAGS; + /* some kernels assume the default is sync rather than + * async. More recent kernels always report one or other, + * but this test makes sure we assume same as kernel + * Ditto for wgather + */ + if (fromkernel) { + ee->e_flags &= ~NFSEXP_ASYNC; + ee->e_flags &= ~NFSEXP_GATHERED_WRITES; + } + ee->e_anonuid = 65534; + ee->e_anongid = 65534; + ee->e_squids = NULL; + ee->e_sqgids = NULL; + ee->e_mountpoint = NULL; + ee->e_fslocmethod = FSLOC_NONE; + ee->e_fslocdata = NULL; + ee->e_secinfo[0].flav = NULL; + ee->e_nsquids = 0; + ee->e_nsqgids = 0; + ee->e_uuid = NULL; +} + struct exportent * getexportent(int fromkernel, int fromexports) { @@ -102,26 +127,7 @@ getexportent(int fromkernel, int fromexports) has_default_opts = 0; has_default_subtree_opts = 0; - def_ee.e_flags = EXPORT_DEFAULT_FLAGS; - /* some kernels assume the default is sync rather than - * async. More recent kernels always report one or other, - * but this test makes sure we assume same as kernel - * Ditto for wgather - */ - if (fromkernel) { - def_ee.e_flags &= ~NFSEXP_ASYNC; - def_ee.e_flags &= ~NFSEXP_GATHERED_WRITES; - } - def_ee.e_anonuid = 65534; - def_ee.e_anongid = 65534; - def_ee.e_squids = NULL; - def_ee.e_sqgids = NULL; - def_ee.e_mountpoint = NULL; - def_ee.e_fslocmethod = FSLOC_NONE; - def_ee.e_fslocdata = NULL; - def_ee.e_secinfo[0].flav = NULL; - def_ee.e_nsquids = 0; - def_ee.e_nsqgids = 0; + init_exportent(&def_ee, fromkernel); ok = getpath(def_ee.e_path, sizeof(def_ee.e_path)); if (ok <= 0) @@ -334,18 +340,7 @@ mkexportent(char *hname, char *path, char *options) { static struct exportent ee; - ee.e_flags = EXPORT_DEFAULT_FLAGS; - ee.e_anonuid = 65534; - ee.e_anongid = 65534; - ee.e_squids = NULL; - ee.e_sqgids = NULL; - ee.e_mountpoint = NULL; - ee.e_fslocmethod = FSLOC_NONE; - ee.e_fslocdata = NULL; - ee.e_secinfo[0].flav = NULL; - ee.e_nsquids = 0; - ee.e_nsqgids = 0; - ee.e_uuid = NULL; + init_exportent(&ee, 0); xfree(ee.e_hostname); ee.e_hostname = xstrdup(hname); -- 2.39.2