From 124b74413e943ab9c02cdff2832d1d81dc9fda87 Mon Sep 17 00:00:00 2001 From: "J. Bruce Fields" Date: Thu, 28 Aug 2008 11:17:34 -0400 Subject: 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 Signed-off-by: Steve Dickson --- support/export/nfsctl.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'support/export/nfsctl.c') diff --git a/support/export/nfsctl.c b/support/export/nfsctl.c index 32d92bd..e2877b9 100644 --- a/support/export/nfsctl.c +++ b/support/export/nfsctl.c @@ -89,11 +89,11 @@ expsetup(struct nfsctl_export *exparg, nfs_export *exp, int unexport) nfs_client *clp = exp->m_client; struct stat stb; - if (stat(exp->m_export.m_path, &stb) < 0) + if (stat(exp->m_export.e_path, &stb) < 0) return 0; memset(exparg, 0, sizeof(*exparg)); - strncpy(exparg->ex_path, exp->m_export.m_path, + strncpy(exparg->ex_path, exp->m_export.e_path, sizeof (exparg->ex_path) - 1); strncpy(exparg->ex_client, clp->m_hostname, sizeof (exparg->ex_client) - 1); -- cgit