From: Alan T. DeKok <aland@freeradius.org>
Date: Wed, 18 Jan 2023 01:00:36 +0000 (-0500)
Subject: use *p_result for "current interpreter result"
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=f027fc7a7641548637417de14aa86656209a48fb;p=thirdparty%2Ffreeradius-server.git

use *p_result for "current interpreter result"

request->rcode should really be a cached version, and should
maybe not even exist at all.
---

diff --git a/src/process/radius/base.c b/src/process/radius/base.c
index 5e2383f15d0..9b5703e1f73 100644
--- a/src/process/radius/base.c
+++ b/src/process/radius/base.c
@@ -377,7 +377,7 @@ RESUME(auth_type);
 
 RESUME(access_request)
 {
-	rlm_rcode_t			rcode = request->rcode;
+	rlm_rcode_t			rcode = *p_result;
 	fr_pair_t			*vp;
 	CONF_SECTION			*cs;
 	fr_dict_enum_value_t const		*dv;
@@ -456,7 +456,7 @@ RESUME(auth_type)
 		[RLM_MODULE_DISALLOW] = FR_RADIUS_CODE_ACCESS_REJECT,
 	};
 
-	rlm_rcode_t			rcode = request->rcode;
+	rlm_rcode_t			rcode = *p_result;
 	fr_pair_t			*vp;
 	fr_process_state_t const	*state;
 
@@ -625,7 +625,7 @@ RESUME(acct_type)
 		[RLM_MODULE_DISALLOW] = FR_RADIUS_CODE_DO_NOT_RESPOND,
 	};
 
-	rlm_rcode_t			rcode = request->rcode;
+	rlm_rcode_t			rcode = *p_result;
 	fr_process_state_t const	*state;
 
 	PROCESS_TRACE;
@@ -655,7 +655,7 @@ RESUME(acct_type)
 
 RESUME(accounting_request)
 {
-	rlm_rcode_t			rcode = request->rcode;
+	rlm_rcode_t			rcode = *p_result;
 	fr_pair_t			*vp;
 	CONF_SECTION			*cs;
 	fr_dict_enum_value_t const		*dv;
diff --git a/src/process/ttls/base.c b/src/process/ttls/base.c
index febed435d45..b0498c83329 100644
--- a/src/process/ttls/base.c
+++ b/src/process/ttls/base.c
@@ -359,7 +359,7 @@ RESUME(auth_type);
 
 RESUME(access_request)
 {
-	rlm_rcode_t			rcode = request->rcode;
+	rlm_rcode_t			rcode = *p_result;
 	fr_pair_t			*vp;
 	CONF_SECTION			*cs;
 	fr_dict_enum_value_t const	*dv;
@@ -438,7 +438,7 @@ RESUME(auth_type)
 		[RLM_MODULE_DISALLOW] = FR_RADIUS_CODE_ACCESS_REJECT,
 	};
 
-	rlm_rcode_t			rcode = request->rcode;
+	rlm_rcode_t			rcode = *p_result;
 	fr_pair_t			*vp;
 	fr_process_state_t const	*state;