Re: [PATCH] efi_loader: using EFI_UNSUPPORTED for private authenticated variables

[Heinrich Schuchardt]

On 10.04.24 14:19, Weizhao Ouyang wrote:

Improve error message for UEFI SCT tests.

Signed-off-by: Weizhao Ouyang <o451686...@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.g...@gmx.de>

                goto err;
        }