]> Cypherpunks.ru repositories - gostls13.git/commit
[release-branch.go1.21] net/http: limit chunked data overhead
authorDamien Neil <dneil@google.com>
Tue, 7 Nov 2023 18:47:56 +0000 (10:47 -0800)
committerCarlos Amedee <carlos@golang.org>
Tue, 5 Dec 2023 17:18:23 +0000 (17:18 +0000)
commitec8c526e4be720e94b98ca509e6364f0efaf28f7
treed634834dc11bff7e27c479fe1ea09d9f4a088e8f
parentf7a79cb5fc0a57cabcf2cf2651f4e7573b4be1de
[release-branch.go1.21] net/http: limit chunked data overhead

The chunked transfer encoding adds some overhead to
the content transferred. When writing one byte per
chunk, for example, there are five bytes of overhead
per byte of data transferred: "1\r\nX\r\n" to send "X".

Chunks may include "chunk extensions",
which we skip over and do not use.
For example: "1;chunk extension here\r\nX\r\n".

A malicious sender can use chunk extensions to add
about 4k of overhead per byte of data.
(The maximum chunk header line size we will accept.)

Track the amount of overhead read in chunked data,
and produce an error if it seems excessive.

Updates #64433
Fixes #64435
Fixes CVE-2023-39326

Change-Id: I40f8d70eb6f9575fb43f506eb19132ccedafcf39
Reviewed-on: https://team-review.git.corp.google.com/c/golang/go-private/+/2076135
Reviewed-by: Tatiana Bradley <tatianabradley@google.com>
Reviewed-by: Roland Shoemaker <bracewell@google.com>
(cherry picked from commit 3473ae72ee66c60744665a24b2fde143e8964d4f)
Reviewed-on: https://team-review.git.corp.google.com/c/golang/go-private/+/2095408
Run-TryBot: Roland Shoemaker <bracewell@google.com>
Reviewed-by: Damien Neil <dneil@google.com>
Reviewed-on: https://go-review.googlesource.com/c/go/+/547356
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
src/net/http/internal/chunked.go
src/net/http/internal/chunked_test.go