diff --git a/0064-Backport-net-http-update-bundled-golang.org-x-net-ht.patch b/0064-Backport-net-http-update-bundled-golang.org-x-net-ht.patch new file mode 100644 index 0000000000000000000000000000000000000000..1a84c461dc0eefff699f29db677c8eb46929b290 --- /dev/null +++ b/0064-Backport-net-http-update-bundled-golang.org-x-net-ht.patch @@ -0,0 +1,88 @@ +From 7f16ae9ccc785bbffa7c64e14c5e9cb17cf2dc34 Mon Sep 17 00:00:00 2001 +From: Damien Neil +Date: Thu, 28 Mar 2024 16:49:40 -0700 +Subject: [PATCH] [Backport] net/http: update bundled golang.org/x/net/http2 + +Offering: Cloud Core Network +CVE: CVE-2023-45288 +Reference: https://go-review.googlesource.com/c/go/+/576075 + +Disable cmd/internal/moddeps test, since this update includes PRIVATE +track fixes. + +Note: The upstream does not submit this change to go1.17 according to the rules of MinorReleases. +Corego3.x are based on go1.17.8. Therefore, it need to submit the change to corego3.x. + +Edited-by: machangwang m00509938 + +Fixes CVE-2023-45288 +For #65051 +Fixes #65387 + +Change-Id: I17da6da2fe0dd70062b49f94377875acb34829a1 +Reviewed-on: https://team-review.git.corp.google.com/c/golang/go-private/+/2197267 +Reviewed-by: Dmitri Shuralyov +Run-TryBot: Damien Neil +Reviewed-by: Tatiana Bradley +Reviewed-on: https://go-review.googlesource.com/c/go/+/576075 +TryBot-Bypass: Dmitri Shuralyov +Commit-Queue: Dmitri Shuralyov +Auto-Submit: Dmitri Shuralyov +Reviewed-by: Than McIntosh +Signed-off-by: Ma Chang Wang machangwang@huawei.com +--- + src/net/http/h2_bundle.go | 31 +++++++++++++++++++++++++++++++ + 1 file changed, 31 insertions(+) + +diff --git a/src/net/http/h2_bundle.go b/src/net/http/h2_bundle.go +index b87d14b6ea..29b82abb66 100644 +--- a/src/net/http/h2_bundle.go ++++ b/src/net/http/h2_bundle.go +@@ -2842,6 +2842,7 @@ func (fr *http2Framer) readMetaFrame(hf *http2HeadersFrame) (*http2MetaHeadersFr + if size > remainSize { + hdec.SetEmitEnabled(false) + mh.Truncated = true ++ remainSize = 0 + return + } + remainSize -= size +@@ -2854,6 +2855,36 @@ func (fr *http2Framer) readMetaFrame(hf *http2HeadersFrame) (*http2MetaHeadersFr + var hc http2headersOrContinuation = hf + for { + frag := hc.HeaderBlockFragment() ++ ++ // Avoid parsing large amounts of headers that we will then discard. ++ // If the sender exceeds the max header list size by too much, ++ // skip parsing the fragment and close the connection. ++ // ++ // "Too much" is either any CONTINUATION frame after we've already ++ // exceeded the max header list size (in which case remainSize is 0), ++ // or a frame whose encoded size is more than twice the remaining ++ // header list bytes we're willing to accept. ++ if int64(len(frag)) > int64(2*remainSize) { ++ if http2VerboseLogs { ++ log.Printf("http2: header list too large") ++ } ++ // It would be nice to send a RST_STREAM before sending the GOAWAY, ++ // but the struture of the server's frame writer makes this difficult. ++ return nil, http2ConnectionError(http2ErrCodeProtocol) ++ } ++ ++ // Also close the connection after any CONTINUATION frame following an ++ // invalid header, since we stop tracking the size of the headers after ++ // an invalid one. ++ if invalid != nil { ++ if http2VerboseLogs { ++ log.Printf("http2: invalid header: %v", invalid) ++ } ++ // It would be nice to send a RST_STREAM before sending the GOAWAY, ++ // but the struture of the server's frame writer makes this difficult. ++ return nil, http2ConnectionError(http2ErrCodeProtocol) ++ } ++ + if _, err := hdec.Write(frag); err != nil { + return nil, http2ConnectionError(http2ErrCodeCompression) + } +-- +2.33.0 + diff --git a/golang.spec b/golang.spec index 23e16f5ea088593a48053c83acfba34b0f34972f..43c26b4175305fcc312aad1193fd4bf1b0f67053 100644 --- a/golang.spec +++ b/golang.spec @@ -63,7 +63,7 @@ Name: golang Version: 1.17.3 -Release: 31 +Release: 32 Summary: The Go Programming Language License: BSD and Public Domain URL: https://golang.org/ @@ -213,6 +213,7 @@ Patch6060: 0060-release-branch.go1.21-net-http-net-http-cookiejar-av.patch Patch6061: 0061-Backport-net-mail-properly-handle-special-characters.patch Patch6062: 0062-1.17-backport-runtime-decrement-netpollWaiters-in-ne.patch Patch6063: 0063-1.17-backport-runtime-adjust-netpollWaiters-after-go.patch +Patch6064: 0064-Backport-net-http-update-bundled-golang.org-x-net-ht.patch ExclusiveArch: %{golang_arches} @@ -451,6 +452,12 @@ fi %files devel -f go-tests.list -f go-misc.list -f go-src.list %changelog +* Tue Apr 16 2024 hanchao - 1.17.3-32 +- Type:CVE +- CVE:CVE-2023-45288 +- SUG:NA +- DESC:fix CVE-2023-45288 + * Fri Mar 29 2024 wangshuo - 1.17.3-31 - Type:bugfix - ID:NA