From: Iker Pedrosa <ipedrosa@redhat.com>
Date: Fri, 6 Feb 2026 15:31:56 +0000 (+0100)
Subject: lib/chkhash.c: fix yescrypt hash length comment
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=bc09948273c34a4eeceec028036ec69b3fd4b754;p=thirdparty%2Fshadow.git

lib/chkhash.c: fix yescrypt hash length comment

Fix misleading comment that stated "43-char (minimum) hash" when
the actual regex pattern requires exactly 43 characters. Update
comment to accurately reflect the implementation behavior.

Signed-off-by: Iker Pedrosa <ipedrosa@redhat.com>
Reviewed-by: Alejandro Colomar <alx@kernel.org>
---

diff --git a/lib/chkhash.c b/lib/chkhash.c
index 9123038ad..6ecdbac79 100644
--- a/lib/chkhash.c
+++ b/lib/chkhash.c
@@ -53,7 +53,7 @@ is_valid_hash(const char *hash)
 	if (strlen(hash) < 13)
 		return false;
 
-	// Yescrypt: $y$ + algorithm parameters + $ + salt + $ + 43-char (minimum) hash
+	// Yescrypt: $y$ + algorithm parameters + $ + salt + $ + 43-char hash
 	if (match_regex("^\\$y\\$[./A-Za-z0-9]+\\$[./A-Za-z0-9]{1,86}\\$[./A-Za-z0-9]{43}$", hash))
 		return true;