Browse Source

[zebra] Bug #351: Don't redistribute routes to ipv4 link-local prefixes

2007-04-07 Paul Jakma <paul.jakma@sun.com>

	* lib/prefix.h: Add define to match IPv4 Link-Local addresses
	* zebra/redistribute.c: (zebra_check_addr) Don't redistribute routes
	  to IPv4 link-local prefixes, fixes bug #351.
	* zebra/redistribute.h: Export zebra_check_addr.
	* zebra/router-id.c: (router_id_bad_address) re-use zebra_check_addr
	  rather than implementing similar logic.
Paul Jakma 14 years ago
parent
commit
6dc686a29e
6 changed files with 24 additions and 11 deletions
  1. 4 0
      lib/ChangeLog
  2. 1 0
      lib/prefix.h
  3. 8 0
      zebra/ChangeLog
  4. 4 2
      zebra/redistribute.c
  5. 2 0
      zebra/redistribute.h
  6. 5 9
      zebra/router-id.c

+ 4 - 0
lib/ChangeLog

@@ -1,3 +1,7 @@
+2007-04-07 Paul Jakma <paul.jakma@sun.com>
+
+	* prefix.h: Add define to match IPv4 Link-Local addresses
+
 2007-03-20 Andrew J. Schorr <ajschorr@alumni.princeton.edu>
 
 	* log.c: (mes_lookup) Make the function more robust: check for

+ 1 - 0
lib/prefix.h

@@ -111,6 +111,7 @@ struct prefix_rd
 
 #define IPV4_NET0(a)    ((((u_int32_t) (a)) & 0xff000000) == 0x00000000)
 #define IPV4_NET127(a)  ((((u_int32_t) (a)) & 0xff000000) == 0x7f000000)
+#define IPV4_LINKLOCAL(a) ((((u_int32_t) (a)) & 0xffff0000) == 0xa9fe0000)
 
 /* Max bit/byte length of IPv6 address. */
 #define IPV6_MAX_BYTELEN    16

+ 8 - 0
zebra/ChangeLog

@@ -1,3 +1,11 @@
+2007-04-07 Paul Jakma <paul.jakma@sun.com>
+
+	* redistribute.c: (zebra_check_addr) Don't redistribute routes
+	  to IPv4 link-local prefixes, fixes bug #351.
+	* redistribute.h: Export zebra_check_addr.
+	* router-id.c: (router_id_bad_address) re-use zebra_check_addr
+	  rather than implementing similar logic.
+
 2007-03-06 Paul Jakma <paul.jakma@sun.com>
 
 	* kernel_socket.c: (ifam_read) Do not update interface metric on

+ 4 - 2
zebra/redistribute.c

@@ -40,7 +40,7 @@
 /* master zebra server structure */
 extern struct zebra_t zebrad;
 
-static int
+int
 zebra_check_addr (struct prefix *p)
 {
   if (p->family == AF_INET)
@@ -50,7 +50,9 @@ zebra_check_addr (struct prefix *p)
       addr = p->u.prefix4.s_addr;
       addr = ntohl (addr);
 
-      if (IPV4_NET127 (addr) || IN_CLASSD (addr))
+      if (IPV4_NET127 (addr)
+          || IN_CLASSD (addr)
+          || IPV4_LINKLOCAL(addr))
 	return 0;
     }
 #ifdef HAVE_IPV6

+ 2 - 0
zebra/redistribute.h

@@ -46,5 +46,7 @@ extern void zebra_interface_address_add_update (struct interface *,
 extern void zebra_interface_address_delete_update (struct interface *,
 						   struct connected *c);
 
+extern int zebra_check_addr (struct prefix *);
+
 #endif /* _ZEBRA_REDISTRIBUTE_H */
 

+ 5 - 9
zebra/router-id.c

@@ -39,6 +39,7 @@
 
 #include "zebra/zserv.h"
 #include "zebra/router-id.h"
+#include "zebra/redistribute.h"
 
 static struct list rid_all_sorted_list;
 static struct list rid_lo_sorted_list;
@@ -63,18 +64,13 @@ router_id_find_node (struct list *l, struct connected *ifc)
 static int
 router_id_bad_address (struct connected *ifc)
 {
-  struct prefix n;
-
   if (ifc->address->family != AF_INET)
     return 1;
-
-  n.u.prefix4.s_addr = htonl (INADDR_LOOPBACK);
-  n.prefixlen = 8;
-  n.family = AF_INET;
-
-  if (prefix_match (&n, ifc->address))
+  
+  /* non-redistributable addresses shouldn't be used for RIDs either */
+  if (!zebra_check_addr (ifc->address))
     return 1;
-
+  
   return 0;
 }