log: fix pattern matching
authorFerruh Yigit <ferruh.yigit@intel.com>
Wed, 16 May 2018 14:09:28 +0000 (15:09 +0100)
committerThomas Monjalon <thomas@monjalon.net>
Mon, 21 May 2018 13:49:27 +0000 (15:49 +0200)
loglevel set wrong when ":" is used as separator, like
--log-type="user:debug"

This is because fnmatch returns zero on success. Fixed fnmatch return
value check.

Fixes: 7f0bb634a140 ("log: add ability to match log type with globbing")

Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
lib/librte_eal/common/eal_common_log.c

index c660ca6..8181189 100644 (file)
@@ -184,7 +184,7 @@ rte_log_set_level_pattern(const char *pattern, uint32_t level)
                if (rte_logs.dynamic_types[i].name == NULL)
                        continue;
 
-               if (fnmatch(pattern, rte_logs.dynamic_types[i].name, 0))
+               if (fnmatch(pattern, rte_logs.dynamic_types[i].name, 0) == 0)
                        rte_logs.dynamic_types[i].loglevel = level;
        }