[PATCH v6 24/26] powerpc/pseries: Implement secvars for dynamic secure boot

Michael Ellerman michaele at au1.ibm.com
Mon Feb 13 16:26:45 AEDT 2023


Andrew Donnellan <ajd at linux.ibm.com> writes:
> On Fri, 2023-02-10 at 16:28 -0500, Stefan Berger wrote:
>> > > +err:
>> > > +    kfree(var.data);
>> > 
>> > remove the kfree()
>> 
>> Actually don't remove it but it should probably be
>> 
>> if (var.data != &version)
>>      kfree(var.data);
>> 
>
> Argh, thanks for catching this.
>
> I don't think the condition is needed - we can assume the var.data is
> unmodified.
>
> mpe, are you able to fix this up in merge?

Yeah, can you reply here with the delta you want applied.

cheers


More information about the Linuxppc-dev mailing list