]> git.decadent.org.uk Git - nfs-utils.git/blobdiff - utils/statd/monitor.c
Tell NFS/lockd client what that local state number is.
[nfs-utils.git] / utils / statd / monitor.c
index b45208205b7baff811db49ba1e1d5d3132eb9989..bbc1dec67f423de605cfbfd228fe09cd8e8a7df6 100644 (file)
@@ -43,11 +43,11 @@ sm_mon_1_svc(struct mon *argp, struct svc_req *rqstp)
        int             fd;
        notify_list     *clnt;
        struct in_addr  my_addr;
+       char            *dnsname;
 #ifdef RESTRICTED_STATD
        struct in_addr  caller;
-#else
-       struct hostent  *hostinfo = NULL;
 #endif
+       struct hostent  *hostinfo = NULL;
 
        /* Assume that we'll fail. */
        result.res_stat = STAT_FAIL;
@@ -104,6 +104,12 @@ sm_mon_1_svc(struct mon *argp, struct svc_req *rqstp)
                goto failure;
        }
 #endif
+#else
+       if (!(hostinfo = gethostbyname(my_name))) {
+               note(N_WARNING, "gethostbyname error for %s", my_name);
+               goto failure;
+       } else
+               my_addr = *(struct in_addr *) hostinfo->h_addr;
 #endif
        /*
         * Check hostnames.  If I can't look them up, I won't monitor.  This
@@ -116,21 +122,27 @@ sm_mon_1_svc(struct mon *argp, struct svc_req *rqstp)
                     "or starting '.': %s", mon_name);
                note(N_CRIT, "POSSIBLE SPOOF/ATTACK ATTEMPT!");
                goto failure;
-       } else if (gethostbyname(mon_name) == NULL) {
+       } else if ((hostinfo = gethostbyname(mon_name)) == NULL) {
                note(N_WARNING, "gethostbyname error for %s", mon_name);
                goto failure;
        }
-#ifndef RESTRICTED_STATD
-       if (!(hostinfo = gethostbyname(my_name))) {
-               note(N_WARNING, "gethostbyname error for %s", my_name);
-               goto failure;
-       } else
-               my_addr = *(struct in_addr *) hostinfo->h_addr;
-#endif
 
        /*
         * Hostnames checked OK.
-        * Now check to see if this is a duplicate, and warn if so.
+        * Now choose a hostname to use for matching.  We cannot
+        * really trust much in the incoming NOTIFY, so to make
+        * sure that multi-homed hosts work nicely, we get an
+        * FQDN now, and use that for matching
+        */
+       hostinfo = gethostbyaddr(hostinfo->h_addr,
+                                hostinfo->h_length,
+                                hostinfo->h_addrtype);
+       if (hostinfo)
+               dnsname = xstrdup(hostinfo->h_name);
+       else
+               dnsname = xstrdup(my_name);
+
+       /* Now check to see if this is a duplicate, and warn if so.
         * I will also return STAT_FAIL. (I *think* this is how I should
         * handle it.)
         *
@@ -139,27 +151,24 @@ sm_mon_1_svc(struct mon *argp, struct svc_req *rqstp)
         * I'll just do a quickie success return and things should
         * be happy.
         */
-       if (rtnl) {
-               notify_list    *temp = rtnl;
-
-               while ((temp = nlist_gethost(temp, mon_name, 0))) {
-                       if (matchhostname(NL_MY_NAME(temp), my_name) &&
-                               NL_MY_PROC(temp) == id->my_proc &&
-                               NL_MY_PROG(temp) == id->my_prog &&
-                               NL_MY_VERS(temp) == id->my_vers) {
-                               /* Hey!  We already know you guys! */
-                               dprintf(N_DEBUG,
-                                       "Duplicate SM_MON request for %s "
-                                       "from procedure on %s",
-                                       mon_name, my_name);
+       clnt = rtnl;
 
-                               /* But we'll let you pass anyway. */
-                               result.res_stat = STAT_SUCC;
-                               result.state = MY_STATE;
-                               return (&result);
-                       }
-                       temp = NL_NEXT(temp);
+       while ((clnt = nlist_gethost(clnt, mon_name, 0))) {
+               if (matchhostname(NL_MY_NAME(clnt), my_name) &&
+                   NL_MY_PROC(clnt) == id->my_proc &&
+                   NL_MY_PROG(clnt) == id->my_prog &&
+                   NL_MY_VERS(clnt) == id->my_vers &&
+                   memcmp(NL_PRIV(clnt), argp->priv, SM_PRIV_SIZE) == 0) {
+                       /* Hey!  We already know you guys! */
+                       dprintf(N_DEBUG,
+                               "Duplicate SM_MON request for %s "
+                               "from procedure on %s",
+                               mon_name, my_name);
+
+                       /* But we'll let you pass anyway. */
+                       goto success;
                }
+               clnt = NL_NEXT(clnt);
        }
 
        /*
@@ -176,13 +185,14 @@ sm_mon_1_svc(struct mon *argp, struct svc_req *rqstp)
        NL_MY_VERS(clnt) = id->my_vers;
        NL_MY_PROC(clnt) = id->my_proc;
        memcpy(NL_PRIV(clnt), argp->priv, SM_PRIV_SIZE);
+       clnt->dns_name = dnsname;
 
        /*
         * Now, Create file on stable storage for host.
         */
 
-       path=xmalloc(strlen(SM_DIR)+strlen(mon_name)+2);
-       sprintf(path, "%s/%s", SM_DIR, mon_name);
+       path=xmalloc(strlen(SM_DIR)+strlen(dnsname)+2);
+       sprintf(path, "%s/%s", SM_DIR, dnsname);
        if ((fd = open(path, O_WRONLY|O_SYNC|O_CREAT|O_APPEND,
                       S_IRUSR|S_IWUSR)) < 0) {
                /* Didn't fly.  We won't monitor. */
@@ -210,10 +220,20 @@ sm_mon_1_svc(struct mon *argp, struct svc_req *rqstp)
        ha_callout("add-client", mon_name, my_name, -1);
        nlist_insert(&rtnl, clnt);
        close(fd);
-
+       dprintf(N_DEBUG, "MONITORING %s for %s", mon_name, my_name);
+ success:
        result.res_stat = STAT_SUCC;
+       /* SUN's sm_inter.x says this should be "state number of local site".
+        * X/Open says '"state" will be contain the state of the remote NSM.'
+        * href=http://www.opengroup.org/onlinepubs/9629799/SM_MON.htm
+        * Linux lockd currently (2.6.21 and prior) ignores whatever is
+        * returned, and given the above contraction, it probably always will..
+        * So we just return what we always returned.  If possible, we
+        * have already told lockd about our state number via a sysctl.
+        * If lockd wants the remote state, it will need to
+        * use SM_STAT (and prayer).
+        */
        result.state = MY_STATE;
-       dprintf(N_DEBUG, "MONITORING %s for %s", mon_name, my_name);
        return (&result);
 
 failure:
@@ -269,6 +289,7 @@ void load_state(void)
                        NL_MY_PROG(clnt) = prog;
                        NL_MY_VERS(clnt) = vers;
                        NL_MY_PROC(clnt) = proc;
+                       clnt->dns_name = xstrdup(de->d_name);
                        memcpy(NL_PRIV(clnt), priv, SM_PRIV_SIZE);
                        nlist_insert(&rtnl, clnt);
                }