Module: kamailio
Branch: master
Commit: 1b291315b43da1148cee5f3821ddbe49b0c86b7f
URL: 
https://github.com/kamailio/kamailio/commit/1b291315b43da1148cee5f3821ddbe49b0c86b7f

Author: Stefan Mititelu <stefan.mitit...@govoip.ro>
Committer: Daniel-Constantin Mierla <mico...@gmail.com>
Date: 2023-07-04T18:41:23+02:00

ims_qos: check and log for NULL sessionId

---

Modified: src/modules/ims_qos/rx_aar.c

---

Diff:  
https://github.com/kamailio/kamailio/commit/1b291315b43da1148cee5f3821ddbe49b0c86b7f.diff
Patch: 
https://github.com/kamailio/kamailio/commit/1b291315b43da1148cee5f3821ddbe49b0c86b7f.patch

---

diff --git a/src/modules/ims_qos/rx_aar.c b/src/modules/ims_qos/rx_aar.c
index e5e0bddba23..8c0fc4e937d 100644
--- a/src/modules/ims_qos/rx_aar.c
+++ b/src/modules/ims_qos/rx_aar.c
@@ -152,6 +152,11 @@ void async_aar_callback(
                                cdp_result);
                counter_inc(ims_qos_cnts_h.successful_media_aars);
 
+               if(!aaa->sessionId) {
+                       LM_ERR("async_aar_callback: NULL AAA sessionId from 
PCRF!\n");
+                       goto error;
+               }
+
                LM_DBG("Auth session ID [%.*s]", aaa->sessionId->data.len,
                                aaa->sessionId->data.s);
 
@@ -309,6 +314,12 @@ void async_aar_reg_callback(
                        create_return_code(result);
                        goto done;
                }
+
+               if(!aaa->sessionId) {
+                       LM_ERR("async_aar_reg_callback: NULL AAA sessionId from 
PCRF!\n");
+                       goto error;
+               }
+
                //need to set Rx auth data to say this session has been 
successfully opened
                //This is used elsewhere to prevent acting on termination 
events when the session has not been opened
                //getting auth session

_______________________________________________
Kamailio (SER) - Development Mailing List
To unsubscribe send an email to sr-dev-le...@lists.kamailio.org

Reply via email to