From f5d2e7b779b417449109e672627e68cb69c14385 Mon Sep 17 00:00:00 2001
From: Michael Orlitzky <michael@orlitzky.com>
Date: Mon, 4 Nov 2024 18:47:46 -0500
Subject: [PATCH] ext/standard/tests/crypt/des_fallback_invalid_salt.phpt: less
 valid salt

Musl's crypt() implementation of DES tries to handle invalid salts and
can make this test fail because it returns an answer and not an
error. Even musl however will reject a salt with a ':' in it, so we
can make this test cross-platform by supplying an even less valid
salt.
---
 ext/standard/tests/crypt/des_fallback_invalid_salt.phpt | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/ext/standard/tests/crypt/des_fallback_invalid_salt.phpt b/ext/standard/tests/crypt/des_fallback_invalid_salt.phpt
index b0797657d80..8b00c81bbd1 100644
--- a/ext/standard/tests/crypt/des_fallback_invalid_salt.phpt
+++ b/ext/standard/tests/crypt/des_fallback_invalid_salt.phpt
@@ -3,8 +3,11 @@ Test DES with invalid fallback
 --FILE--
 <?php
 
-var_dump(crypt("test", "$#"));
-var_dump(crypt("test", "$5zd$01"));
+// musl tries to support invalid DES salts unless they would violate
+// the /etc/passwd format, so we include colons to ensure that musl
+// crypt() will fail on these inputs as well.
+var_dump(crypt("test", "$:#"));
+var_dump(crypt("test", "$:5zd$01\n"));
 
 ?>
 --EXPECT--