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 --- utils/mountd/rmtab.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) (limited to 'utils/mountd/rmtab.c') diff --git a/utils/mountd/rmtab.c b/utils/mountd/rmtab.c index e8aff5a..5787ed6 100644 --- a/utils/mountd/rmtab.c +++ b/utils/mountd/rmtab.c @@ -162,10 +162,8 @@ mountlist_del_all(struct sockaddr_in *sin) } while ((rep = getrmtabent(1, NULL)) != NULL) { if (strcmp(rep->r_client, hp->h_name) == 0 && - (exp = auth_authenticate("umountall", sin, rep->r_path))) { - export_reset(exp); + (exp = auth_authenticate("umountall", sin, rep->r_path))) continue; - } fputrmtabent(fp, rep, NULL); } if (slink_safe_rename(_PATH_RMTABTMP, _PATH_RMTAB) < 0) { -- cgit