From: Arran Cudbard-Bell <a.cudbardb@freeradius.org>
Date: Fri, 11 Oct 2019 18:32:34 +0000 (-0500)
Subject: Remove redundant check that also breaks subrequests used to call submodules
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=8798e58f2d581b35b8da07422c77702e1560177a;p=thirdparty%2Ffreeradius-server.git

Remove redundant check that also breaks subrequests used to call submodules
---

diff --git a/src/lib/unlang/subrequest.c b/src/lib/unlang/subrequest.c
index 673e1f6301b..09412951d8f 100644
--- a/src/lib/unlang/subrequest.c
+++ b/src/lib/unlang/subrequest.c
@@ -159,13 +159,6 @@ static unlang_action_t unlang_subrequest_start(REQUEST *request, rlm_rcode_t *pr
 	unlang_stack_frame_t		*frame = &stack->frame[stack->depth];
 	unlang_frame_state_subrequest_t	*state = talloc_get_type_abort(frame->state, unlang_frame_state_subrequest_t);
 	REQUEST				*child = state->child;
-	unlang_group_t			*g;
-
-	g = unlang_generic_to_group(frame->instruction);
-	if (!g->num_children) {
-		*presult = RLM_MODULE_NOOP;
-		return UNLANG_ACTION_CALCULATE_RESULT;
-	}
 
 	/*
 	 *	Restore state from the parent to the