Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

I found out that the msgstart and the binderoffset are calculated incorrectly for DTLS 1.3. I'm not sure exactly why, but it seems like it is off with the size of the handshake message header. Maybe you have a suggestion to a better fix than my original commit? It also appears that the maxfragmentlen is parsed wrongly. #6

Open
almesh opened this issue May 24, 2024 · 1 comment
Open

I found out that the msgstart and the binderoffset are calculated incorrectly for DTLS 1.3. I'm not sure exactly why, but it seems like it is off with the size of the handshake message header. Maybe you have a suggestion to a better fix than my original commit? It also appears that the maxfragmentlen is parsed wrongly. #6

almesh opened this issue May 24, 2024 · 1 comment
Labels
bug Something isn't working documentation Improvements or additions to documentation duplicate This issue or pull request already exists enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed invalid This doesn't seem right question Further information is requested wontfix This will not be worked on

Comments

@almesh
Copy link
Owner

almesh commented May 24, 2024

I found out that the msgstart and the binderoffset are calculated incorrectly for DTLS 1.3. I'm not sure exactly why, but it seems like it is off with the size of the handshake message header. Maybe you have a suggestion to a better fix than my original commit? It also appears that the maxfragmentlen is parsed wrongly.

Originally posted by @fwh-dc in openssl/openssl#24426
@almesh almesh self-assigned this May 24, 2024
@almesh almesh added bug Something isn't working documentation Improvements or additions to documentation duplicate This issue or pull request already exists enhancement New feature or request help wanted Extra attention is needed good first issue Good for newcomers invalid This doesn't seem right question Further information is requested wontfix This will not be worked on labels May 24, 2024
@almesh
Copy link
Owner Author

almesh commented May 24, 2024

abuwady1405@gmail.com

@almesh almesh removed their assignment Jul 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working documentation Improvements or additions to documentation duplicate This issue or pull request already exists enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed invalid This doesn't seem right question Further information is requested wontfix This will not be worked on
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@almesh