Dear sirs,
Next report. File
edk2/MdeModulePkg/Core/Dxe/SectionExtraction/CoreSectionExtraction.c:
line 1007. Resource leak: CurrentChildNode
-
Status = CreateChildNode (SourceStream, 0, &CurrentChildNode);
if (EFI_ERROR (Status)) {
return Status;
}
-
because the proc
Sergey,
Thanks for you to report this issue, we will follow up to clean it.
Thanks,
Eric
From: Sergey Isakov [mailto:isakov...@bk.ru]
Sent: Tuesday, July 15, 2014 9:26 PM
To: edk2-devel@lists.sourceforge.net
Subject: [edk2] Resource leak in BdsBoot.c
Dear sirs,
File edk2/IntelFrameworkModulePkg
Dear sirs,
File edk2/OptionRomPkg/CirrusLogic5430Dxe/Edid.c line:143
———
*EdidDataBlock = AllocateCopyPool (
sizeof (EDID_BLOCK_SIZE),
ValidEdid
);
——
There must be EDID_BLOCK_SIZE not sizeof().
Summary: Suspicious usage of 'sizeof
Mars,
If the actual memory usage when the OS is booted is less than the peak memory
usage, then you could imagine a potential optimization from the current design.
However, if there is a driver that allocates a large buffer for a short period
of time that exceeds the current bin size, you can
The one in the shell package is for being built into the shell as a native
command. The other is a standalone application.
-Erik
-Original Message-
From: Blibbet [mailto:blib...@gmail.com]
Sent: Wednesday, July 09, 2014 9:14 PM
To: edk2-devel@lists.sourceforge.net
Subject: [edk2] Multi
Dear sirs,
File edk2/IntelFrameworkModulePkg/Library/GenericBdsLib/BdsBoot.c
Procedure BdsRefreshBbsTableForBoot()
Local array DeviceType is AllocatePooled at line 1587 but never FreePooled.
Sergey--
Want fast and easy ac
Mike,
Thanks for your comment. I have additional comment for current design.
I know there is a requirement for UEFI S4 resume that need to have consistent
memory map and OS using gBS->GetMemoryMap() for such check.
CoreGetMemoryMap returned UEFI memory map based on the information conveyed via
g
