diff options
author | J. Bruce Fields <bfields@citi.umich.edu> | 2008-08-28 11:17:34 -0400 |
---|---|---|
committer | Steve Dickson <steved@redhat.com> | 2008-08-28 11:17:34 -0400 |
commit | 124b74413e943ab9c02cdff2832d1d81dc9fda87 (patch) | |
tree | 7b42e1d53a6bf73e79e8af75f4cd62ee6faec290 /support/nfs/exports.c | |
parent | ec16220910f0bf1bf2b8e1d50e78df7da11d8b1e (diff) | |
download | nfs-utils-124b74413e943ab9c02cdff2832d1d81dc9fda87.tar.gz nfs-utils-124b74413e943ab9c02cdff2832d1d81dc9fda87.tar.xz nfs-utils-124b74413e943ab9c02cdff2832d1d81dc9fda87.zip |
Remove redundant m_path field
Contrary to the comment above its definition, the field m_path always
has the same value as e_path: the *only* modifications of m_path are all
of the form:
strncpy(exp->m_export.m_path, exp->m_export.e_path,
sizeof (exp->m_export.m_path) - 1);
exp->m_export.m_path[sizeof (exp->m_export.m_path) - 1] = '\0';
So m_path is always just a copy of e_path. In places where we need to
store a path to a submount of a CROSSMNT-exported filesystem, as in
cache.c, we just use a local variable.
Signed-off-by: J. Bruce Fields <bfields@citi.umich.edu>
Signed-off-by: Steve Dickson <steved@redhat.com>
Diffstat (limited to 'support/nfs/exports.c')
-rw-r--r-- | support/nfs/exports.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/support/nfs/exports.c b/support/nfs/exports.c index 334c08e..1aaebf4 100644 --- a/support/nfs/exports.c +++ b/support/nfs/exports.c @@ -127,8 +127,6 @@ getexportent(int fromkernel, int fromexports) if (ok <= 0) return NULL; - strncpy (def_ee.m_path, def_ee.e_path, sizeof (def_ee.m_path) - 1); - def_ee.m_path [sizeof (def_ee.m_path) - 1] = '\0'; ok = getexport(exp, sizeof(exp)); } if (ok < 0) { @@ -187,8 +185,6 @@ getexportent(int fromkernel, int fromexports) rpath[sizeof (rpath) - 1] = '\0'; strncpy(ee.e_path, rpath, sizeof (ee.e_path) - 1); ee.e_path[sizeof (ee.e_path) - 1] = '\0'; - strncpy (ee.m_path, ee.e_path, sizeof (ee.m_path) - 1); - ee.m_path [sizeof (ee.m_path) - 1] = '\0'; } return ⅇ @@ -360,8 +356,6 @@ mkexportent(char *hname, char *path, char *options) } strncpy(ee.e_path, path, sizeof (ee.e_path)); ee.e_path[sizeof (ee.e_path) - 1] = '\0'; - strncpy (ee.m_path, ee.e_path, sizeof (ee.m_path) - 1); - ee.m_path [sizeof (ee.m_path) - 1] = '\0'; if (parseopts(options, &ee, 0, NULL) < 0) return NULL; return ⅇ |