[sr-dev] git:5.1:e1fd2ed4: ims_charging: fixed check result code at mscc level (#1504)

Module: kamailio Branch: 5.1 Commit: e1fd2ed4bfdc2704f1c89f1b64d852599c0b6726 URL: https://github.com/kamailio/kamailio/commit/e1fd2ed4bfdc2704f1c89f1b64d8525… Author: yuri-r <yuri.rootshtein(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)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/e1fd2ed4bfdc2704f1c89f1b64d8525… Patch: https://github.com/kamailio/kamailio/commit/e1fd2ed4bfdc2704f1c89f1b64d8525… --- 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;