kenmirrors/asterisk
1
0
Fork 0
mirror of https://github.com/asterisk/asterisk.git synced 2025-09-30 02:26:23 +00:00
Code Issues Packages Projects Releases Wiki Activity

pjsip_distributor.c: Use correct rdata info access method (Part 2).

Browse Source 
The pjproject doxygen for rdata->msg_info.info says to call
pjsip_rx_data_get_info() instead of accessing the struct member directly.
You need to call the function mostly because the function will generate
the struct member value if it is not already setup.

Change-Id: I4d519385a577f3e9d9193a88125e493cf17fa799
This commit is contained in:
Richard Mudgett
2016-05-31 13:02:15 -05:00
parent ad31e5bb1c
commit aec7916595
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
res/res_pjsip/pjsip_distributor.c
View File

@@ -297,13 +297,13 @@ static pj_bool_t distributor(pjsip_rx_data *rdata)
if (dlg) { if (dlg) {
ast_debug(3, "Searching for serializer on dialog %s for %s\n", ast_debug(3, "Searching for serializer on dialog %s for %s\n",
dlg->obj_name, rdata->msg_info.info); dlg->obj_name, pjsip_rx_data_get_info(rdata));
dist = pjsip_dlg_get_mod_data(dlg, distributor_mod.id); dist = pjsip_dlg_get_mod_data(dlg, distributor_mod.id);
if (dist) { if (dist) {
serializer = ao2_bump(dist->serializer); serializer = ao2_bump(dist->serializer);
if (serializer) { if (serializer) {
ast_debug(3, "Found serializer %s on dialog %s\n", ast_debug(3, "Found serializer %s on dialog %s\n",
ast_taskprocessor_name(serializer), dlg->obj_name); ast_taskprocessor_name(serializer), dlg->obj_name);
} }
} }
pjsip_dlg_dec_lock(dlg); pjsip_dlg_dec_lock(dlg);
@@ -313,7 +313,7 @@ static pj_bool_t distributor(pjsip_rx_data *rdata)
/* We have a serializer so we know where to send the message. */ /* We have a serializer so we know where to send the message. */
} else if (rdata->msg_info.msg->type == PJSIP_RESPONSE_MSG) { } else if (rdata->msg_info.msg->type == PJSIP_RESPONSE_MSG) {
ast_debug(3, "No dialog serializer for response %s. Using request transaction as basis\n", ast_debug(3, "No dialog serializer for response %s. Using request transaction as basis\n",
rdata->msg_info.info); pjsip_rx_data_get_info(rdata));
serializer = find_request_serializer(rdata); serializer = find_request_serializer(rdata);
} else if (!pjsip_method_cmp(&rdata->msg_info.msg->line.req.method, &pjsip_cancel_method) } else if (!pjsip_method_cmp(&rdata->msg_info.msg->line.req.method, &pjsip_cancel_method)
|| !pjsip_method_cmp(&rdata->msg_info.msg->line.req.method, &pjsip_bye_method)) { || !pjsip_method_cmp(&rdata->msg_info.msg->line.req.method, &pjsip_bye_method)) {