Skip to content

[Core] Improve HPACK#33597

Merged
veblush merged 2 commits intogrpc:masterfrom
veblush:hpack-patch
Jul 5, 2023
Merged

[Core] Improve HPACK#33597
veblush merged 2 commits intogrpc:masterfrom
veblush:hpack-patch

Conversation

@veblush
Copy link
Copy Markdown
Contributor

@veblush veblush commented Jul 5, 2023

No description provided.

@veblush veblush added the release notes: no Indicates if PR should not be in release notes label Jul 5, 2023
@veblush veblush requested a review from ctiller as a code owner July 5, 2023 00:57
@veblush veblush changed the title Improve HPACK [Core] Improve HPACK Jul 5, 2023
@veblush veblush requested a review from yashykt July 5, 2023 00:59
@veblush veblush merged commit 415f2f9 into grpc:master Jul 5, 2023
@copybara-service copybara-service bot added the imported Specifies if the PR has been imported to the internal repository label Jul 5, 2023
mario-vimal pushed a commit to mario-vimal/grpc that referenced this pull request Jul 5, 2023
veblush added a commit that referenced this pull request Jul 5, 2023
veblush added a commit that referenced this pull request Jul 5, 2023
mario-vimal pushed a commit to mario-vimal/grpc that referenced this pull request Jul 13, 2023
@dn-codes
Copy link
Copy Markdown

dn-codes commented Sep 1, 2023

@santiagorr
Copy link
Copy Markdown

Dear grpc develpers, I would like to study how feasible is to fix https://security-tracker.debian.org/tracker/CVE-2023-33953 in the supported debian versions, and knowing which commits fixed (and introduced) the issue would help a lot. Could you please confirm if this MR fixes that CVE or not?

I know you recommend upgrading to >= 1.53.2, but that is not possible in released debian versions.

TIA.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/core bloat/low imported Specifies if the PR has been imported to the internal repository lang/core per-call-memory/neutral per-channel-memory/neutral release notes: no Indicates if PR should not be in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants