From: Corey Farrell <git@cfware.com>
Date: Fri, 27 Oct 2017 18:41:15 +0000 (-0400)
Subject: res_pjsip_pubsub: Resolve potential crash in allocate_subscription.
X-Git-Tag: 13.19.0-rc1~183
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=987b16e8f9f9090f8ab3789bcafda7b27cd99412;p=thirdparty%2Fasterisk.git

res_pjsip_pubsub: Resolve potential crash in allocate_subscription.

When allocate_subscription fails to initialize fields of the new sub it
calls destroy_subscription.

Change-Id: I5b79c915ec216dc00c13c1e4172137864a4bec85
---

diff --git a/res/res_pjsip_pubsub.c b/res/res_pjsip_pubsub.c
index ae61a49003..e2c6f6ced5 100644
--- a/res/res_pjsip_pubsub.c
+++ b/res/res_pjsip_pubsub.c
@@ -1111,7 +1111,7 @@ static void remove_subscription(struct sip_subscription_tree *obj)
 static void destroy_subscription(struct ast_sip_subscription *sub)
 {
 	ast_debug(3, "Destroying SIP subscription from '%s->%s'\n",
-		sub->tree->endpoint ? ast_sorcery_object_get_id(sub->tree->endpoint) : "Unknown",
+		sub->tree && sub->tree->endpoint ? ast_sorcery_object_get_id(sub->tree->endpoint) : "Unknown",
 		sub->resource);
 
 	ast_free(sub->body_text);