From f92b439ff1c1cee3bf592debdd839c5ad70699a8 Mon Sep 17 00:00:00 2001 From: Johan Wikman Date: Tue, 8 Dec 2015 09:03:28 +0200 Subject: [PATCH] Fix coverity report of log_manager.cc - Too small array. - Missing lock release. --- log_manager/log_manager.cc | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/log_manager/log_manager.cc b/log_manager/log_manager.cc index 5dad7bff4..d64fc49d2 100644 --- a/log_manager/log_manager.cc +++ b/log_manager/log_manager.cc @@ -132,7 +132,7 @@ __thread mxs_log_info_t mxs_log_tls = {0, 0}; * Global counter for each log file type. It indicates for how many sessions * each log type is currently enabled. */ -ssize_t mxs_log_session_count[LOG_DEBUG] = {0}; +ssize_t mxs_log_session_count[LOG_DEBUG + 1] = {0}; /** * BUFSIZ comes from the system. It equals with block size or @@ -926,6 +926,7 @@ static char* blockbuf_get_writepos(blockbuf_t** p_bb, */ if ((bb = blockbuf_init()) == NULL) { + simple_mutex_unlock(&bb_list->mlist_mutex); return NULL; }