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