Branch: refs/heads/master
  Home:   https://github.com/tianocore/edk2
  Commit: 256c4470f86e53661c070f8c64a1052e975f9ef0
      
https://github.com/tianocore/edk2/commit/256c4470f86e53661c070f8c64a1052e975f9ef0
  Author: Michael D Kinney <[email protected]>
  Date:   2020-07-14 (Tue, 14 Jul 2020)

  Changed paths:
    M MdeModulePkg/Universal/Variable/RuntimeDxe/Variable.c

  Log Message:
  -----------
  MdeModulePkg/Variable/RuntimeDxe: Fix return status from Reclaim()

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2844

Update Reclaim() to return the error status from the reclaim
operation and not the status of SynchronizeRuntimeVariableCache()
that can be EFI_SUCCESS even through the status from reclaim
is an error.  Without this change, the return status from
SetVariable() can be EFI_SUCCESS even though the variable was
not actually set.  This occurs if the variable store is full
and a Reclaim() is invoked to free up space and even after all
possible space is freed, there is still not enough room for
the variable being set.  This condition should return
EFI_OUT_OF_RESOURCES.

Cc: Hao A Wu <[email protected]>
Cc: Liming Gao <[email protected]>
Signed-off-by: Michael D Kinney <[email protected]>
Reviewed-by: Hao A Wu <[email protected]>
Reviewed-by: Jian J Wang <[email protected]>




_______________________________________________
edk2-commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/edk2-commits

Reply via email to