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

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


Richard Mudgett has uploaded a new change for review.

  https://gerrit.asterisk.org/1247

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/47/1247/1

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: newchange
Gerrit-Change-Id: I15debd0f717f16ee2f78e7f56151c3b3b97b72fc
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Richard Mudgett <rmudgett at digium.com>



More information about the asterisk-code-review mailing list