From: Nick Porter <nick@portercomputing.co.uk>
Date: Tue, 5 Nov 2024 08:52:48 +0000 (+0000)
Subject: Don't store session state for child requests
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=c00af2ea297e48a05b335ca2c4d7b13687f05cc6;p=thirdparty%2Ffreeradius-server.git

Don't store session state for child requests

This allows the parent request to use fr_state_store_in_parent to store
the child's session state.
---

diff --git a/src/process/radius/base.c b/src/process/radius/base.c
index 8755c7c117e..62f561c12ca 100644
--- a/src/process/radius/base.c
+++ b/src/process/radius/base.c
@@ -592,11 +592,12 @@ RESUME(access_challenge)
 	PROCESS_TRACE;
 
 	/*
-	 *	Cache the state context.
+	 *	Cache the state context, unless this is a subrequest.
+	 *	Subrequest state context will be handled by the caller.
 	 *
 	 *	If this fails, don't respond to the request.
 	 */
-	if (fr_request_to_state(inst->auth.state_tree, request) < 0) {
+	if (!request->parent && fr_request_to_state(inst->auth.state_tree, request) < 0) {
 		return CALL_SEND_TYPE(FR_RADIUS_CODE_DO_NOT_RESPOND);
 	}