summaryrefslogtreecommitdiffstats
path: root/utils/mountd/auth.c
diff options
context:
space:
mode:
authorJ. Bruce Fields <bfields@redhat.com>2012-05-03 15:01:20 -0400
committerSteve Dickson <steved@redhat.com>2012-05-03 15:12:09 -0400
commit236025253c6ec534cd816390464435ec18c499f7 (patch)
tree78980950e3def72eef76c9f2313288397e8a7e18 /utils/mountd/auth.c
parent005c49a063544b4005b5b7c3c28ed48c197f66ac (diff)
downloadnfs-utils-236025253c6ec534cd816390464435ec18c499f7.tar.gz
nfs-utils-236025253c6ec534cd816390464435ec18c499f7.tar.xz
nfs-utils-236025253c6ec534cd816390464435ec18c499f7.zip
mountd: add trivial helpers for client-matching
From: "J. Bruce Fields" <bfields@redhat.com> Pull out a tiny bit of common logic from three functions. Possibly minor overkill, but simplifies the next patch. Signed-off-by: J. Bruce Fields <bfields@redhat.com> Signed-off-by: Steve Dickson <steved@redhat.com>
Diffstat (limited to 'utils/mountd/auth.c')
-rw-r--r--utils/mountd/auth.c21
1 files changed, 18 insertions, 3 deletions
diff --git a/utils/mountd/auth.c b/utils/mountd/auth.c
index 3843756..688bf78 100644
--- a/utils/mountd/auth.c
+++ b/utils/mountd/auth.c
@@ -131,6 +131,23 @@ get_client_hostname(const struct sockaddr *caller, struct addrinfo *ai,
return strdup("DEFAULT");
}
+bool ipaddr_client_matches(nfs_export *exp, struct addrinfo *ai)
+{
+ return client_check(exp->m_client, ai);
+}
+
+bool namelist_client_matches(nfs_export *exp, char *dom)
+{
+ return client_member(dom, exp->m_client->m_hostname);
+}
+
+bool client_matches(nfs_export *exp, char *dom, struct addrinfo *ai)
+{
+ if (use_ipaddr)
+ return ipaddr_client_matches(exp, ai);
+ return namelist_client_matches(exp, dom);
+}
+
/* return static nfs_export with details filled in */
static nfs_export *
auth_authenticate_newcache(const struct sockaddr *caller,
@@ -155,9 +172,7 @@ auth_authenticate_newcache(const struct sockaddr *caller,
for (exp = exportlist[i].p_head; exp; exp = exp->m_next) {
if (strcmp(path, exp->m_export.e_path))
continue;
- if (!use_ipaddr && !client_member(my_client.m_hostname, exp->m_client->m_hostname))
- continue;
- if (use_ipaddr && !client_check(exp->m_client, ai))
+ if (!client_matches(exp, my_client.m_hostname, ai))
continue;
if (exp->m_export.e_flags & NFSEXP_V4ROOT)
/* not acceptable for v[23] export */