summaryrefslogtreecommitdiff
path: root/main/message.c
diff options
context:
space:
mode:
authorCorey Farrell <git@cfware.com>2017-11-06 16:10:56 -0500
committerCorey Farrell <git@cfware.com>2017-11-06 16:13:49 -0500
commit66d7c8495ab9bc9c9f67822f941684f79607b1f8 (patch)
treec4667ebe5fdd6fb280c0accfffc9fe81bc9004d7 /main/message.c
parent457abd06b4b6e89f80e106ef710bf2cf8932b7f8 (diff)
Messaging: Report error on failure to register tech or handler.
Message tech and handler registrations use a vector which could fail to expand. If it does log and error and return error. Change-Id: I593a8de81a07fb0452e9b0efd5d4018b77bca6f4
Diffstat (limited to 'main/message.c')
-rw-r--r--main/message.c14
1 files changed, 12 insertions, 2 deletions
diff --git a/main/message.c b/main/message.c
index fcdf705fe..ac7965ea7 100644
--- a/main/message.c
+++ b/main/message.c
@@ -1362,7 +1362,12 @@ int ast_msg_tech_register(const struct ast_msg_tech *tech)
return -1;
}
- AST_VECTOR_APPEND(&msg_techs, tech);
+ if (AST_VECTOR_APPEND(&msg_techs, tech)) {
+ ast_log(LOG_ERROR, "Failed to register message technology for '%s'\n",
+ tech->name);
+ ast_rwlock_unlock(&msg_techs_lock);
+ return -1;
+ }
ast_verb(3, "Message technology '%s' registered.\n", tech->name);
ast_rwlock_unlock(&msg_techs_lock);
@@ -1417,7 +1422,12 @@ int ast_msg_handler_register(const struct ast_msg_handler *handler)
return -1;
}
- AST_VECTOR_APPEND(&msg_handlers, handler);
+ if (AST_VECTOR_APPEND(&msg_handlers, handler)) {
+ ast_log(LOG_ERROR, "Failed to register message handler for '%s'\n",
+ handler->name);
+ ast_rwlock_unlock(&msg_handlers_lock);
+ return -1;
+ }
ast_verb(2, "Message handler '%s' registered.\n", handler->name);
ast_rwlock_unlock(&msg_handlers_lock);