On Mon Oct 16 15:16:12 2023 +0000, Giovanni Mascellani wrote:
I don't think you're checking the number of operands available in this node, are you? You do the check in `sm6_parser_resources_load_cbv()`, but that's only for CBV and it's too late anyway.
Yeah, I moved this code up so parsing starts at operand 0, but that makes it unchecked. I think reading the id and range after the `switch` is better than redundant validation, so I moved them back down.