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.

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

Issue 196 #197

Merged
merged 2 commits into from
Jul 14, 2023
Merged

Issue 196 #197

merged 2 commits into from
Jul 14, 2023

Conversation

boaks
Copy link
Contributor

@boaks boaks commented Apr 28, 2023

Fixes issue #196

@boaks boaks force-pushed the issue_196 branch 2 times, most recently from 5e8ee87 to 1cdbbab Compare April 30, 2023 06:06
@boaks
Copy link
Contributor Author

boaks commented May 17, 2023

Added logging message in case of length violations.

dtls.c Outdated Show resolved Hide resolved
dtls.c Show resolved Hide resolved
dtls.c Outdated Show resolved Hide resolved
boaks added 2 commits July 14, 2023 09:18
Add GET_VAR_FIELD.

Signed-off-by: Achim Kraus <achim.kraus@cloudcoap.net>
Check for field length before reading the length to skip from the field.

Signed-off-by: Achim Kraus <achim.kraus@cloudcoap.net>
Copy link
Contributor

@obgm obgm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@boaks boaks merged commit 7667849 into eclipse:main Jul 14, 2023
@boaks boaks deleted the issue_196 branch August 7, 2023 13:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants