From: Nick Porter <nick@portercomputing.co.uk>
Date: Fri, 11 Apr 2025 16:14:34 +0000 (+0100)
Subject: With m_size = -1, freeradius module doesn't clear up correctly
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=2ebc6af4bfe7b7e4e23fc732a1cce70c9e6bedfb;p=thirdparty%2Ffreeradius-server.git

With m_size = -1, freeradius module doesn't clear up correctly

Each module instance is getting it's own copy of the module, whereas
m_size = -1 implies it is global and causes a crash on exit when there
are multiple interpreters.
---

diff --git a/src/modules/rlm_python/rlm_python.c b/src/modules/rlm_python/rlm_python.c
index 2d86f0dbf76..35d3e45f650 100644
--- a/src/modules/rlm_python/rlm_python.c
+++ b/src/modules/rlm_python/rlm_python.c
@@ -871,7 +871,7 @@ static PyObject *python_module_init(void)
 		PyModuleDef_HEAD_INIT,
 		.m_name = "freeradius",
 		.m_doc = "freeRADIUS python module",
-		.m_size = -1,
+		.m_size = 0,
 		.m_methods = module_methods
 	};