Changeset 278:50bfb29bf036 in freeDiameter for extensions/app_radgw/rgw_worker.c

Timestamp:

Apr 23, 2010, 5:22:09 PM (14 years ago)

Author:

Sebastien Decugis <sdecugis@nict.go.jp>

Branch:

default

Phase:

public

Message:

Try better handling several pending requests for the same session (not very clean yet)

File:

• extensions/app_radgw/rgw_worker.c (modified) (4 diffs)

extensions/app_radgw/rgw_worker.c

@@ -147 +147 @@
                         {
                                 /* An error occurred, discard message */
+                                if (diam_msg) {
+                                        CHECK_FCT_DO( fd_msg_free(diam_msg), );
+                                        diam_msg = NULL;
+                                }
+                                if (session) {
+                                        CHECK_FCT_DO( fd_sess_destroy(&session), );
+                                }
                                 rgw_msg_free(&msg);
                                 rgw_clients_dispose(&cli);
@@ -153 +160 @@
                 if (msg == NULL) {
                         rgw_clients_dispose(&cli);
+                        if (diam_msg) {
+                                CHECK_FCT_DO( fd_msg_free(diam_msg), );
+                                diam_msg = NULL;
+                        }
+                        if (session) {
+                                CHECK_FCT_DO( fd_sess_destroy(&session), );
+                        }
                         continue; /* the message was handled already */
                 }
@@ -176 +190 @@
                 }
                 
-                /* Check the session is correct */
+                /* Check the session is correct (for debug) */
                 ASSERT(session != NULL);
                 
@@ -298 +312 @@
         }
         
+        /* Clear the RADIUS request */
+        if (pa->rad) {
+                rgw_msg_free(&pa->rad);
+        }
+        
         /* Release reference on the client */
         rgw_clients_dispose(&pa->cli);