doveconf: Skip verifying settings for modules we don't care about.

This was done sometimes, but not always.

diff --git a/src/config/config-parser.c b/src/config/config-parser.c
index ccb4b7e051a2f15b9bdb670ac9da7d983aef7243..fbeb02aabc08caf88ea0d936eccb5fd6129b8bb8 100644 (file)
--- a/src/config/config-parser.c
+++ b/src/config/config-parser.c
@@ -286,12 +286,13 @@ config_filter_parser_check(struct config_parser_context *ctx,
                           const struct config_module_parser *p,
                           const char **error_r)
 {
-       /* skip checking settings we don't care about */
-       if (*ctx->module != '\0' &&
-           !config_module_want_parser(ctx->root_parsers, ctx->module, p->root))
-               return 0;
-
        for (; p->root != NULL; p++) {
+               /* skip checking settings we don't care about */
+               if (*ctx->module != '\0' &&
+                   !config_module_want_parser(ctx->root_parsers,
+                                              ctx->module, p->root))
+                       continue;
+
                settings_parse_var_skip(p->parser);
                if (!settings_parser_check(p->parser, ctx->pool, error_r))
                        return -1;