Module: kamailio Branch: 5.1 Commit: e1fd2ed4bfdc2704f1c89f1b64d852599c0b6726 URL: https://github.com/kamailio/kamailio/commit/e1fd2ed4bfdc2704f1c89f1b64d85259...
Author: yuri-r yuri.rootshtein@gmail.com Committer: Daniel-Constantin Mierla miconda@gmail.com Date: 2018-04-24T12:40:26+02:00
ims_charging: fixed check result code at mscc level (#1504)
* ims_charging: fixed check result code at mscc level
according to RFC 4006 the Result-Code in Multiple-Services-Credit-Control AVP is optional. resume_on_initial_ccr() stops proceeding CCA if no Result-Code in MSCC and goes to error1 even though the Result-Code on CCR itself is 2001 (SUCCESSFUL)
* ims_charging: fixed check result code at mscc level
according to RFC 4006 the Result-Code in Multiple-Services-Credit-Control AVP is optional. resume_on_initial_ccr() stops proceeding CCA if no Result-Code in MSCC and goes to error1 even though the Result-Code on CCR itself is 2001 (SUCCESSFUL)
(cherry picked from commit 352440d299f1b9f431cc05397f4ea6a7b5967c8a)
---
Modified: src/modules/ims_charging/ims_ro.c
---
Diff: https://github.com/kamailio/kamailio/commit/e1fd2ed4bfdc2704f1c89f1b64d85259... Patch: https://github.com/kamailio/kamailio/commit/e1fd2ed4bfdc2704f1c89f1b64d85259...
---
diff --git a/src/modules/ims_charging/ims_ro.c b/src/modules/ims_charging/ims_ro.c index 735f5b30be..497f65dd72 100644 --- a/src/modules/ims_charging/ims_ro.c +++ b/src/modules/ims_charging/ims_ro.c @@ -1361,7 +1361,7 @@ static void resume_on_initial_ccr(int is_timeout, void *param, AAAMessage *cca, create_cca_fui_avps(fui_action, redirecturi); /* check result code at mscc level */ - if (ro_cca_data->mscc->resultcode != 2001) { + if (ro_cca_data->mscc->resultcode && ro_cca_data->mscc->resultcode != 2001) { LM_DBG("CCA failure at MSCC level with resultcode [%d]\n", ro_cca_data->mscc->resultcode); error_code = RO_RETURN_FALSE; goto error1;
-
Daniel-Constantin Mierla