diff options
author | Chuck Lever <chuck.lever@oracle.com> | 2010-05-24 06:01:22 -0400 |
---|---|---|
committer | Steve Dickson <steved@redhat.com> | 2010-05-24 06:01:22 -0400 |
commit | 9e398e49be79af62fca8efb21849d8a2714dc92a (patch) | |
tree | 10be0a9c7d920d45a3fd774b3b772f9a3d13296e /support/export/rmtab.c | |
parent | 2fd2dfd9ebb2de145d50039233bd2a53b7f03062 (diff) | |
download | nfs-utils-9e398e49be79af62fca8efb21849d8a2714dc92a.tar.gz nfs-utils-9e398e49be79af62fca8efb21849d8a2714dc92a.tar.xz nfs-utils-9e398e49be79af62fca8efb21849d8a2714dc92a.zip |
libexport.a: Refactor rmtab_read()
Clean up: Make it easier to add IPv6 support by refactoring part of
rmtab_read() into a helper function.
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Signed-off-by: Steve Dickson <steved@redhat.com>
Diffstat (limited to 'support/export/rmtab.c')
-rw-r--r-- | support/export/rmtab.c | 64 |
1 files changed, 41 insertions, 23 deletions
diff --git a/support/export/rmtab.c b/support/export/rmtab.c index b49e1aa..0ec495c 100644 --- a/support/export/rmtab.c +++ b/support/export/rmtab.c @@ -19,39 +19,57 @@ #include "xio.h" #include "xlog.h" +/* + * See if the entry already exists. If not, + * this was an instantiated wild card, and we + * must add it. + */ +static void +rmtab_read_wildcard(struct rmtabent *rep) +{ + nfs_export *exp, *exp2; + struct hostent *hp; + + hp = gethostbyname(rep->r_client); + if (hp == NULL) + return; + hp = hostent_dup(hp); + if (hp == NULL) + return; + + exp = export_allowed(hp, rep->r_path); + free(hp); + if (exp == NULL) + return; + + exp2 = export_lookup(rep->r_client, exp->m_export.e_path, 0); + if (exp2 == NULL) { + struct exportent ee; + + memset(&ee, 0, sizeof(ee)); + dupexportent(&ee, &exp->m_export); + + ee.e_hostname = rep->r_client; + exp2 = export_create(&ee, 0); + exp2->m_changed = exp->m_changed; + } + exp2->m_mayexport = 1; +} + int rmtab_read(void) { struct rmtabent *rep; - nfs_export *exp = NULL; setrmtabent("r"); while ((rep = getrmtabent(1, NULL)) != NULL) { - struct hostent *hp = NULL; int htype; - + htype = client_gettype(rep->r_client); - if ((htype == MCL_FQDN || htype == MCL_SUBNETWORK) - && (hp = gethostbyname (rep->r_client)) - && (hp = hostent_dup (hp), - exp = export_allowed (hp, rep->r_path))) { - /* see if the entry already exists, otherwise this was an instantiated - * wild card, and we must add it - */ - nfs_export *exp2 = export_lookup(rep->r_client, - exp->m_export.e_path, 0); - if (!exp2) { - struct exportent ee; - dupexportent(&ee, &exp->m_export); - ee.e_hostname = rep->r_client; - exp2 = export_create(&ee, 0); - exp2->m_changed = exp->m_changed; - } - free (hp); - exp2->m_mayexport = 1; - } else if (hp) /* export_allowed failed */ - free(hp); + if (htype == MCL_FQDN || htype == MCL_SUBNETWORK) + rmtab_read_wildcard(rep); } + if (errno == EINVAL) { /* Something goes wrong. We need to fix the rmtab file. */ |