[Asterisk-code-review] res pjsip pubsub.c: Add missing error return in load module(). (asterisk[13])

Richard Mudgett asteriskteam at digium.com
Thu Sep 10 13:18:42 CDT 2015


Richard Mudgett has uploaded a new change for review.

  https://gerrit.asterisk.org/1239

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(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/39/1239/1

diff --git a/res/res_pjsip_pubsub.c b/res/res_pjsip_pubsub.c
index d409a8e..088f807 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/1239
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I15debd0f717f16ee2f78e7f56151c3b3b97b72fc
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Owner: Richard Mudgett <rmudgett at digium.com>



More information about the asterisk-code-review mailing list