doveconf: Fix infinite loop when hiding sensitive information

For example with "nopassword=y". Broken in
fc02343fd81a6fab272ac0366b15fc60ec44a8b6

diff --git a/src/config/doveconf.c b/src/config/doveconf.c
index 1705329d80748016deac6d04d46d9be445aa6ad4..873a3cb24b98fda5758766fa44a3a067cb9ffe4f 100644 (file)
--- a/src/config/doveconf.c
+++ b/src/config/doveconf.c
@@ -222,6 +222,11 @@ hide_secrets_from_value(struct ostream *output, const char *key,
                                        ptr++;
                        }
                        optr = ptr;
+               } else {
+                       /* "secret" is prefixed with alphanumeric character,
+                          e.g. "nopassword". So it's not really a secret.
+                          Skip forward to avoid infinite loop. */
+                       ptr++;
                }
        }
        /* if we are dealing with output, send rest here */