1. In event.c, a function's name is not spelled correctly, a typo.

2. In Graphics.c, Memory Leak in Graphics Library, ConvertBmpToUgaBlt().
3. In HobLib.c, PeiBuildHobModule and three other functions do not zero hob reserved data area according to HOB spec v0.9.
4. In statuscode.h, the number of definition of EFI_SW_PEIM_PC_CAPSULE_START is equal to EFI_SW_PEIM_PC_CAPSULE_LOAD.


git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@1861 6f19259b-4bc3-4df7-8a09-765794883524

2 files changed, 10 insertions, 3 deletions

diff --git a/EdkModulePkg/Core/Dxe/Event/event.c b/EdkModulePkg/Core/Dxe/Event/event.c
index ae6b0a849d..ade810c5c1 100644
--- a/EdkModulePkg/Core/Dxe/Event/event.c
+++ b/EdkModulePkg/Core/Dxe/Event/event.c
@@ -363,7 +363,7 @@ Returns:
 static

 VOID

 EFIAPI

-EventNofitySignalAllNullEvent (

+EventNotifySignalAllNullEvent (

   IN EFI_EVENT                Event,

   IN VOID                     *Context

   )

@@ -426,7 +426,7 @@ Returns:
   //

   if (Type & EFI_EVENT_NOTIFY_SIGNAL_ALL) {

     Type &= ~EFI_EVENT_NOTIFY_SIGNAL_ALL;

-    Function = EventNofitySignalAllNullEvent;

+    Function = EventNotifySignalAllNullEvent;

   }

 

   //

diff --git a/EdkModulePkg/Library/EdkGraphicsLib/Graphics.c b/EdkModulePkg/Library/EdkGraphicsLib/Graphics.c
index 01c612abd3..52936e3c40 100644
--- a/EdkModulePkg/Library/EdkGraphicsLib/Graphics.c
+++ b/EdkModulePkg/Library/EdkGraphicsLib/Graphics.c
@@ -159,7 +159,8 @@ Returns:
   UINTN             Height;

   UINTN             Width;

   UINTN             ImageIndex;

-

+  BOOLEAN           IsAllocated;

+  

   BmpHeader = (BMP_IMAGE_HEADER *) BmpImage;

   if (BmpHeader->CharB != 'B' || BmpHeader->CharM != 'M') {

     return EFI_UNSUPPORTED;

@@ -182,12 +183,14 @@ Returns:
   ImageHeader   = Image;

 

   BltBufferSize = BmpHeader->PixelWidth * BmpHeader->PixelHeight * sizeof (EFI_UGA_PIXEL);

+  IsAllocated   = FALSE;

   if (*UgaBlt == NULL) {

     *UgaBltSize = BltBufferSize;

     *UgaBlt     = AllocatePool (*UgaBltSize);

     if (*UgaBlt == NULL) {

       return EFI_OUT_OF_RESOURCES;

     }

+    IsAllocated = TRUE;

   } else {

     if (*UgaBltSize < BltBufferSize) {

       *UgaBltSize = BltBufferSize;

@@ -256,6 +259,10 @@ Returns:
         break;

 

       default:

+        if (IsAllocated) {

+          gBS->FreePool (*UgaBlt);

+          *UgaBlt = NULL;

+        }

         return EFI_UNSUPPORTED;

         break;

       };