noip.c: Have parsers fail if there's trailing junk.
[preload-hacks] / noip.c
diff --git a/noip.c b/noip.c
index a1e141a..12c9f9c 100644 (file)
--- a/noip.c
+++ b/noip.c
@@ -34,6 +34,7 @@
 #include <assert.h>
 #include <ctype.h>
 #include <errno.h>
+#include <stdarg.h>
 #include <stddef.h>
 #include <stdio.h>
 #include <stdlib.h>
@@ -125,7 +126,8 @@ static aclnode *connect_real,  **connect_tail = &connect_real;
   _(recvfrom, ssize_t, (int, void *buf, size_t, int,                   \
                        struct sockaddr *from, socklen_t *fromlen))     \
   _(sendmsg, ssize_t, (int, const struct msghdr *, int))               \
-  _(recvmsg, ssize_t, (int, struct msghdr *, int))
+  _(recvmsg, ssize_t, (int, struct msghdr *, int))                     \
+  _(ioctl, int, (int, unsigned long, ...))
 
 /* Function pointers to set up. */
 #define DECL(imp, ret, args) static ret (*real_##imp) args;
@@ -159,8 +161,10 @@ static void import(void)
 /* Debugging */
 #ifdef DEBUG
 #  define D(body) { if (debug) { body } }
+#  define Dpid pid_t pid = debug ? getpid() : -1
 #else
 #  define D(body) ;
+#  define Dpid
 #endif
 
 /* Preservation of error status */
@@ -509,7 +513,7 @@ static void dump_aclnode(const aclnode *a)
   const char *p;
   int plen;
 
-  fprintf(stderr, "noip:   %c ", a->act ? '+' : '-');
+  fprintf(stderr, "noip(%d):   %c ", getpid(), a->act ? '+' : '-');
   plen = common_prefix_length(a->af, &a->minaddr, &a->maxaddr);
   p = inet_ntop(a->af, &a->minaddr, buf, sizeof(buf));
   fprintf(stderr, strchr(p, ':') ? "[%s]" : "%s", p);
@@ -534,7 +538,7 @@ static void dump_acl(const aclnode *a)
     dump_aclnode(a);
     act = a->act;
   }
-  fprintf(stderr, "noip:   [default policy: %s]\n",
+  fprintf(stderr, "noip(%d):   [default policy: %s]\n", getpid(),
          act == ALLOW ? "DENY" : "ALLOW");
 }
 
