02/02: [FREELDR] PcMemGetBiosMemoryMap(): Add a default handling of entry with "EA.ErrorLog = 1". CORE-14150

[Serge Gautherie]

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=c1d0d409e7c2dfea17f83…

commit c1d0d409e7c2dfea17f83272c1332936967dffa5
Author:     Serge Gautherie <reactos-git_serge_171003(a)gautherie.fr>
AuthorDate: Fri Jan 12 06:30:53 2018 +0100
Commit:     Timo Kreuzer <timo.kreuzer(a)reactos.org>
CommitDate: Sun Jul 1 12:03:50 2018 +0200

    [FREELDR] PcMemGetBiosMemoryMap(): Add a default handling of entry with
"EA.ErrorLog = 1". CORE-14150
    
    This FIXME is meant as a reminder/help to actually use this data.
---
 boot/freeldr/freeldr/arch/i386/pcmem.c | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/boot/freeldr/freeldr/arch/i386/pcmem.c
b/boot/freeldr/freeldr/arch/i386/pcmem.c
index b6024ff73a..28d6e06e4c 100644
--- a/boot/freeldr/freeldr/arch/i386/pcmem.c
+++ b/boot/freeldr/freeldr/arch/i386/pcmem.c
@@ -351,6 +351,18 @@ PcMemGetBiosMemoryMap(PFREELDR_MEMORY_DESCRIPTOR MemoryMap, ULONG
MaxMemoryMapSi
         TRACE("Type: 0x%lx\n", PcBiosMemoryMap[PcBiosMapCount].Type);
         TRACE("ExtendedAttributesAsULONG: 0x%08lx\n",
PcBiosMemoryMap[PcBiosMapCount].ExtendedAttributesAsULONG);
 
+        if (PcBiosMemoryMap[PcBiosMapCount].ExtendedAttributes.ErrorLog == 1)
+        {
+            FIXME("EA.ErrorLog = 1. Please report this to CORE-14150. "
+                   "(PcBiosMapCount = %lu, BaseAddress = 0x%llx, Length = 0x%llx,
Type = 0x%lx, ExtendedAttributesAsULONG = 0x%08lx)\n",
+                  PcBiosMapCount,
+                  PcBiosMemoryMap[PcBiosMapCount].BaseAddress,
+                  PcBiosMemoryMap[PcBiosMapCount].Length,
+                  PcBiosMemoryMap[PcBiosMapCount].Type,
+                  PcBiosMemoryMap[PcBiosMapCount].ExtendedAttributesAsULONG);
+            // NotWantedForPublicBuilds: ASSERTMSG("EA.ErrorLog = 1. Check/Report
then CONTinue.", FALSE);
+        }
+
         if (PcBiosMemoryMap[PcBiosMapCount].Length == 0)
         {
             TRACE("Discard empty range. (would-be-PcBiosMapCount = %lu, BaseAddress
= 0x%llx, Length = 0)\n",