t5562: do not depend on /dev/zero
It was reported [1] that NonStop platform does not have /dev/zero. The test uses /dev/zero as a dummy input. Passing case (http-backed failed because of too big input size) should not be reading anything from it. If http-backend would erroneously try to read any data returning EOF probably would be even safer than providing some meaningless data. Replace /dev/zero with /dev/null to avoid issues with platforms which do not have /dev/zero. [1] https://public-inbox.org/git/20190209185930.5256-4-randall.s.becker@rogers.com/ Reported-by: Randall S. Becker <rsbecker@nexbridge.com> Signed-off-by: Max Kirillov <max@max630.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Max Kirillov committed
Feb 15, 2019 at 18:42 UTC
709417507567d1423acd06f55fb6256af7ad3847
1 file changed
+1
-1
t/t5562-http-backend-content-length.sh
+1
-1
@@ -150,7 +150,7 @@ test_expect_success 'CONTENT_LENGTH overflow ssite_t' '
150
GIT_HTTP_EXPORT_ALL=TRUE \
151
REQUEST_METHOD=POST \
152
CONTENT_LENGTH="$NOT_FIT_IN_SSIZE" \
153
- git http-backend </dev/zero >/dev/null 2>err &&
153
+ git http-backend </dev/null >/dev/null 2>err &&
154
grep "fatal:.*CONTENT_LENGTH" err
155
'
156