[Skiboot] [PATCH 3/7] secvar/backend: fix an integer underflow bug

Daniel Axtens dja at axtens.net
Thu Jul 8 19:57:43 AEST 2021


Nayna <nayna at linux.vnet.ibm.com> writes:

> On 7/1/21 8:41 AM, Daniel Axtens wrote:
>> If a declared size is smaller than uuid size, we end up allocating
>> with an allocation of a 'negative' number, which is a huge 64 bit
>> number.
>
> Wouldn't Nick's patch 
> (https://lists.ozlabs.org/pipermail/skiboot/2021-July/017641.html) handle 
> this ?

I don't think so: I had applied an earlier version of that patch when I
was developing this fuzzing and my test cases.

v2 has a test case so you could test with and without Nick's commit if
you wanted to be sure.

Kind regards,
Daniel

> Thanks & Regards,
>
>       - Nayna


More information about the Skiboot mailing list