From: Arran Cudbard-Bell <a.cudbardb@freeradius.org>
Date: Fri, 19 Jan 2018 23:33:44 +0000 (-0700)
Subject: Don't mess with request->module on resumption
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=a4cb5618cefb44d3b29a1c8775eeefc752c2ef22;p=thirdparty%2Ffreeradius-server.git

Don't mess with request->module on resumption

We need a better way of handling this...
---

diff --git a/src/main/unlang_interpret.c b/src/main/unlang_interpret.c
index 475ae8ef387..da535866c92 100644
--- a/src/main/unlang_interpret.c
+++ b/src/main/unlang_interpret.c
@@ -1319,8 +1319,6 @@ static unlang_action_t unlang_resume(REQUEST *request, rlm_rcode_t *presult, int
 		return UNLANG_ACTION_CALCULATE_RESULT;
 	}
 
-	request->module = mr->self.debug_name;
-
 	/*
 	 *	Run the resume callback associated with
 	 *	the original frame which was used to
@@ -1328,8 +1326,6 @@ static unlang_action_t unlang_resume(REQUEST *request, rlm_rcode_t *presult, int
 	 */
 	action = unlang_ops[mr->parent->type].resume(request, presult, mr->resume_ctx);
 
-	request->module = NULL;
-
 	/*
 	 *	Leave mr alone, it will be freed when the request is done.
 	 */