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

Add support for custom constructed ASN.1 types #624

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from
Open

Conversation

sjaeckel
Copy link
Member

  • tests are added or updated

Check commit messages for details.

This should fix #622. @stevemit can you please test it for your use case besides the toy example?

@sjaeckel sjaeckel closed this Jun 21, 2023
@sjaeckel sjaeckel reopened this Jun 21, 2023
@sjaeckel sjaeckel closed this Aug 7, 2023
@sjaeckel sjaeckel deleted the fix-622 branch August 7, 2023 13:53
@sjaeckel sjaeckel restored the fix-622 branch August 7, 2023 13:54
@sjaeckel sjaeckel reopened this Aug 7, 2023
Don't read more than the length indicated by the length field.

Signed-off-by: Steffen Jaeckel <[email protected]>
So we don't have to `#ifdef` whether it's available or not.

Signed-off-by: Steffen Jaeckel <[email protected]>
This fixes #622

Signed-off-by: Steffen Jaeckel <[email protected]>
@levitte
Copy link
Collaborator

levitte commented Aug 21, 2024

I just had a quick look, and ran the test too, and this does appear to answer #622.
I would merge this.

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.

How to encode [0] IMPLICIT SET OF?
2 participants