Commit 27aca88e authored by G Edhaya Chandran's avatar G Edhaya Chandran
Browse files

Added clarification AuthVariableDERConfTest

Added clarification comments in AuthVariableDERConfTest for EFI_VARIABLE_AUTHENTICATED_WRITE_ACCESS flag testing
parent dc6efa23
......@@ -22,7 +22,7 @@ Abstract:
--*/
#include "SctLib.h"
#include "SctLib.h"
#include "VariableServicesBBTestMain.h"
#include "AuthVariableServicesBBTest.h"
......@@ -228,6 +228,10 @@ AuthVariableDERConfTest (
for (Index = 0; Index < sizeof (AttrArray) / sizeof (AttrArray[0]); Index = Index + 1) {
Attr = AttrArray[Index];
//According to UEFI Specifications: EFI_VARIABLE_AUTHENTICATED_WRITE_ACCESS is deprecated and should not be used.
//Platforms should return EFI_UNSUPPORTED if a caller to SetVariable() specifies this attribute.
//The below case of appending EFI_VARIABLE_AUTHENTICATED_WRITE_ACCESS to the Attr verifies the same.
//Expected return value is EFI_UNSUPPORTED
Attr |= EFI_VARIABLE_AUTHENTICATED_WRITE_ACCESS;
Status = RT->SetVariable (
......@@ -287,7 +291,7 @@ AuthVariableDERConfTest (
&MaximumVariableSize
);
if (Status == EFI_SUCCESS || Status == EFI_UNSUPPORTED) {
if (Status == EFI_SUCCESS || Status == EFI_UNSUPPORTED) {
Result = EFI_TEST_ASSERTION_PASSED;
} else {
Result = EFI_TEST_ASSERTION_FAILED;
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment