Branch: refs/heads/master
Home: https://github.com/tianocore/edk2
Commit: 048e36d5af8c536087f4f821a8db1f5340441181
https://github.com/tianocore/edk2/commit/048e36d5af8c536087f4f821a8db1f5340441181
Author: Luigi Leonardi <[email protected]>
Date: 2026-07-20 (Mon, 20 Jul 2026)
Changed paths:
R OvmfPkg/VirtMmCommunicationDxe/QemuX64.c
M OvmfPkg/VirtMmCommunicationDxe/VirtMmCommunication.inf
Log Message:
-----------
OvmfPkg/VirtMmCommunicationDxe: Remove unused QemuX64.c
QemuX64.c implemented PIO-based MM communication for the uefi-vars-isa
device. It was already commented out in VirtMmCommunication.inf and is
not referenced by any build file. Remove the dead code.
Signed-off-by: Luigi Leonardi <[email protected]>
Commit: 5dc785e7f664af82f06e9d450df398618edd0396
https://github.com/tianocore/edk2/commit/5dc785e7f664af82f06e9d450df398618edd0396
Author: Luigi Leonardi <[email protected]>
Date: 2026-07-20 (Mon, 20 Jul 2026)
Changed paths:
M OvmfPkg/VirtMmCommunicationDxe/QemuFdt.c
Log Message:
-----------
OvmfPkg/VirtMmCommunicationDxe: Fix unchecked VirtMmGetProp return values
VirtMmHwFind() calls VirtMmGetProp() twice but does not assign the
return value to Status. The subsequent EFI_ERROR(Status) checks test
the stale Status from LocateProtocol(), which always succeeds at that
point, so FDT lookup failures are silently ignored and the function
proceeds with uninitialised Ranges/Reg pointers.
Assign both VirtMmGetProp() return values to Status so the error
checks work as intended.
Fixes: 9dd47eeea1 ("OvmfPkg: add new VirtMmCommunicationDxe driver")
Signed-off-by: Luigi Leonardi <[email protected]>
Commit: 99942e353274d6a4641c71e21399cd4ab0feca11
https://github.com/tianocore/edk2/commit/99942e353274d6a4641c71e21399cd4ab0feca11
Author: Luigi Leonardi <[email protected]>
Date: 2026-07-20 (Mon, 20 Jul 2026)
Changed paths:
M OvmfPkg/VirtMmCommunicationDxe/VirtMmCommunication.c
Log Message:
-----------
OvmfPkg/VirtMmCommunicationDxe: Fix NULL dereference of optional CommSize
VirtMmCommunication2Communicate() dereferences *CommSize in a DEBUG
statement before checking whether CommSize is NULL. CommSize is an
optional parameter, so callers may legitimately pass NULL, causing a
NULL pointer dereference in debug builds.
Guard the dereference with a ternary so the DEBUG prints 0 when
CommSize is NULL.
Fixes: 9dd47eeea1 ("OvmfPkg: add new VirtMmCommunicationDxe driver")
Signed-off-by: Luigi Leonardi <[email protected]>
Compare: https://github.com/tianocore/edk2/compare/264f812b70c8...99942e353274
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
