Changeset 535:72c511043f05 in freeDiameter

Timestamp:

Sep 9, 2010, 2:17:54 PM (14 years ago)

Author:

Sebastien Decugis <sdecugis@nict.go.jp>

Branch:

default

Phase:

public

Message:

A few bug fixes on the RADIUS/Diameter gateway.

Location:

extensions/app_radgw

Files:

• rgw_worker.c (modified) (2 diffs)
• rgwx_acct.c (modified) (4 diffs)

extensions/app_radgw/rgw_worker.c

@@ -285 +285 @@
         
         
-        if (!keepsession) {
-                /* Destroy remaining session data (stateless gateway) */
-                CHECK_FCT_DO( fd_sess_destroy(&pa->sess),  );
-        }
-        
         /* Now try and send the RADIUS answer */
         if (rad_ans) {
@@ -302 +297 @@
         }
         
+        if (!keepsession) {
+                /* Destroy remaining session data (stateless gateway) */
+                CHECK_FCT_DO( fd_sess_destroy(&pa->sess),  );
+        }
+        
         /* Clear the RADIUS request */
         if (pa->rad) {

extensions/app_radgw/rgwx_acct.c

@@ -490 +490 @@
                 CHECK_FCT( fd_sess_fromsid ( (char *)/* cast should be removed later */si, si_len, session, NULL) );
                 
-                TRACE_DEBUG(FULL, "[auth.rgwx] Translating new accounting message for session '%.*s'...", si_len, si);
+                TRACE_DEBUG(FULL, "[acct.rgwx] Translating new accounting message for session '%.*s'...", si_len, si);
                 
                 /* Add the Session-Id AVP as first AVP */
@@ -1262 +1262 @@
                 
                 default:
-                        fd_log_debug("[auth.rgwx] Received Diameter answer with error code '%d' from server '%.*s', session %.*s, not translating into Accounting-Response\n",
+                        fd_log_debug("[acct.rgwx] Received Diameter answer with error code '%d' from server '%.*s', session %.*s, not translating into Accounting-Response\n",
                                         ahdr->avp_value->u32, 
                                         oh->avp_value->os.len, oh->avp_value->os.data,
@@ -1269 +1269 @@
                         if (avp) {
                                 CHECK_FCT( fd_msg_avp_hdr ( avp, &ahdr ) );
-                                fd_log_debug("[auth.rgwx]   Error-Message content: '%.*s'\n",
+                                fd_log_debug("[acct.rgwx]   Error-Message content: '%.*s'\n",
                                                 ahdr->avp_value->os.len, ahdr->avp_value->os.data);
                         }
@@ -1275 +1275 @@
                         if (avp) {
                                 CHECK_FCT( fd_msg_avp_hdr ( avp, &ahdr ) );
-                                fd_log_debug("[auth.rgwx]   Error-Reporting-Host: '%.*s'\n",
+                                fd_log_debug("[acct.rgwx]   Error-Reporting-Host: '%.*s'\n",
                                                 ahdr->avp_value->os.len, ahdr->avp_value->os.data);
                         }
                         CHECK_FCT( fd_msg_search_avp (*diam_ans, cs->dict.Failed_AVP, &avp) );
                         if (avp) {
-                                fd_log_debug("[auth.rgwx]   Failed-AVP was included in the message.\n");
+                                fd_log_debug("[acct.rgwx]   Failed-AVP was included in the message.\n");
                                 /* Dump its content ? */
                         }