diff --git a/res/res_pjsip_pubsub.c b/res/res_pjsip_pubsub.c index 76c9b33675d93aa696a4373ad850ad57e783c7cc..57c85a4c73df6496ccaccd779a8b78356fcc219d 100644 --- a/res/res_pjsip_pubsub.c +++ b/res/res_pjsip_pubsub.c @@ -561,7 +561,7 @@ static void subscription_persistence_update(struct sip_subscription_tree *sub_tr expires = expires_hdr ? expires_hdr->ivalue : DEFAULT_PUBLISH_EXPIRES; sub_tree->persistence->expires = ast_tvadd(ast_tvnow(), ast_samp2tv(expires, 1)); - ast_copy_string(sub_tree->persistence->packet, rdata->pkt_info.packet, + pjsip_msg_print(rdata->msg_info.msg, sub_tree->persistence->packet, sizeof(sub_tree->persistence->packet)); ast_copy_string(sub_tree->persistence->src_name, rdata->pkt_info.src_name, sizeof(sub_tree->persistence->src_name)); @@ -1338,6 +1338,13 @@ static int subscription_persistence_recreate(void *obj, void *arg, int flags) return 0; } + if (rdata.msg_info.msg->type != PJSIP_REQUEST_MSG) { + ast_log(LOG_NOTICE, "Endpoint %s persisted a SIP response instead of a subscribe request. Unable to reload subscription.\n", + ast_sorcery_object_get_id(endpoint)); + ast_sorcery_delete(ast_sip_get_sorcery(), persistence); + return 0; + } + request_uri = pjsip_uri_get_uri(rdata.msg_info.msg->line.req.uri); resource_size = pj_strlen(&request_uri->user) + 1; resource = alloca(resource_size);