Branch: refs/heads/master
  Home:   https://github.com/tianocore/edk2
  Commit: 67d3be644f9361c678f3b21a6f0b91ad8fda0383
      
https://github.com/tianocore/edk2/commit/67d3be644f9361c678f3b21a6f0b91ad8fda0383
  Author: Jiaxin Wu <jiaxin...@intel.com>
  Date:   2024-07-07 (Sun, 07 Jul 2024)

  Changed paths:
    M MdeModulePkg/Universal/FaultTolerantWriteDxe/FaultTolerantWriteSmm.c
    M MdeModulePkg/Universal/FaultTolerantWriteDxe/FaultTolerantWriteSmmCommon.h
    M 
MdeModulePkg/Universal/FaultTolerantWriteDxe/FaultTolerantWriteStandaloneMm.c
    M 
MdeModulePkg/Universal/FaultTolerantWriteDxe/FaultTolerantWriteTraditionalMm.c

  Log Message:
  -----------
  MdeModulePkg/FaultTolerantWriteSmm: Update buffer valid check func name

In the MdeModulePkg/FaultTolerantWriteSmm, the Primary Buffer
(CommBuffer) check function has been updated to match the buffer
validation behavior:

For SMM, the SMM Handlers is to validate the buffer outside MMRAM.

For MM, the MM Handlers do not need to validate the buffer if it is
the CommBuffer passed from MmCore through the MmiHandler() parameter.
Return TRUE directly in this case.

There is no function impact.

Signed-off-by: Jiaxin Wu <jiaxin...@intel.com>
Cc: Liming Gao <gaolim...@byosoft.com.cn>
Cc: Ray Ni <ray...@intel.com>
Cc: Star Zeng <star.z...@intel.com>
Cc: Hongbin1 Zhang <hongbin1.zh...@intel.com>
Cc: Wei6 Xu <wei6...@intel.com>
Cc: Dun Tan <dun....@intel.com>
Cc: Yuanhao Xie <yuanhao....@intel.com>


  Commit: c0021d31f847f9005cdc1386f8675375d455e8f2
      
https://github.com/tianocore/edk2/commit/c0021d31f847f9005cdc1386f8675375d455e8f2
  Author: Jiaxin Wu <jiaxin...@intel.com>
  Date:   2024-07-07 (Sun, 07 Jul 2024)

  Changed paths:
    M MdeModulePkg/Library/VarCheckPolicyLib/VarCheckPolicyLib.c
    M MdeModulePkg/Library/VarCheckPolicyLib/VarCheckPolicyLib.h
    M MdeModulePkg/Library/VarCheckPolicyLib/VarCheckPolicyLibStandaloneMm.c
    M MdeModulePkg/Library/VarCheckPolicyLib/VarCheckPolicyLibTraditional.c

  Log Message:
  -----------
  MdeModulePkg/VarCheckPolicyLib: Update buffer valid check func name

In the MdeModulePkg/VarCheckPolicyLib, the Primary Buffer
(CommBuffer) check function has been updated to match the buffer
validation behavior.

For SMM, the SMM Handlers is to validate the buffer outside MMRAM.

For MM, the MM Handlers do not need to validate the buffer if it is
the CommBuffer passed from MmCore through the MmiHandler() parameter.
Return TRUE directly in this case. Existing code is incorrect for
the MM check. This will be fixed in the following patch.

There is no function impact.

Signed-off-by: Jiaxin Wu <jiaxin...@intel.com>
Cc: Liming Gao <gaolim...@byosoft.com.cn>
Cc: Ray Ni <ray...@intel.com>
Cc: Star Zeng <star.z...@intel.com>
Cc: Hongbin1 Zhang <hongbin1.zh...@intel.com>
Cc: Wei6 Xu <wei6...@intel.com>
Cc: Dun Tan <dun....@intel.com>
Cc: Yuanhao Xie <yuanhao....@intel.com>


  Commit: acfdb6771cdf5db13f8a829b4e1c9f9b45178151
      
https://github.com/tianocore/edk2/commit/acfdb6771cdf5db13f8a829b4e1c9f9b45178151
  Author: Jiaxin Wu <jiaxin...@intel.com>
  Date:   2024-07-07 (Sun, 07 Jul 2024)

  Changed paths:
    M MdeModulePkg/Library/VarCheckPolicyLib/VarCheckPolicyLibStandaloneMm.c

  Log Message:
  -----------
  MdeModulePkg/VarCheckPolicyLib: Fix buffer valid check for MM

For MM, the MM Handlers do not need to validate the buffer if it is
the CommBuffer passed from MmCore through the MmiHandler() parameter.
Return TRUE directly in this case. Fix buffer valid check for MM in
this patch.

Signed-off-by: Jiaxin Wu <jiaxin...@intel.com>
Cc: Liming Gao <gaolim...@byosoft.com.cn>
Cc: Ray Ni <ray...@intel.com>
Cc: Star Zeng <star.z...@intel.com>
Cc: Hongbin1 Zhang <hongbin1.zh...@intel.com>
Cc: Wei6 Xu <wei6...@intel.com>
Cc: Dun Tan <dun....@intel.com>
Cc: Yuanhao Xie <yuanhao....@intel.com>


  Commit: 8befdb144193f0fe10c39ab0c21e138c59018b05
      
