From 919e5a78344c6197187aa2612db01a0a3d5148f5 Mon Sep 17 00:00:00 2001
From: David Goodwin <david@codepoets.co.uk>
Date: Fri, 30 Dec 2022 20:41:28 +0000
Subject: [PATCH] stop non-string value for length breaking stuff - see
 https://github.com/postfixadmin/postfixadmin/issues/697 - thanks
 @bmatthewshea

---
 functions.inc.php | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/functions.inc.php b/functions.inc.php
index 7c81564b..a6de2b3a 100644
--- a/functions.inc.php
+++ b/functions.inc.php
@@ -877,10 +877,14 @@ function validate_password($password) {
     $result = array();
     $val_conf = Config::read_array('password_validation');
 
+    # legacy, used up to 2.3.x - check for backwards compatability.
     if (Config::has('min_password_length')) {
-        $minlen = (int)Config::read_string('min_password_length'); # used up to 2.3.x - check it for backward compatibility
-        if ($minlen > 0) {
-            $val_conf['/.{' . $minlen . '}/'] = "password_too_short $minlen";
+        $val = Config::read('min_password_length');
+        if (is_numeric($val)) {
+            $minlen = (int) $val;
+            if ($minlen > 0) {
+                $val_conf['/.{' . $minlen . '}/'] = "password_too_short $minlen";
+            }
         }
     }