[Asterisk-code-review] pjsip distributor.c: Use correct rdata info access method. (asterisk[13])

Anonymous Coward asteriskteam at digium.com
Thu May 26 17:17:41 CDT 2016


Anonymous Coward #1000019 has submitted this change and it was merged.

Change subject: pjsip_distributor.c: Use correct rdata info access method.
......................................................................


pjsip_distributor.c: Use correct rdata info access method.

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: Iafe8b01242b7deb0ebfdc36685e21374a43936d2
---
M res/res_pjsip/pjsip_distributor.c
1 file changed, 2 insertions(+), 1 deletion(-)

Approvals:
  George Joseph: Looks good to me, but someone else must approve
  Anonymous Coward #1000019: Verified
  Joshua Colp: Looks good to me, approved



diff --git a/res/res_pjsip/pjsip_distributor.c b/res/res_pjsip/pjsip_distributor.c
index 2ab954e..0d3df06 100644
--- a/res/res_pjsip/pjsip_distributor.c
+++ b/res/res_pjsip/pjsip_distributor.c
@@ -257,7 +257,8 @@
 
 	tsx = pjsip_tsx_layer_find_tsx(&tsx_key, PJ_TRUE);
 	if (!tsx) {
-		ast_debug(3, "Could not find matching transaction for %s\n", rdata->msg_info.info);
+		ast_debug(3, "Could not find matching transaction for %s\n",
+			pjsip_rx_data_get_info(rdata));
 		return NULL;
 	}
 

-- 
To view, visit https://gerrit.asterisk.org/2908
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Iafe8b01242b7deb0ebfdc36685e21374a43936d2
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Owner: Richard Mudgett <rmudgett at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>



More information about the asterisk-code-review mailing list