search

mouse07410 / asn1c

The ASN.1 Compiler
http://lionet.info/asn1c/
BSD 2-Clause "Simplified" License
101 stars 73 forks source link

Sequenceof setof infinite loop in constraint check #64

Closed robstradling closed 4 years ago

robstradling commented 4 years ago

From https://github.com/vlm/asn1c/pull/380

robstradling commented 4 years ago

Hmm, I don't think I created this PR correctly! Sorry.

mouse07410 commented 4 years ago

@robstradling That's fine. Note that the repos do differ (which is one reason why a rebase is necessary). Quite a few of the PRs from the upstream have been adopted and merged here.

Your original PR #380 was pretty small. The main reason I didn't just merge it was that it broke the CI, without an obvious (to me) cause.

If you have time, I'd recommend cloning this repo, testing it against your use case, assuming it exhibits the same problem - applying your fix (a two-liner?), and (now the hard part) figuring why CI becomes unhappy when it sees it.