nfs-utils: move check for active knfsd to helper function
authorJeff Layton <jlayton@redhat.com>
Sat, 1 Aug 2009 11:31:36 +0000 (07:31 -0400)
committerSteve Dickson <steved@redhat.com>
Sat, 1 Aug 2009 11:31:36 +0000 (07:31 -0400)
nfssvc_setfds checks to see if knfsd is already running. Move this
check to a helper function. Eventually the nfsd code will call this
directly.

Signed-off-by: Jeff Layton <jlayton@redhat.com>
Signed-off-by: Steve Dickson <steved@redhat.com>
utils/nfsd/nfssvc.c

index 025554d..7ecaea9 100644 (file)
  */
 char buf[128];
 
-static void
-nfssvc_setfds(int port, unsigned int ctlbits, char *haddr)
+/*
+ * Are there already sockets configured? If not, then it is safe to try to
+ * open some and pass them through.
+ *
+ * Note: If the user explicitly asked for 'udp', then we should probably check
+ * if that is open, and should open it if not. However we don't yet. All
+ * sockets have to be opened when the first daemon is started.
+ */
+int
+nfssvc_inuse(void)
 {
-       int fd, n, on=1;
-       int udpfd = -1, tcpfd = -1;
-       struct sockaddr_in sin;
+       int fd, n;
 
        fd = open(NFSD_PORTS_FILE, O_RDONLY);
+
+       /* problem opening file, assume that nothing is configured */
        if (fd < 0)
-               return;
+               return 0;
+
        n = read(fd, buf, sizeof(buf));
        close(fd);
-       if (n != 0)
+
+       xlog(D_GENERAL, "knfsd is currently %s", (n > 0) ? "up" : "down");
+
+       return (n > 0);
+}
+
+static void
+nfssvc_setfds(int port, unsigned int ctlbits, char *haddr)
+{
+       int fd, on=1;
+       int udpfd = -1, tcpfd = -1;
+       struct sockaddr_in sin;
+
+       if (nfssvc_inuse())
                return;
-       /* there are no ports currently open, so it is safe to
-        * try to open some and pass them through.
-        * Note: If the user explicitly asked for 'udp', then
-        * we should probably check if that is open, and should
-        * open it if not.  However we don't yet.  All sockets
-        * have to be opened when the first daemon is started.
-        */
+
        fd = open(NFSD_PORTS_FILE, O_WRONLY);
        if (fd < 0)
                return;