Module: kamailio Branch: master Commit: db31ddb5d5af3d6add5c4aacb9da6cfede04bf04 URL: https://github.com/kamailio/kamailio/commit/db31ddb5d5af3d6add5c4aacb9da6cfe...

Author: Daniel-Constantin Mierla miconda@gmail.com Committer: Daniel-Constantin Mierla miconda@gmail.com Date: 2023-06-13T09:45:13+02:00

cdp: remove unnecessary checks inside Send_ASR()

- updates to log messages

---

Modified: src/modules/cdp/authstatemachine.c

---

Diff: https://github.com/kamailio/kamailio/commit/db31ddb5d5af3d6add5c4aacb9da6cfe... Patch: https://github.com/kamailio/kamailio/commit/db31ddb5d5af3d6add5c4aacb9da6cfe...

---

diff --git a/src/modules/cdp/authstatemachine.c b/src/modules/cdp/authstatemachine.c index dcd56a4abf..65157fabdf 100644 --- a/src/modules/cdp/authstatemachine.c +++ b/src/modules/cdp/authstatemachine.c @@ -971,11 +971,12 @@ void Send_ASR(cdp_session_t *s, AAAMessage *msg) AAA_AVP *avp = 0; peer *p = 0; char x[4]; - LM_DBG("Send_ASR() : sending ASR\n"); + + LM_DBG("sending ASR\n"); asr = AAACreateRequest(s->application_id, IMS_ASR, Flag_Proxyable, s);

if(!asr) { - LM_ERR("Send_ASR(): error creating ASR!\n"); + LM_ERR("error creating ASR!\n"); return; }

@@ -992,14 +993,14 @@ void Send_ASR(cdp_session_t *s, AAAMessage *msg)

p = get_routing_peer(s, asr); if(!p) { - LM_ERR("unable to get routing peer in Send_ASR \n"); - if(asr) - AAAFreeMessage(&asr); //needed in frequency + LM_ERR("unable to get routing peer\n"); + AAAFreeMessage(&asr); //needed in frequency }

if(!peer_send_msg(p, asr)) { - if(asr) - AAAFreeMessage(&asr); //needed in frequency - } else + LM_ERR("unable to send message\n"); + AAAFreeMessage(&asr); //needed in frequency + } else { LM_DBG("success sending ASR\n"); + } }