Activate persistent subscriptions when they are recreated.

Prior to this change, recreating persistent subscriptions would
create the subscription but would not activate it. This led to subscriptions
being listed in the "NULL" state by diagnostics and not sending NOTIFYs
when expected.

Review: https://reviewboard.asterisk.org/r/4261



git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/13@429571 65c4cc65-6c06-0410-ace0-fbb531ad65f3
changes/43/43/1
Mark Michelson 11 years ago
parent 2b8c441096
commit b85f79c0c1

@ -1289,6 +1289,9 @@ static struct sip_subscription_tree *create_subscription_tree(const struct ast_s
return sub_tree; return sub_tree;
} }
static int generate_initial_notify(struct ast_sip_subscription *sub);
static int send_notify(struct sip_subscription_tree *sub_tree, unsigned int force_full_state);
/*! \brief Callback function to perform the actual recreation of a subscription */ /*! \brief Callback function to perform the actual recreation of a subscription */
static int subscription_persistence_recreate(void *obj, void *arg, int flags) static int subscription_persistence_recreate(void *obj, void *arg, int flags)
{ {
@ -1377,6 +1380,10 @@ static int subscription_persistence_recreate(void *obj, void *arg, int flags)
} }
sub_tree->persistence = ao2_bump(persistence); sub_tree->persistence = ao2_bump(persistence);
subscription_persistence_update(sub_tree, &rdata); subscription_persistence_update(sub_tree, &rdata);
if (generate_initial_notify(sub_tree->root)) {
pjsip_evsub_terminate(sub_tree->evsub, PJ_TRUE);
}
send_notify(sub_tree, 1);
} else { } else {
ast_sorcery_delete(ast_sip_get_sorcery(), persistence); ast_sorcery_delete(ast_sip_get_sorcery(), persistence);
} }

Loading…
Cancel
Save