https://github.com/tianocore/edk2/commit/8befdb144193f0fe10c39ab0c21e138c59018b05
  Author: Jiaxin Wu <jiaxin...@intel.com>
  Date:   2024-07-07 (Sun, 07 Jul 2024)

  Changed paths:
    M MdeModulePkg/Universal/Variable/RuntimeDxe/PrivilegePolymorphic.h
    M MdeModulePkg/Universal/Variable/RuntimeDxe/VariableSmm.c
    M MdeModulePkg/Universal/Variable/RuntimeDxe/VariableStandaloneMm.c
    M MdeModulePkg/Universal/Variable/RuntimeDxe/VariableTraditionalMm.c

  Log Message:
  -----------
  MdeModulePkg/VariableSmm: Add func for Primary Buffer valid check

Add a new function (VariableSmmIsPrimaryBufferValid) to check
Primary Buffer valid or not.

original function (VariableSmmIsBufferOutsideSmmValid) is used to
check the buffer outside MMRAM.

Signed-off-by: Jiaxin Wu <jiaxin...@intel.com>
Cc: Liming Gao <gaolim...@byosoft.com.cn>
Cc: Ray Ni <ray...@intel.com>
Cc: Star Zeng <star.z...@intel.com>
Cc: Hongbin1 Zhang <hongbin1.zh...@intel.com>
Cc: Wei6 Xu <wei6...@intel.com>
Cc: Dun Tan <dun....@intel.com>
Cc: Yuanhao Xie <yuanhao....@intel.com>


  Commit: 0986faad973c8d2e98cb8733f9c58d0210f458f4
      
https://github.com/tianocore/edk2/commit/0986faad973c8d2e98cb8733f9c58d0210f458f4
  Author: Jiaxin Wu <jiaxin...@intel.com>
  Date:   2024-07-07 (Sun, 07 Jul 2024)

  Changed paths:
    M MdeModulePkg/Universal/Variable/RuntimeDxe/PrivilegePolymorphic.h
    M MdeModulePkg/Universal/Variable/RuntimeDxe/VariableSmm.c
    M MdeModulePkg/Universal/Variable/RuntimeDxe/VariableStandaloneMm.c
    M MdeModulePkg/Universal/Variable/RuntimeDxe/VariableStandaloneMm.inf
    M MdeModulePkg/Universal/Variable/RuntimeDxe/VariableTraditionalMm.c

  Log Message:
  -----------
  MdeModulePkg/VariableSmm: Fix NonPrimary Buffer check issue

VariableSmmIsBufferOutsideSmmValid function is to check the buffer
is outside SMM or not. This patch fix the issue that always return
true for MM.

Meanwhile, this patch renames VariableSmmIsBufferOutsideSmmValid
to VariableSmmIsNonPrimaryBufferValid.

Signed-off-by: Jiaxin Wu <jiaxin...@intel.com>
Cc: Liming Gao <gaolim...@byosoft.com.cn>
Cc: Ray Ni <ray...@intel.com>
Cc: Star Zeng <star.z...@intel.com>
Cc: Hongbin1 Zhang <hongbin1.zh...@intel.com>
Cc: Wei6 Xu <wei6...@intel.com>
Cc: Dun Tan <dun....@intel.com>
Cc: Yuanhao Xie <yuanhao....@intel.com>


  Commit: d5fad2176cb14283922e07ff1758118d16b17383
      
https://github.com/tianocore/edk2/commit/d5fad2176cb14283922e07ff1758118d16b17383
  Author: Jiaxin Wu <jiaxin...@intel.com>
  Date:   2024-07-07 (Sun, 07 Jul 2024)

  Changed paths:
    M SecurityPkg/Tcg/Tcg2Smm/Tcg2Smm.c
    M SecurityPkg/Tcg/Tcg2Smm/Tcg2Smm.h
    M SecurityPkg/Tcg/Tcg2Smm/Tcg2StandaloneMm.c
    M SecurityPkg/Tcg/Tcg2Smm/Tcg2TraditionalMm.c

  Log Message:
  -----------
  SecurityPkg/Tcg: Correct buffer valid check func

For SMM, the SMM Handlers is to validate the buffer outside MMRAM
including the Primary & NonPrimary buffer.

For MM, the MM Handlers do not need to validate the Primary buffer
if it is passed from MmCore through the MmiHandler() parameter.
Return TRUE directly in this case. But need to validate NonPrimary
buffer that outside MMRAM.

Signed-off-by: Jiaxin Wu <jiaxin...@intel.com>
Cc: Jiewen Yao <jiewen....@intel.com>
Cc: Ray Ni <ray...@intel.com>
Cc: Star Zeng <star.z...@intel.com>
Cc: Hongbin1 Zhang <hongbin1.zh...@intel.com>
Cc: Wei6 Xu <wei6...@intel.com>
Cc: Dun Tan <dun....@intel.com>
Cc: Yuanhao Xie <yuanhao....@intel.com>


Compare: https://github.com/tianocore/edk2/compare/26bc42f1e34c...d5fad2176cb1

To unsubscribe from these emails, change your notification settings at 
https://github.com/tianocore/edk2/settings/notifications


_______________________________________________
edk2-commits mailing list
edk2-commits@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/edk2-commits

Reply via email to