check for debug BEFORE calling debug function

map_list_mod_debug() does a lot of unconditional allocations.
So don't call it unless we know it's debugging

diff --git a/src/lib/server/map.c b/src/lib/server/map.c
index 2ba8c512216455d5b7ef4b63ae4e4dd49e1a105c..795511e84f056da7b29898e0a230c86de1d31b91 100644 (file)
--- a/src/lib/server/map.c
+++ b/src/lib/server/map.c
@@ -1774,9 +1774,16 @@ int map_list_mod_apply(REQUEST *request, vp_list_mod_t const *vlm)
                rad_assert(((mod->op == T_OP_CMP_FALSE) && tmpl_is_null(mod->rhs)) ||
                           tmpl_is_data(mod->rhs));
 
-               for (vb = &mod->rhs->tmpl_value;
-                    vb;
-                    vb = vb->next) map_list_mod_debug(request, map, mod, vb->type != FR_TYPE_INVALID ? vb : NULL);
+               /*
+                *      map_list_mod_debug()
+                */
+               if (RDEBUG_ENABLED2) {
+                       for (vb = &mod->rhs->tmpl_value;
+                            vb;
+                            vb = vb->next) {
+                               map_list_mod_debug(request, map, mod, vb->type != FR_TYPE_INVALID ? vb : NULL);
+                       }
+               }
        }
        mod = vlm->mod; /* Reset */