summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorNeilBrown <neilb@suse.com>2016-12-06 13:18:26 -0500
committerSteve Dickson <steved@redhat.com>2016-12-20 13:29:04 -0500
commite54b49cb13f9ecc6a9133957cda4c6d786967841 (patch)
tree4411e3e705855eda65e232c5c28ab7eb4eb586aa
parent6f6fb2fb99cc4408bcde26828d7cea0b1389de83 (diff)
downloadnfs-utils-e54b49cb13f9ecc6a9133957cda4c6d786967841.tar.gz
nfs-utils-e54b49cb13f9ecc6a9133957cda4c6d786967841.tar.xz
nfs-utils-e54b49cb13f9ecc6a9133957cda4c6d786967841.zip
conffile: ignore setting of empty string
A value setting like foo = is now equivalent to not setting "foo" at all. This is likely to be least confusing. Signed-off-by: NeilBrown <neilb@suse.com> Signed-off-by: Steve Dickson <steved@redhat.com>
-rw-r--r--support/nfs/conffile.c2
-rw-r--r--systemd/nfs.conf.man3
2 files changed, 4 insertions, 1 deletions
diff --git a/support/nfs/conffile.c b/support/nfs/conffile.c
index 947bf9b..eaff5f5 100644
--- a/support/nfs/conffile.c
+++ b/support/nfs/conffile.c
@@ -748,6 +748,8 @@ conf_set(int transaction, char *section, char *arg,
{
struct conf_trans *node;
+ if (!value || !*value)
+ return 0;
node = conf_trans_node(transaction, CONF_SET);
if (!node)
return 1;
diff --git a/systemd/nfs.conf.man b/systemd/nfs.conf.man
index 8e2f034..8cf5566 100644
--- a/systemd/nfs.conf.man
+++ b/systemd/nfs.conf.man
@@ -36,7 +36,8 @@ in the current section to
Leading and trailing spaces and tab
are ignored, as are spaces and tabs surrounding the equals sign.
Single and double quotes surrounding the assigned value are also
-removed.
+removed. If the resulting string is empty, the whole assignment
+is ignored.
.PP
Any line starting with
.RB \*(lq # \*(rq