From e0222c00293573e83d85fb5dfc43d67b4b837dda Mon Sep 17 00:00:00 2001 From: Stefan Eissing Date: Sat, 4 Jul 2026 17:20:13 +0200 Subject: [PATCH] test2114: fix reading from stdin The test provides an invalid username, but no ':' with password, so that the curl tool tries to read the password from stdin. That makes the test hang, also in CI. The check for invalid characters is happening when the CURLOPT_USERPWD is applied to the URL, so when the transfer is started. Maybe the check should happen also in setopt already. The test hangs because tool_paramhlp.c:548 `checkpasswd()` does not see a ':' and tries to read the password from stdin. Fix: add a ':' in the tests invalid user setting, so that the read does not happen. Follow-up to ecaa344a7a17874ff0c6e5 Closes #22264 --- tests/data/test2114 | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/data/test2114 b/tests/data/test2114 index fe8bc1cf98..eaeefa6a52 100644 --- a/tests/data/test2114 +++ b/tests/data/test2114 @@ -21,7 +21,7 @@ FTP username with embedded CRLF is rejected # read the credentials from a config file so the raw CR LF reaches curl intact # regardless of how the platform passes command line arguments -user = "anonymous\r\nSTOR /%TESTNUMBER" +user = "anonymous\r\nSTOR: /%TESTNUMBER" ftp://%HOSTIP:%FTPPORT/%TESTNUMBER -K %LOGDIR/test%TESTNUMBER.config