Cleaned up handling of interface patterns:

   o  Parsing of interface patterns moved to generic code,
      introduced this_ipatt which works similarly to this_iface.
   o  Interface patterns now support selection by both interface
      names and primary IP addresses.
   o  Proto `direct' updated.
   o  RIP updated as well, it also seems the memory corruption
      bug there is gone.

4 files changed, 47 insertions, 34 deletions

diff --git a/nest/config.Y b/nest/config.Y
index bb1328cc..0c653211 100644
--- a/nest/config.Y
+++ b/nest/config.Y
@@ -9,12 +9,11 @@
 CF_HDR
 
 static struct proto_config *this_proto;
+static struct iface_patt *this_ipatt;
 
 #include "nest/rt-dev.h"
 #include "nest/password.h"
 
-void rt_dev_add_iface(char *);
-
 CF_DECLS
 
 CF_KEYWORDS(ROUTER, ID, PROTOCOL, PREFERENCE, DISABLED, DEBUG, ALL, OFF, DIRECT)
@@ -102,18 +101,23 @@ rtable:
    }
  ;
 
+/* Interface patterns */
+
+iface_patt:
+   TEXT { this_ipatt->pattern = $1; this_ipatt->prefix = IPA_NONE; this_ipatt->pxlen = 0; }
+ | IPA pxlen { this_ipatt->pattern = NULL; this_ipatt->prefix = $1; this_ipatt->pxlen = $2; }
+ | TEXT IPA pxlen { this_ipatt->pattern = $1; this_ipatt->prefix = $2; this_ipatt->pxlen = $3; }
+ ;
+
 /* Direct device route protocol */
 
 CF_ADDTO(proto, dev_proto '}')
 
 dev_proto_start: proto_start DIRECT {
      struct rt_dev_config *p = proto_config_new(&proto_device, sizeof(struct rt_dev_config));
-     struct iface_patt *k = cfg_alloc(sizeof(struct iface_patt));
      this_proto = &p->c;
      p->c.preference = DEF_PREF_DIRECT;
      init_list(&p->iface_list);
-     k->pattern = "*";
-     add_tail(&p->iface_list, &k->n);
    }
  ;
 
@@ -123,15 +127,26 @@ dev_proto:
  | dev_proto dev_iface_list ';'
  ;
 
-dev_iface_list:
-   INTERFACE TEXT {
-     /* FIXME: Beware of obscure semantics. */
-     init_list(&((struct rt_dev_config *) this_proto)->iface_list);
-     rt_dev_add_iface($2);
+dev_iface_entry_init:
+   /* EMPTY */ {
+     struct rt_dev_config *p = (void *) this_proto;
+     struct iface_patt *k = cfg_allocz(sizeof(struct iface_patt));
+     add_tail(&p->iface_list, &k->n);
+     this_ipatt = k;
    }
- | dev_iface_list ',' TEXT { rt_dev_add_iface($3); }
  ;
 
+dev_iface_entry:
+   dev_iface_entry_init iface_patt
+ ;
+
+dev_iface_list:
+   INTERFACE dev_iface_entry
+ | dev_iface_list ',' dev_iface_entry
+ ;
+
+/* Password lists */
+
 password_begin: 
    PASSWORD TEXT {
      last_password_item = cfg_alloc(sizeof (struct password_item));
@@ -162,14 +177,4 @@ password_list:
 
 CF_CODE
 
-void
-rt_dev_add_iface(char *n)
-{
-  struct rt_dev_config *p = (void *) this_proto;
-  struct iface_patt *k = cfg_alloc(sizeof(struct iface_patt));
-
-  k->pattern = cfg_strdup(n);
-  add_tail(&p->iface_list, &k->n);
-}
-
 CF_END
diff --git a/nest/iface.c b/nest/iface.c
index 19ed7fbd..3a5d9c6b 100644
--- a/nest/iface.c
+++ b/nest/iface.c
@@ -588,13 +588,19 @@ iface_patt_match(list *l, struct iface *i)
     {
       char *t = p->pattern;
       int ok = 1;
-      if (*t == '-')
+      if (t)
 	{
-	  t++;
-	  ok = 0;
+	  if (*t == '-')
+	    {
+	      t++;
+	      ok = 0;
+	    }
+	  if (!patmatch(t, i->name))
+	    continue;
 	}
-      if (patmatch(t, i->name))
-	return ok ? p : NULL;
+      if (!i->addr || !ipa_in_net(i->addr->ip, p->prefix, p->pxlen))
+	continue;
+      return ok ? p : NULL;
     }
   return NULL;
 }
@@ -608,7 +614,10 @@ iface_patts_equal(list *a, list *b, int (*comp)(struct iface_patt *, struct ifac
   y = HEAD(*b);
   while (x->n.next && y->n.next)
     {
-      if (strcmp(x->pattern, y->pattern) || comp && !comp(x, y))
+      if (strcmp(x->pattern, y->pattern) ||
+	  !ipa_equal(x->prefix, y->prefix) ||
+	  x->pxlen != y->pxlen ||
+	  comp && !comp(x, y))
 	return 0;
       x = (void *) x->n.next;
       y = (void *) y->n.next;
diff --git a/nest/iface.h b/nest/iface.h
index 1d26e7b8..9f3a2395 100644
--- a/nest/iface.h
+++ b/nest/iface.h
@@ -122,13 +122,10 @@ void neigh_prune(void);
 struct iface_patt {
   node n;
   byte *pattern;			/* Interface name pattern */
+  ip_addr prefix;			/* Interface prefix */
+  int pxlen;
 
-  /* Protocol-specific data follow, but keep them like this:
-     struct rip_iface_patt {
-        struct iface_patt i;
-	whatever you (need);
-     }
-   */
+  /* Protocol-specific data follow after this structure */
 };
 
 struct iface_patt *iface_patt_match(list *, struct iface *);
diff --git a/nest/rt-dev.c b/nest/rt-dev.c
index 63548fc7..c07b5de6 100644
--- a/nest/rt-dev.c
+++ b/nest/rt-dev.c
@@ -23,7 +23,9 @@ dev_ifa_notify(struct proto *p, unsigned c, struct ifa *ad)
 {
   struct rt_dev_config *P = (void *) p->cf;
 
-  if (!iface_patt_match(&P->iface_list, ad->iface))
+  if (!EMPTY_LIST(P->iface_list) &&
+      !iface_patt_match(&P->iface_list, ad->iface))
+    /* Empty list is automagically treated as "*" */
     return;
   if (c & IF_CHANGE_DOWN)
     {