Branch: refs/heads/master
  Home:   https://github.com/tianocore/edk2
  Commit: a7c119589fe2c5829011e3bbe717285f2ff2ab28
      
https://github.com/tianocore/edk2/commit/a7c119589fe2c5829011e3bbe717285f2ff2ab28
  Author: Ray Ni <[email protected]>
  Date:   2026-02-27 (Fri, 27 Feb 2026)

  Changed paths:
    M MdeModulePkg/Library/PeiDebugLibDebugPpi/DebugLib.c

  Log Message:
  -----------
  MdeModulePkg/PeiDebugLibDebugPpi: Fix comma parsing in VaListToBaseList

The VaListToBaseList function in PeiDebugLibDebugPpi/DebugLib.c does not
recognize the comma (`,`) flag in printf-style format strings. When a
driver uses format strings with thousand separators like "%,ld" to format
large numbers (e.g., 1234567 as "1,234,567"), the VaListToBaseList parser
fails to recognize the comma flag and causes incorrect argument extraction
from the VA_LIST, leading to unexpected or corrupted debug output.

The change is to add a check for the comma character in the format string
parsing logic to align with BasePrintLib's format parsing logic,
ensuring consistent behavior across all print library implementations.

Signed-off-by: Ray Ni <[email protected]>


  Commit: 4d0946c082dfaf1556a2ecb0fd496d27acb8cecd
      
https://github.com/tianocore/edk2/commit/4d0946c082dfaf1556a2ecb0fd496d27acb8cecd
  Author: Ray Ni <[email protected]>
  Date:   2026-02-27 (Fri, 27 Feb 2026)

  Changed paths:
    M MdeModulePkg/Library/PeiDxeDebugLibReportStatusCode/DebugLib.c

  Log Message:
  -----------
  MdeModulePkg/PeiDxeDebugLibReportStatusCode: Fix comma parsing

The DebugPrintMarker function in PeiDxeDebugLibReportStatusCode/DebugLib.c
does not recognize the comma (`,`) flag in printf-style format strings.
When a driver uses format strings with thousand separators like "%,ld" to
format large numbers (e.g., 1234567 as "1,234,567"), the DebugPrintMarker
parser fails to recognize the comma flag and causes incorrect argument
extraction when building the BASE_LIST for status code reporting, leading
to unexpected or corrupted debug output in status code reports.

The change is to add the missing comma flag check to align with
BasePrintLib's format parsing logic, ensuring consistent behavior across
all print library implementations.

Signed-off-by: Ray Ni <[email protected]>


Compare: https://github.com/tianocore/edk2/compare/67bc9959ae0a...4d0946c082df

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