[PATCH] efi_loader: using EFI_UNSUPPORTED for private authenticated variables

[Ilias Apalodimas]

On Wed, 10 Apr 2024 at 14:29, Heinrich Schuchardt <xypron.glpk at gmx.de> wrote:
>
> On 10.04.24 14:19, Weizhao Ouyang wrote:
> > Improve error message for UEFI SCT tests.
> >
> > Signed-off-by: Weizhao Ouyang <o451686892 at gmail.com>
> > ---
> >   lib/efi_loader/efi_variable.c | 1 +
> >   1 file changed, 1 insertion(+)
> >
> > diff --git a/lib/efi_loader/efi_variable.c b/lib/efi_loader/efi_variable.c
> > index 2951dc78c7..e6c1219a11 100644
> > --- a/lib/efi_loader/efi_variable.c
> > +++ b/lib/efi_loader/efi_variable.c
> > @@ -163,6 +163,7 @@ static efi_status_t efi_variable_authenticate(const u16 *variable,
> >               break;
> >       default:
> >               /* TODO: support private authenticated variables */
> > +             ret = EFI_UNSUPPORTED;
>
> This looks more adequate than EFI_SECURITY_VIOLATION. Thanks.
>
> Reviewed-by: Heinrich Schuchardt <xypron.glpk at gmx.de>
>
> >               goto err;
> >       }
> >
>

Reviewed-by: Ilias Apalodimas <ilias.apalodimas at linaro.org>