[asterisk-commits] mmichelson: trunk r417663 - /trunk/res/res_pjsip_pubsub.c

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Mon Jun 30 13:39:59 CDT 2014


Author: mmichelson
Date: Mon Jun 30 13:39:56 2014
New Revision: 417663

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=417663
Log:
Reverse logic during subscription persistence recreation.

In the abstraction effort, this bit of logic got messed up. We
want to recreate the persistence if things go well, not if things
fail.


Modified:
    trunk/res/res_pjsip_pubsub.c

Modified: trunk/res/res_pjsip_pubsub.c
URL: http://svnview.digium.com/svn/asterisk/trunk/res/res_pjsip_pubsub.c?view=diff&rev=417663&r1=417662&r2=417663
==============================================================================
--- trunk/res/res_pjsip_pubsub.c (original)
+++ trunk/res/res_pjsip_pubsub.c Mon Jun 30 13:39:56 2014
@@ -566,7 +566,7 @@
 			pubsub_module.id, MOD_DATA_PERSISTENCE, persistence);
 
 	resp = handler->notifier->new_subscribe(endpoint, resource);
-	if (!PJSIP_IS_STATUS_IN_CLASS(resp, 200)) {
+	if (PJSIP_IS_STATUS_IN_CLASS(resp, 200)) {
 		sub = notifier_create_subscription(handler, endpoint, &rdata, resource, generator);
 		sub->persistence = ao2_bump(persistence);
 		subscription_persistence_update(sub, &rdata);




More information about the asterisk-commits mailing list