On Wed, 2021-06-02 at 14:12 -0700, Brian Norris wrote: > Signal boost: > I've seen issues in this code in the past, as this is a custom format, > the docs are non-public, it interacts with ACPI tables that Intel > doesn't always get to review, and the parsing is all written in C... > ...I also think Dan's static checker warning below is correct. > > Luca, has this been addressed yet? You're right, the issue that Dan pointed out is indeed an issue. I'll send a fix as explained in my previous email. -- Cheers, Luca.