17 Oct
2023
17 Oct
'23
6 a.m.
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.
-- https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/401#note_48954