Always recurse in pass2
authorAlan T. DeKok <aland@freeradius.org>
Thu, 9 Jul 2015 17:26:13 +0000 (13:26 -0400)
committerAlan T. DeKok <aland@freeradius.org>
Thu, 9 Jul 2015 17:26:13 +0000 (13:26 -0400)
even if the node is marked as PASS2_FIXUP_NONE.  So that
xlat's can be compiled to attribute references

src/main/modcall.c

index 7d4f216..7b54f35 100644 (file)
@@ -3164,17 +3164,9 @@ static bool pass2_callback(void *ctx, fr_cond_t *c)
        }
 
        /*
-        *      Maps have a paircompare fixup applied to them.
-        *      Others get ignored.
+        *      And tons of complicated checks.
         */
-       if (c->pass2_fixup == PASS2_FIXUP_NONE) {
-               if (c->type == COND_TYPE_MAP) {
-                       map = c->data.map;
-                       goto check_paircmp;
-               }
-
-               return true;
-       }
+       rad_assert(c->type == COND_TYPE_MAP);
 
        map = c->data.map;      /* shorter */
 
@@ -3212,7 +3204,6 @@ static bool pass2_callback(void *ctx, fr_cond_t *c)
                c->pass2_fixup = PASS2_FIXUP_NONE;
        }
 
-check_paircmp:
        /*
         *      Just in case someone adds a new fixup later.
         */