Branch: refs/heads/master
  Home:   https://github.com/tianocore/edk2
  Commit: 67d3be644f9361c678f3b21a6f0b91ad8fda0383
      
https://github.com/tianocore/edk2/commit/67d3be644f9361c678f3b21a6f0b91ad8fda0383
  Author: Jiaxin Wu <[email protected]>
  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 <[email protected]>
Cc: Liming Gao <[email protected]>
Cc: Ray Ni <[email protected]>
Cc: Star Zeng <[email protected]>
Cc: Hongbin1 Zhang <[email protected]>
Cc: Wei6 Xu <[email protected]>
Cc: Dun Tan <[email protected]>
Cc: Yuanhao Xie <[email protected]>


  Commit: c0021d31f847f9005cdc1386f8675375d455e8f2
      
https://github.com/tianocore/edk2/commit/c0021d31f847f9005cdc1386f8675375d455e8f2
  Author: Jiaxin Wu <[email protected]>
  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 <[email protected]>
Cc: Liming Gao <[email protected]>
Cc: Ray Ni <[email protected]>
Cc: Star Zeng <[email protected]>
Cc: Hongbin1 Zhang <[email protected]>
Cc: Wei6 Xu <[email protected]>
Cc: Dun Tan <[email protected]>
Cc: Yuanhao Xie <[email protected]>


  Commit: acfdb6771cdf5db13f8a829b4e1c9f9b45178151
      
https://github.com/tianocore/edk2/commit/acfdb6771cdf5db13f8a829b4e1c9f9b45178151
  Author: Jiaxin Wu <[email protected]>
  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 <[email protected]>
Cc: Liming Gao <[email protected]>
Cc: Ray Ni <[email protected]>
Cc: Star Zeng <[email protected]>
Cc: Hongbin1 Zhang <[email protected]>
Cc: Wei6 Xu <[email protected]>
Cc: Dun Tan <[email protected]>
Cc: Yuanhao Xie <[email protected]>


  Commit: 8befdb144193f0fe10c39ab0c21e138c59018b05
      
https://github.com/tianocore/edk2/commit/8befdb144193f0fe10c39ab0c21e138c59018b05
  Author: Jiaxin Wu <[email protected]>
  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 <[email protected]>
Cc: Liming Gao <[email protected]>
Cc: Ray Ni <[email protected]>
Cc: Star Zeng <[email protected]>
Cc: Hongbin1 Zhang <[email protected]>
Cc: Wei6 Xu <[email protected]>
Cc: Dun Tan <[email protected]>
Cc: Yuanhao Xie <[email protected]>


  Commit: 0986faad973c8d2e98cb8733f9c58d0210f458f4
      
https://github.com/tianocore/edk2/commit/0986faad973c8d2e98cb8733f9c58d0210f458f4
  Author: Jiaxin Wu <[email protected]>
  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 <[email protected]>
Cc: Liming Gao <[email protected]>
Cc: Ray Ni <[email protected]>
Cc: Star Zeng <[email protected]>
Cc: Hongbin1 Zhang <[email protected]>
Cc: Wei6 Xu <[email protected]>
Cc: Dun Tan <[email protected]>
Cc: Yuanhao Xie <[email protected]>


  Commit: d5fad2176cb14283922e07ff1758118d16b17383
      
https://github.com/tianocore/edk2/commit/d5fad2176cb14283922e07ff1758118d16b17383
  Author: Jiaxin Wu <[email protected]>
  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 <[email protected]>
Cc: Jiewen Yao <[email protected]>
Cc: Ray Ni <[email protected]>
Cc: Star Zeng <[email protected]>
Cc: Hongbin1 Zhang <[email protected]>
Cc: Wei6 Xu <[email protected]>
Cc: Dun Tan <[email protected]>
Cc: Yuanhao Xie <[email protected]>


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
[email protected]
https://lists.sourceforge.net/lists/listinfo/edk2-commits

Reply via email to