diff options
| author | Max Kirillov <max@max630.net> | 2018-09-07 05:36:07 +0200 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2018-09-07 21:35:51 +0200 |
| commit | 574c513e8dda5598e9e08e8ca2a048bf120a5709 (patch) | |
| tree | a2088dcfd3318c89bfda556e45c8347b31f30341 /http-backend.c | |
| parent | t5562: avoid non-portable "export FOO=bar" construct (diff) | |
| download | git-574c513e8dda5598e9e08e8ca2a048bf120a5709.tar.xz git-574c513e8dda5598e9e08e8ca2a048bf120a5709.zip | |
http-backend: allow empty CONTENT_LENGTH
According to RFC3875, empty environment variable is equivalent to unset,
and for CONTENT_LENGTH it should mean zero body to read.
However, unset CONTENT_LENGTH is also used for chunked encoding to indicate
reading until EOF. At least, the test "large fetch-pack requests can be split
across POSTs" from t5551 starts faliing, if unset or empty CONTENT_LENGTH is
treated as zero length body. So keep the existing behavior as much as possible.
Add a test for the case.
Reported-By: Jelmer Vernooij <jelmer@jelmer.uk>
Signed-off-by: Max Kirillov <max@max630.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'http-backend.c')
| -rw-r--r-- | http-backend.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/http-backend.c b/http-backend.c index e88d29f62b..a1230d7ead 100644 --- a/http-backend.c +++ b/http-backend.c @@ -353,7 +353,7 @@ static ssize_t get_content_length(void) ssize_t val = -1; const char *str = getenv("CONTENT_LENGTH"); - if (str && !git_parse_ssize_t(str, &val)) + if (str && *str && !git_parse_ssize_t(str, &val)) die("failed to parse CONTENT_LENGTH: %s", str); return val; } |