]> git.ipfire.org Git - thirdparty/tvheadend.git/commitdiff
extjs: fix wrong code path in extjs_tvhlog() - coverity
authorJaroslav Kysela <perex@perex.cz>
Fri, 3 Oct 2014 20:48:56 +0000 (22:48 +0200)
committerJaroslav Kysela <perex@perex.cz>
Fri, 3 Oct 2014 20:48:56 +0000 (22:48 +0200)
src/webui/extjs.c

index cf155825f316c31e9f3c6f7f50b6938ad2d95766..b1b90749b13b08d969d8357d1437849b43e917f6 100755 (executable)
@@ -580,6 +580,10 @@ extjs_tvhlog(http_connection_t *hc, const char *remain, void *opaque)
     /* Get config */
     pthread_mutex_lock(&tvhlog_mutex);
     m = htsmsg_create_map();
+    if (!m) {
+      pthread_mutex_unlock(&tvhlog_mutex);
+      return HTTP_STATUS_BAD_REQUEST;
+    }
     htsmsg_add_u32(m, "tvhlog_level",      tvhlog_level);
     htsmsg_add_u32(m, "tvhlog_trace_on",   tvhlog_level > LOG_DEBUG);
     tvhlog_get_trace(str, sizeof(str));
@@ -592,7 +596,6 @@ extjs_tvhlog(http_connection_t *hc, const char *remain, void *opaque)
                    tvhlog_options & TVHLOG_OPT_DBG_SYSLOG);
     pthread_mutex_unlock(&tvhlog_mutex);
     
-    if (!m) return HTTP_STATUS_BAD_REQUEST;
     out = json_single_record(m, "config");
 
   /* Save settings */