[asterisk-commits] res pjsip pubsub.c: Add missing error return in load module(). (asterisk[master])

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Thu Sep 10 15:11:49 CDT 2015


Mark Michelson has submitted this change and it was merged.

Change subject: res_pjsip_pubsub.c: Add missing error return in load_module().
......................................................................


res_pjsip_pubsub.c: Add missing error return in load_module().

Change-Id: I15debd0f717f16ee2f78e7f56151c3b3b97b72fc
---
M res/res_pjsip_pubsub.c
1 file changed, 1 insertion(+), 0 deletions(-)

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



diff --git a/res/res_pjsip_pubsub.c b/res/res_pjsip_pubsub.c
index 6c3d101..c46bbfc 100644
--- a/res/res_pjsip_pubsub.c
+++ b/res/res_pjsip_pubsub.c
@@ -4270,6 +4270,7 @@
 	if (apply_list_configuration(sorcery)) {
 		ast_sip_unregister_service(&pubsub_module);
 		ast_sched_context_destroy(sched);
+		return AST_MODULE_LOAD_FAILURE;
 	}
 
 	ast_sorcery_apply_default(sorcery, "inbound-publication", "config", "pjsip.conf,criteria=type=inbound-publication");

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I15debd0f717f16ee2f78e7f56151c3b3b97b72fc
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Richard Mudgett <rmudgett at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Jonathan Rose <jrose at digium.com>
Gerrit-Reviewer: Mark Michelson <mmichelson at digium.com>



More information about the asterisk-commits mailing list