[Asterisk-code-review] res pjsip session.c: Fix off nominal crash potential in debu... (asterisk[13])

Matt Jordan asteriskteam at digium.com
Thu Jul 16 20:33:30 CDT 2015


Matt Jordan has submitted this change and it was merged.

Change subject: res_pjsip_session.c: Fix off nominal crash potential in debug message.
......................................................................


res_pjsip_session.c: Fix off nominal crash potential in debug message.

Change-Id: I09928297927ee85f7655289acee3a586816466bc
---
M res/res_pjsip_session.c
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Mark Michelson: Looks good to me, but someone else must approve
  Anonymous Coward #1000019: Verified
  Matt Jordan: Looks good to me, approved



diff --git a/res/res_pjsip_session.c b/res/res_pjsip_session.c
index 2c2ea61..e6ed645 100644
--- a/res/res_pjsip_session.c
+++ b/res/res_pjsip_session.c
@@ -1161,7 +1161,7 @@
 	struct ast_sip_session_delayed_request *delay;
 
 	ast_debug(3, "Destroying SIP session with endpoint %s\n",
-			ast_sorcery_object_get_id(session->endpoint));
+		session->endpoint ? ast_sorcery_object_get_id(session->endpoint) : "<none>");
 
 	while ((supplement = AST_LIST_REMOVE_HEAD(&session->supplements, next))) {
 		if (supplement->session_destroy) {

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I09928297927ee85f7655289acee3a586816466bc
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Owner: Richard Mudgett <rmudgett at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Mark Michelson <mmichelson at digium.com>
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>



More information about the asterisk-code-review mailing list