[edk2] FW: [PATCH] UefiCpuPkg/SecMain: Add NORETURN decorator to SecStartup().

2018-05-05 Thread Marvin Häuser
Hey Eric and Laszlo, If I am not mistaken, this patch has neither been reviewed by Eric, nor was it pushed. Just forwarding it in case it was forgotten about. Thank you very much! Best regards, Marvin. > -Ursprüngliche Nachricht- > Von: Laszlo Ersek > Gesendet:

[edk2] [PATCH v2 1/2] MdePkg/UefiLib: Add 'OUT' decorator where necessary.

2018-05-05 Thread Marvin Häuser
The functions AddUnicodeString() and AddUnicodeString2() might return a new value into their parameter UnicodeStringTable, hence add the appropiate 'OUT' decorator. V2: Update FrameworkUefiLib as well. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Marvin Haeuser

[edk2] [PATCH] MdePkg/PeiServicesLib: Decorate 'PpiDescriptor' as OPTIONAL for LocatePpi().

2018-05-05 Thread Marvin Häuser
The UEFI PI specification defines PpiDescriptor to be OPTIONAL for the LocatePpi PEI Service. This patch reflects this in the function declaration and definition of the corresponding PeiServices library function. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Marvin

[edk2] [PATCH v2 2/2] IntelFrameworkPkg/FrameworkUefiLib: Add 'OUT' decorator where necessary.

2018-05-05 Thread Marvin Häuser
The functions AddUnicodeString() and AddUnicodeString2() might return a new value into their parameter UnicodeStringTable, hence add the appropiate 'OUT' decorator. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Marvin Haeuser ---

[edk2] [PATCH] ShellPkg/UefiShellLevel2CommandsLib: Support unspecified local Timezone.

2018-05-05 Thread Marvin Häuser
EFI_RUNTIME_SERVICES.GetTime() might return an unspecified Timezone, such as when SetTime() has not been called after the PC-AT CMOS RTC was cut off power. Consider this case by not attempting Timezone translations for when it is invalid. Contributed-under: TianoCore Contribution Agreement 1.1

[edk2] [PATCH] MdePkg/UefiFileHandleLib: Fix potential NULL dereference.

2018-05-05 Thread Marvin Häuser
Move the NULL-check in FileHandleGetInfo() to directly after the allocation to prevent potential NULL dereferences. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Marvin Haeuser --- MdePkg/Library/UefiFileHandleLib/UefiFileHandleLib.c | 30

[edk2] [PATCH] ShellPkg/UefiShellCommandLib: Do not error-exit when PlatformLang is missing.

2018-05-05 Thread Marvin Häuser
Currently, when the PlatformLang variable is missing, UefiShellCommandLib error-exits. To prevent the entire UEFI Shell from failing to load over a missing variable, "en-US" is assumed when the variable location fails. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Marvin

[edk2] [PATCH] MdePkg/UefiFileHandleLib: Fix the Root directory determination.

2018-05-05 Thread Marvin Häuser
The current implementation of the FileHandleGetFileName() function assumes that the Root directory always has the FileName '\0'. However, the only requirement the UEFI specification defines is that a prepended '\\' must be supported to access files and folders relative to the Root directory. This

[edk2] [PATCH] ShellPkg/UefiShellLevel2CommandsLib: Fix ls on entirely empty directories.

2018-05-05 Thread Marvin Häuser
Currently, when 'ls' is run on an entirely empty directory (this includes not having '.' and '..'), the output is always 'File not found'. For when not filtering its children, this patch rather displays the usual header and footer. Contributed-under: TianoCore Contribution Agreement 1.1

[edk2] [PATCH] MdePkg/BaseLib: Globally include ChkStk.c

2018-05-05 Thread Marvin Häuser
Initially added for GCC build support, this patch includes the function for all compilers and all architectures. This is done as huge variables on the stack may cause the generation of calls to this intrinsic function for Microsoft compilers, even for the IA32 architecture, too.

[edk2] [PATCH] MdeModulePkg/AcpiPlatformDxe: Unload after execution.

2018-05-05 Thread Marvin Häuser
AcpiPlatformDxe solely performs one-time tasks and does not expose any services or create any events. Hence it can safely be unloaded after the Entry Point execution by returning an error code. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Marvin Haeuser