From: Nick Porter <nick@portercomputing.co.uk>
Date: Mon, 22 Jan 2024 12:31:00 +0000 (+0000)
Subject: sqlippool_queries don't use SQL-User-Name attribute, so don't create it
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=6b1271a04e0a320e4f5ed296b06bcce0bcc10318;p=thirdparty%2Ffreeradius-server.git

sqlippool_queries don't use SQL-User-Name attribute, so don't create it
---

diff --git a/src/modules/rlm_sqlippool/rlm_sqlippool.c b/src/modules/rlm_sqlippool/rlm_sqlippool.c
index 18e09020c63..2e17de9b199 100644
--- a/src/modules/rlm_sqlippool/rlm_sqlippool.c
+++ b/src/modules/rlm_sqlippool/rlm_sqlippool.c
@@ -372,10 +372,6 @@ static unlang_action_t CC_HINT(nonnull) mod_alloc(rlm_rcode_t *p_result, module_
 
 	RESERVE_CONNECTION(handle, inst->sql->pool, request);
 
-	if (inst->sql->sql_set_user(inst->sql, request, NULL) < 0) {
-		RETURN_MODULE_FAIL;
-	}
-
 	DO_PART(alloc_begin);
 
 	/*
@@ -519,10 +515,6 @@ static unlang_action_t CC_HINT(nonnull) mod_update(rlm_rcode_t *p_result, module
 
 	RESERVE_CONNECTION(handle, inst->sql->pool, request);
 
-	if (inst->sql->sql_set_user(inst->sql, request, NULL) < 0) {
-		RETURN_MODULE_FAIL;
-	}
-
 	DO_PART(update_begin);
 
 	/*
@@ -567,10 +559,6 @@ static unlang_action_t CC_HINT(nonnull) mod_release(rlm_rcode_t *p_result, modul
 
 	RESERVE_CONNECTION(handle, inst->sql->pool, request);
 
-	if (inst->sql->sql_set_user(inst->sql, request, NULL) < 0) {
-		RETURN_MODULE_FAIL;
-	}
-
 	DO_PART(release_begin);
 	DO_PART(release_clear);
 	DO_PART(release_commit);
@@ -593,10 +581,6 @@ static unlang_action_t CC_HINT(nonnull) mod_bulk_release(rlm_rcode_t *p_result,
 
 	RESERVE_CONNECTION(handle, inst->sql->pool, request);
 
-	if (inst->sql->sql_set_user(inst->sql, request, NULL) < 0) {
-		RETURN_MODULE_FAIL;
-	}
-
 	DO_PART(bulk_release_begin);
 	DO_PART(bulk_release_clear);
 	DO_PART(bulk_release_commit);
@@ -620,10 +604,6 @@ static unlang_action_t CC_HINT(nonnull) mod_mark(rlm_rcode_t *p_result, module_c
 
 	RESERVE_CONNECTION(handle, inst->sql->pool, request);
 
-	if (inst->sql->sql_set_user(inst->sql, request, NULL) < 0) {
-		RETURN_MODULE_FAIL;
-	}
-
 	DO_PART(mark_begin);
 	DO_PART(mark_update);
 	DO_PART(mark_commit);