Skip to content

h3+ech fix#13822

Closed
sftcd wants to merge 2 commits intocurl:masterfrom
sftcd:h3+ech-fix
Closed

h3+ech fix#13822
sftcd wants to merge 2 commits intocurl:masterfrom
sftcd:h3+ech-fix

Conversation

@sftcd
Copy link
Contributor

@sftcd sftcd commented May 29, 2024

This PR addresses a crash (in ECH code) when the experimental ECH feature is attempted with the experimental h3 feature. Note that the fix doesn't mean h3+ECH will work, it just means it won't cause a crash.

This addresses #13818

@github-actions github-actions bot added the TLS label May 29, 2024
@sftcd
Copy link
Contributor Author

sftcd commented May 29, 2024

I just noticed there could be a similar issue if using wolfssl+ECH+h3. Will check that out now.

I built a version with wolf+ECH+h3 and it doesn't have the same issue, so no change to the PR needed (for that reason:-) it seems.

@bagder bagder closed this in 48292d8 Jun 1, 2024
@bagder
Copy link
Member

bagder commented Jun 1, 2024

Thanks!

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

Labels

Development

Successfully merging this pull request may close these issues.

2 participants