Fix flaw in detecting log format

The log message regex has been

[0-9]{4}-[0-9]{2}-[0-9]{2} [0-9]{2}:[0-9]{2}:[0-9]{2}\.[0-9]{3}\+[0-9]{4}: [0-9]+: debug|info|warning|error :

The precedence of '|' is high though, so this is equivalent to matching

   [0-9]{4}-[0-9]{2}-[0-9]{2} [0-9]{2}:[0-9]{2}:[0-9]{2}\.[0-9]{3}\+[0-9]{4}: [0-9]+: debug

Or

   info

Or

   warning

Or

   error :

Which is clearly not what it should have done. This caused the code to
skip over things which are not log messages. The solution is to simply
add brackets.

A test case is also added to validate correctness.

Signed-off-by: Daniel P. Berrange <berrange@redhat.com>

diff --git a/.gitignore b/.gitignore
index b5824dcd6a9596885f89a82cd326ad39c83c3f3a..db5abcdb427e589b9e6aa40022e783b6cb8cfc66 100644 (file)
--- a/.gitignore
+++ b/.gitignore
@@ -210,6 +210,7 @@
 /tests/virkeycodetest
 /tests/virkeyfiletest
 /tests/virlockspacetest
+/tests/virlogtest
 /tests/virnet*test
 /tests/virportallocatortest
 /tests/virshtest

diff --git a/src/util/virlog.c b/src/util/virlog.c
index 7ee5117088116b621d4526ce1f355c4218b559d2..cc5032fa37f7d2c555636295630c7976c0e7b6cd 100644 (file)
--- a/src/util/virlog.c
+++ b/src/util/virlog.c
@@ -83,7 +83,7 @@ static regex_t *virLogRegex = NULL;
 #define VIR_LOG_DATE_REGEX "[0-9]{4}-[0-9]{2}-[0-9]{2}"
 #define VIR_LOG_TIME_REGEX "[0-9]{2}:[0-9]{2}:[0-9]{2}\\.[0-9]{3}\\+[0-9]{4}"
 #define VIR_LOG_PID_REGEX "[0-9]+"
-#define VIR_LOG_LEVEL_REGEX "debug|info|warning|error"
+#define VIR_LOG_LEVEL_REGEX "(debug|info|warning|error)"
 
 #define VIR_LOG_REGEX \
     VIR_LOG_DATE_REGEX " " VIR_LOG_TIME_REGEX ": " \
@@ -1623,6 +1623,9 @@ virLogSetFromEnv(void)
 {
     char *debugEnv;
 
+    if (virLogInitialize() < 0)
+        return;
+
     debugEnv = getenv("LIBVIRT_DEBUG");
     if (debugEnv && *debugEnv)
         virLogParseDefaultPriority(debugEnv);

diff --git a/tests/Makefile.am b/tests/Makefile.am
index 3eda522fc9c43279ccc83b2e5f31a429f79fbab5..250cd8ca8b8dcdbc9a50bd4998503161034afdc7 100644 (file)
--- a/tests/Makefile.am
+++ b/tests/Makefile.am
@@ -126,6 +126,7 @@ test_programs = virshtest sockettest \
        viridentitytest \
        virkeycodetest \
        virlockspacetest \
+       virlogtest \
        virstringtest \
         virportallocatortest \
        sysinfotest \
@@ -716,6 +717,10 @@ virlockspacetest_SOURCES = \
        virlockspacetest.c testutils.h testutils.c
 virlockspacetest_LDADD = $(LDADDS)
 
+virlogtest_SOURCES = \
+       virlogtest.c testutils.h testutils.c
+virlogtest_LDADD = $(LDADDS)
+
 virportallocatortest_SOURCES = \
        virportallocatortest.c testutils.h testutils.c
 virportallocatortest_LDADD = $(LDADDS)

diff --git a/tests/virlogtest.c b/tests/virlogtest.c
new file mode 100644 (file)
index 0000000..dfe0f75
--- /dev/null
+++ b/tests/virlogtest.c
@@ -0,0 +1,67 @@
+/*
+ * Copyright (C) 2013 Red Hat, Inc.
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation; either
+ * version 2.1 of the License, or (at your option) any later version.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with this library.  If not, see
+ * <http://www.gnu.org/licenses/>.
+ *
+ */
+
+#include <config.h>
+
+#include "testutils.h"
+
+#include "virlog.h"
+
+struct testLogMatchData {
+    const char *str;
+    bool res;
+};
+
+static int
+testLogMatch(const void *opaque)
+{
+    const struct testLogMatchData *data = opaque;
+
+    bool got = virLogProbablyLogMessage(data->str);
+    if (got != data->res) {
+        fprintf(stderr, "Expected '%d' but got '%d' for '%s'\n",
+                data->res, got, data->str);
+        return -1;
+    }
+    return 0;
+}
+
+
+static int
+mymain(void)
+{
+    int ret = 0;
+
+#define TEST_LOG_MATCH(str, res)                                        \
+    do {                                                                \
+        struct testLogMatchData data = {                                \
+            str, res                                                    \
+        };                                                              \
+        if (virtTestRun("testLogMatch " # str, testLogMatch, &data) < 0) \
+            ret = -1;                                                   \
+    } while (0)
+
+    TEST_LOG_MATCH("2013-10-11 15:43:43.866+0000: 28302: info : libvirt version: 1.1.3", true);
+
+    TEST_LOG_MATCH("libvirt:  error : cannot execute binary /usr/libexec/libvirt_lxc: No such file or directory", false);
+
+    return ret;
+}
+
+VIRT_TEST_MAIN(mymain)