Skip to content

Commit

Permalink
ims_qos: check and log for NULL sessionId
Browse files Browse the repository at this point in the history
  • Loading branch information
smititelu authored and miconda committed Jul 4, 2023
1 parent b5a3768 commit 1b29131
Showing 1 changed file with 11 additions and 0 deletions.
11 changes: 11 additions & 0 deletions src/modules/ims_qos/rx_aar.c
Original file line number Diff line number Diff line change
Expand Up @@ -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);

Expand Down Expand Up @@ -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
Expand Down

0 comments on commit 1b29131

Please sign in to comment.