reset request->module to NULL

when unlang calls the process function, it thinks that it's
calling a module, so it sets request->module = name of process
function.

This means that all debug messages (such as "received packet")
are printed out, prefixed with the process name.  This makes
debug messages harder to read.

Since the first thing the process function does is call an unlang
section, we can safely set request->module=NULL, which omits the
"radius - " prefix from the packet we're printing

diff --git a/src/process/arp/base.c b/src/process/arp/base.c
index 87fc9fca3f4c1516700b4a2b068539d0719c5365..1569f9c349dd36f76f15d3e4622ea6db9495eee7 100644 (file)
--- a/src/process/arp/base.c
+++ b/src/process/arp/base.c
@@ -188,6 +188,7 @@ static unlang_action_t mod_process(rlm_rcode_t *p_result, module_ctx_t const *mc
        fr_assert(PROCESS_PACKET_CODE_VALID(request->packet->code));
 
        request->component = "arp";
+       request->module = NULL;
        fr_assert(request->dict == dict_arp);
 
        UPDATE_STATE(packet);

diff --git a/src/process/dhcpv4/base.c b/src/process/dhcpv4/base.c
index a2a3cb3f358d983eb4e4a93c8e05dfc1f0b6d5fd..e07ccf19ae38287f0871a188128ab6344ac1dd25 100644 (file)
--- a/src/process/dhcpv4/base.c
+++ b/src/process/dhcpv4/base.c
@@ -316,6 +316,7 @@ static unlang_action_t mod_process(rlm_rcode_t *p_result, module_ctx_t const *mc
        fr_assert(PROCESS_PACKET_CODE_VALID(request->packet->code));
 
        request->component = "dhcpv4";
+       request->module = NULL;
        fr_assert(request->dict == dict_dhcpv4);
 
        UPDATE_STATE(packet);

diff --git a/src/process/dhcpv6/base.c b/src/process/dhcpv6/base.c
index 134ca5159250f00ea2b99d184eea0a4f87a546f2..6d8c6f1528e435526d0d83e03a876319ad3894e1 100644 (file)
--- a/src/process/dhcpv6/base.c
+++ b/src/process/dhcpv6/base.c
@@ -482,6 +482,7 @@ static unlang_action_t mod_process(rlm_rcode_t *p_result, module_ctx_t const *mc
        fr_assert(PROCESS_PACKET_CODE_VALID(request->packet->code));
 
        request->component = "dhcpv6";
+       request->module = NULL;
        fr_assert(request->dict == dict_dhcpv6);
 
        UPDATE_STATE(packet);

diff --git a/src/process/radius/base.c b/src/process/radius/base.c
index 00f9a7fbb6dec6069a12ada84e30a57db21e3afe..4e0de31f3bfa4e99c40647aeef4ebe99443ae611 100644 (file)
--- a/src/process/radius/base.c
+++ b/src/process/radius/base.c
@@ -783,6 +783,7 @@ static unlang_action_t mod_process(rlm_rcode_t *p_result, module_ctx_t const *mc
        fr_assert(FR_RADIUS_PACKET_CODE_VALID(request->packet->code));
 
        request->component = "radius";
+       request->module = NULL;
        fr_assert(request->dict == dict_radius);
 
        UPDATE_STATE(packet);

diff --git a/src/process/tacacs/base.c b/src/process/tacacs/base.c
index c526e3133c908c6fd334589a2a6e468fdfdede18..0220876297c1ba9dfea9e58f1ac7c8f922979f71 100644 (file)
--- a/src/process/tacacs/base.c
+++ b/src/process/tacacs/base.c
@@ -569,6 +569,7 @@ static unlang_action_t mod_process(rlm_rcode_t *p_result, module_ctx_t const *mc
        fr_assert(PROCESS_PACKET_CODE_VALID(request->packet->code));
 
        request->component = "tacacs";
+       request->module = NULL;
        fr_assert(request->dict == dict_tacacs);
 
        UPDATE_STATE(packet);

diff --git a/src/process/vmps/base.c b/src/process/vmps/base.c
index 4adb67ed6b3762664fdf87770dc1ac1b2220f979..e6469d463408d0c524d295852d05f21a4d142470 100644 (file)
--- a/src/process/vmps/base.c
+++ b/src/process/vmps/base.c
@@ -212,6 +212,7 @@ static unlang_action_t mod_process(rlm_rcode_t *p_result, module_ctx_t const *mc
        fr_assert(PROCESS_PACKET_CODE_VALID(request->packet->code));
 
        request->component = "vmps";
+       request->module = NULL;
        fr_assert(request->dict == dict_vmps);
 
        UPDATE_STATE(packet);