regex match: Require *all* regular expressions to match.

Before, when there were multiple regular expressions for the same field,
for example the host, only of of those regular expressions needed to
match for a successful overall match.

The new behavior is not that *all* regular expressions need to match for
a successful overall match. This makes the behavior much easier to
understand.

diff --git a/src/match_regex.c b/src/match_regex.c
index 15f3c77..9233d80 100644 (file)
--- a/src/match_regex.c
+++ b/src/match_regex.c
@@ -109,17 +109,17 @@ static int mr_match_regexen (mr_regex_t *re_head, /* {{{ */
                {
                        DEBUG ("regex match: Regular expression `%s' matches `%s'.",
                                        re->re_str, string);
-                       return (FC_MATCH_MATCHES);
                }
                else
                {
                        DEBUG ("regex match: Regular expression `%s' does not match `%s'.",
                                        re->re_str, string);
+                       return (FC_MATCH_NO_MATCH);
                }
 
        }
 
-       return (FC_MATCH_NO_MATCH);
+       return (FC_MATCH_MATCHES);
 } /* }}} int mr_match_regexen */
 
 static int mr_config_add_regex (mr_regex_t **re_head, /* {{{ */