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

asn1: Fix handling of SIZE constraints #7575

Merged
merged 1 commit into from
Aug 21, 2023

Conversation

bjorng
Copy link
Contributor

@bjorng bjorng commented Aug 18, 2023

A constraint such as:

(SIZE (1..4), ...)

is supposed to be equivalent to:

(SIZE (1..4, ...))

but the former variant was ignored by the ASN.1 compiler, causing the size to be incorrectly encoded and decoded by PER backends.

A constraint such as:

    (SIZE (1..4), ...)

is supposed to be equivalent to:

    (SIZE (1..4, ...))

but the former variant was ignored by the ASN.1 compiler, causing the
size to be incorrectly encoded and decoded by PER backends.
@bjorng bjorng added team:VM Assigned to OTP team VM fix testing currently being tested, tag is used by OTP internal CI labels Aug 18, 2023
@bjorng bjorng self-assigned this Aug 18, 2023
@bjorng bjorng added this to the OTP-26.1 milestone Aug 18, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Aug 18, 2023

CT Test Results

    2 files    15 suites   21m 32s ⏱️
225 tests 223 ✔️ 2 💤 0
246 runs  244 ✔️ 2 💤 0

Results for commit 6b098b7.

♻️ This comment has been updated with latest results.

To speed up review, make sure that you have read Contributing to Erlang/OTP and that all checks pass.

See the TESTING and DEVELOPMENT HowTo guides for details about how to run test locally.

Artifacts

// Erlang/OTP Github Action Bot

@bjorng bjorng merged commit f5c45b6 into erlang:maint Aug 21, 2023
16 checks passed
@bjorng bjorng deleted the bjorn/asn1/constraints/OTP-18729 branch August 21, 2023 08:23
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix team:VM Assigned to OTP team VM testing currently being tested, tag is used by OTP internal CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant