Browse Source

2005-08-09 Paul Jakma <paul.jakma@sun.com>

	* ospf6_asbr.c: (ospf6_asbr_redistribute_add) Fix sense of testing
	  of route_map_apply return code, it can return many things other
	  than RMAP_MATCH which do not indicate DENY. Should test explicitly
	  for equality to RMAP_DENYMATCH instead.

	[backport candidate]
paul 15 years ago
parent
commit
47828747f4
2 changed files with 8 additions and 1 deletions
  1. 7 0
      ospf6d/ChangeLog
  2. 1 1
      ospf6d/ospf6_asbr.c

+ 7 - 0
ospf6d/ChangeLog

@@ -1,3 +1,10 @@
+2005-08-09 Paul Jakma <paul.jakma@sun.com>
+
+	* ospf6_asbr.c: (ospf6_asbr_redistribute_add) Fix sense of testing
+	  of route_map_apply return code, it can return many things other
+	  than RMAP_MATCH which do not indicate DENY. Should test explicitly
+	  for equality to RMAP_DENYMATCH instead.
+
 2005-06-24 Harald Welte <laforge@gnumonks.org>
 
 	* ospf6_abr.[ch], ospf6_area.[ch]: Add area filter-list (in|out)

+ 1 - 1
ospf6d/ospf6_asbr.c

@@ -478,7 +478,7 @@ ospf6_asbr_redistribute_add (int type, int ifindex, struct prefix *prefix,
 
       ret = route_map_apply (ospf6->rmap[type].map, prefix,
                              RMAP_OSPF6, &troute);
-      if (ret != RMAP_MATCH)
+      if (ret == RMAP_DENYMATCH)
         {
           if (IS_OSPF6_DEBUG_ASBR)
             zlog_debug ("Denied by route-map \"%s\"", ospf6->rmap[type].name);