@@ -545,20 +549,23 @@ static int acl_allows_p(const aclnode *a, const struct sockaddr *sa)
 {
   unsigned short port = port_from_sockaddr(sa);
   int act = ALLOW;
+  Dpid;
 
   D({ char buf[ADDRBUFSZ];
-      fprintf(stderr, "noip: check %s\n",
+      fprintf(stderr, "noip(%d): check %s\n", pid,
              present_sockaddr(sa, 0, buf, sizeof(buf))); })
   for (; a; a = a->next) {
     D( dump_aclnode(a); )
     if (sockaddr_in_range_p(sa, &a->minaddr, &a->maxaddr) &&
        a->minport <= port && port <= a->maxport) {
-      D( fprintf(stderr, "noip: aha!  %s\n", a->act ? "ALLOW" : "DENY"); )
+      D( fprintf(stderr, "noip(%d): aha!  %s\n", pid,
+                a->act ? "ALLOW" : "DENY"); )
       return (a->act);
     }
     act = a->act;
   }
-  D( fprintf(stderr, "noip: nothing found: %s\n", act ? "DENY" : "ALLOW"); )
+  D( fprintf(stderr, "noip(%d): nothing found: %s\n", pid,
+            act ? "DENY" : "ALLOW"); )
   return (!act);
 }
 
@@ -632,7 +639,7 @@ static int encode_inet_addr(struct sockaddr_un *sun,
   char buf[ADDRBUFSZ];
   int rc;
 
-  D( fprintf(stderr, "noip: encode %s (%s)",
+  D( fprintf(stderr, "noip(%d): encode %s (%s)", getpid(),
             present_sockaddr(sa, 0, buf, sizeof(buf)),
             want == WANT_EXISTING ? "EXISTING" : "FRESH"); )
   sun->sun_family = AF_UNIX;
@@ -696,7 +703,7 @@ static int decode_inet_addr(struct sockaddr *sa, int af_hint,
   if (len > sizeof(*sun)) return (-1);
   ((char *)sun)[len] = 0;
   nn = strlen(sun->sun_path);
-  D( fprintf(stderr, "noip: decode `%s'", sun->sun_path); )
+  D( fprintf(stderr, "noip(%d): decode `%s'", getpid(), sun->sun_path); )
   if (af_hint && !sun->sun_path[0]) {
     wildcard_address(af_hint, sa);
     D( fprintf(stderr, " -- unbound socket\n"); )
@@ -715,9 +722,11 @@ static int decode_inet_addr(struct sockaddr *sa, int af_hint,
 
 /* SK is (or at least might be) a Unix-domain socket we created when an
  * Internet socket was asked for.  We've decided it should be an Internet
- * socket after all, with family AF_HINT, so convert it.
+ * socket after all, with family AF_HINT, so convert it.  If TMP is not null,
+ * then don't replace the existing descriptor: store the new socket in *TMP
+ * and return zero.
  */
-static int fixup_real_ip_socket(int sk, int af_hint)
+static int fixup_real_ip_socket(int sk, int af_hint, int *tmp)
 {
   int nsk;
   int type;
@@ -761,18 +770,22 @@ static int fixup_real_ip_socket(int sk, int af_hint)
 } while (0);
   OPTS(FIX)
 #undef FIX
-  if ((f = fcntl(sk, F_GETFL)) < 0 ||
-      (fd = fcntl(sk, F_GETFD)) < 0 ||
-      fcntl(nsk, F_SETFL, f) < 0 ||
-      dup2(nsk, sk) < 0) {
+  if (tmp)
+    *tmp = nsk;
+  else {
+    if ((f = fcntl(sk, F_GETFL)) < 0 ||
+       (fd = fcntl(sk, F_GETFD)) < 0 ||
+       fcntl(nsk, F_SETFL, f) < 0 ||
+       dup2(nsk, sk) < 0) {
+      close(nsk);
+      return (-1);
+    }
+    unlink(sun.sun_path);
     close(nsk);
-    return (-1);
-  }
-  unlink(sun.sun_path);
-  close(nsk);
-  if (fcntl(sk, F_SETFD, fd) < 0) {
-    perror("noip: fixup_real_ip_socket F_SETFD");
-    abort();
+    if (fcntl(sk, F_SETFD, fd) < 0) {
+      perror("noip: fixup_real_ip_socket F_SETFD");
+      abort();
+    }
   }
   return (0);
 }
@@ -788,7 +801,7 @@ static int do_implicit_bind(int sk, const struct sockaddr **sa,
   socklen_t mylen = sizeof(*sun);
 
   if (acl_allows_p(connect_real, *sa)) {
-    if (fixup_real_ip_socket(sk, (*sa)->sa_family)) return (-1);
+    if (fixup_real_ip_socket(sk, (*sa)->sa_family, 0)) return (-1);
   } else {
     if (real_getsockname(sk, SA(sun), &mylen) < 0) return (-1);
     if (sun->sun_family == AF_UNIX) {
@@ -1031,6 +1044,7 @@ static void parse_acl_line(char **pp, aclnode ***tail)
        maxaddr = minaddr;
        mask_address(af, &minaddr, n, 0);
        mask_address(af, &maxaddr, n, 1);
+       RESCAN(del);
       } else
        maxaddr = minaddr;
       parse_ports(&p, &minport, &maxport);
@@ -1040,11 +1054,12 @@ static void parse_acl_line(char **pp, aclnode ***tail)
     if (*p != ',') break;
     if (*p) p++;
   }
+  if (*p) goto bad;
   *pp = p;
   return;
 
 bad:
-  D( fprintf(stderr, "noip: bad acl spec (ignored)\n"); )
+  D( fprintf(stderr, "noip(%d): bad acl spec (ignored)\n", getpid()); )
   return;
 }
 
@@ -1061,11 +1076,12 @@ static void parse_autoports(char **pp)
   if (*p != '-') goto bad; p++;
   NEXTNUMBER(q, del); y = strtoul(q, 0, 0); RESCAN(del);
   minautoport = x; maxautoport = y;
+  SKIPSPC; if (*p) goto bad;
   *pp = p;
   return;
 
 bad:
-  D( fprintf(stderr, "bad port range (ignored)\n"); )
+  D( fprintf(stderr, "noip(%d): bad port range (ignored)\n", getpid()); )
   return;
 }
 
@@ -1089,6 +1105,7 @@ static void readconfig(void)
   char buf[1024];
   size_t n;
   char *p, *q, *cmd;
+  Dpid;
 
   parse_acl_env("NOIP_REALBIND_BEFORE", &bind_tail);
   parse_acl_env("NOIP_REALCONNECT_BEFORE", &connect_tail);
@@ -1099,11 +1116,11 @@ static void readconfig(void)
   }
   if ((p = getenv("NOIP_CONFIG")) == 0)
     snprintf(p = buf, sizeof(buf), "%s/.noip", home());
-  D( fprintf(stderr, "noip: config file: %s\n", p); )
+  D( fprintf(stderr, "noip(%d): config file: %s\n", pid, p); )
 
   if ((fp = fopen(p, "r")) == 0) {
-    D( fprintf(stderr, "noip: couldn't read config: %s\n",
-              strerror(errno)); )
+    D( fprintf(stderr, "noip(%d): couldn't read config: %s\n",
+              pid, strerror(errno)); )
     goto done;
   }
   while (fgets(buf, sizeof(buf), fp)) {
@@ -1144,12 +1161,12 @@ done:
     snprintf(buf, sizeof(buf), "%s/noip-%s", tmpdir(), user());
     sockdir = xstrdup(buf);
   }
-  D( fprintf(stderr, "noip: socketdir: %s\n", sockdir);
-     fprintf(stderr, "noip: autoports: %u-%u\n",
-            minautoport, maxautoport);
-     fprintf(stderr, "noip: realbind acl:\n");
+  D( fprintf(stderr, "noip(%d): socketdir: %s\n", pid, sockdir);
+     fprintf(stderr, "noip(%d): autoports: %u-%u\n",
+            pid, minautoport, maxautoport);
+     fprintf(stderr, "noip(%d): realbind acl:\n", pid);
      dump_acl(bind_real);
-     fprintf(stderr, "noip: realconnect acl:\n");
+     fprintf(stderr, "noip(%d): realconnect acl:\n", pid);
      dump_acl(connect_real); )
 }
 
@@ -1189,7 +1206,7 @@ int bind(int sk, const struct sockaddr *sa, socklen_t len)
   if (family_known_p(sa->sa_family)) {
     PRESERVING_ERRNO({
       if (acl_allows_p(bind_real, sa)) {
-       if (fixup_real_ip_socket(sk, sa->sa_family))
+       if (fixup_real_ip_socket(sk, sa->sa_family, 0))
          return (-1);
       } else {
        encode_inet_addr(&sun, sa, WANT_FRESH);
@@ -1361,6 +1378,36 @@ int setsockopt(int sk, int lev, int opt, const void *p, socklen_t len)
   return (real_setsockopt(sk, lev, opt, p, len));
 }
 
+int ioctl(int fd, unsigned long op, ...)
+{
+  va_list ap;
+  void *arg;
+  int sk;
+  int rc;
+
+  va_start(ap, op);
+  arg = va_arg(ap, void *);
+
+  switch (op) {
+    case SIOCGIFADDR:
+    case SIOCGIFBRDADDR:
+    case SIOCGIFDSTADDR:
+    case SIOCGIFNETMASK:
+      PRESERVING_ERRNO({
+       if (fixup_real_ip_socket(fd, AF_INET, &sk)) goto real;
+      });
+      rc = real_ioctl(sk, op, arg);
+      PRESERVING_ERRNO({ close(sk); });
+      break;
+    default:
+    real:
+      rc = real_ioctl(fd, op, arg);
+      break;
+  }
+  va_end(ap);
+  return (rc);
+}
+
 /*----- Initialization ----------------------------------------------------*/
 
 /* Clean up the socket directory, deleting stale sockets. */
@@ -1371,6 +1418,7 @@ static void cleanup_sockdir(void)
   address addr;
   struct sockaddr_un sun;
   struct stat st;
+  Dpid;
 
   if ((dir = opendir(sockdir)) == 0) return;
   sun.sun_family = AF_UNIX;
@@ -1381,13 +1429,13 @@ static void cleanup_sockdir(void)
     if (decode_inet_addr(&addr.sa, 0, &sun, SUN_LEN(&sun)) ||
        stat(sun.sun_path, &st) ||
        !S_ISSOCK(st.st_mode)) {
-      D( fprintf(stderr, "noip: ignoring unknown socketdir entry `%s'\n",
-                sun.sun_path); )
+      D( fprintf(stderr, "noip(%d): ignoring unknown socketdir entry `%s'\n",
+                pid, sun.sun_path); )
       continue;
     }
     if (unix_socket_status(&sun, 0) == STALE) {
-      D( fprintf(stderr, "noip: clearing away stale socket %s\n",
-                d->d_name); )
+      D( fprintf(stderr, "noip(%d): clearing away stale socket %s\n",
+                pid, d->d_name); )
       unlink(sun.sun_path);
     }
   }
@@ -1402,7 +1450,9 @@ static void get_local_ipaddrs(void)
   struct ifaddrs *ifa_head, *ifa;
   ipaddr a;
   int i;
+  Dpid;
 
+  D( fprintf(stderr, "noip(%d): fetching local addresses...\n", pid); )
   if (getifaddrs(&ifa_head)) { perror("getifaddrs"); return; }
   for (n_local_ipaddrs = 0, ifa = ifa_head;
        n_local_ipaddrs < MAX_LOCAL_IPADDRS && ifa;
@@ -1411,7 +1461,8 @@ static void get_local_ipaddrs(void)
       continue;
     ipaddr_from_sockaddr(&a, ifa->ifa_addr);
     D({ char buf[ADDRBUFSZ];
-       fprintf(stderr, "noip: local addr %s = %s", ifa->ifa_name,
+       fprintf(stderr, "noip(%d):   local addr %s = %s", pid,
+               ifa->ifa_name,
                inet_ntop(ifa->ifa_addr->sa_family, &a,
                          buf, sizeof(buf))); })
     for (i = 0; i < n_local_ipaddrs; i++) {