I fixed following bugs.

1. In dependency.c, DXE Dispatcher doesn't check the dependency expression boundary correctly.
2. In page.c, a misspelling code in DXE core memory service code, duplicate code.
3. In gcd.c, according to DXE CIS 0-91 spec, When GcdAllocateType is EfiGcdAllocateMaxAddressSearchBottomUp or EfiGcdAllocateMaxAddressSearchTopDown, then the GCD memory space map is searched from the lowest address up to BaseAddress (or from BaseAddress down to the lowest address) looking for unallocated memory ranges of Length bytes. Here, BaseAddress itself is inclusive, that is, any range in [0, BaseAddress] is allowable. But current code BaseAddress to be exclusive.
4. In Ebcinit.c, EbcDebugRegisterPeriodicCallback() not check whether it is already registered.
5. In Ebcinit.c, EbcDebugRegisterExceptionCallback() not check whether it is already registered.
6. In Ebcinit.c, EbcDebugSignalException() does not assign value to EbcContext.ControlFlags.
7. In Ebcinit.c, EBC: StatusCodeValue should use EFI_SOFTWARE_EBC_EXCEPTION.
8. In Ebcinit.c, EbcDebugRegisterExceptionCallback() does not check ExceptionType
9. In ElTorito.c, BlockIo and DiskIo failed in SCT when there is CD in CD-ROM.
10. In Mbr.c, MBR code can't handle some situations. That is when there are 3 partitions in HD, if delete the first partition, it will not be able to recognize the other two partitions, and if delete the second partition, it will not be able to recognize the third partition. 


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

1 files changed, 37 insertions, 19 deletions

diff --git a/EdkModulePkg/Universal/Ebc/Dxe/EbcInt.c b/EdkModulePkg/Universal/Ebc/Dxe/EbcInt.c
index 7b15a1a..2a5d48a 100644
--- a/EdkModulePkg/Universal/Ebc/Dxe/EbcInt.c
+++ b/EdkModulePkg/Universal/Ebc/Dxe/EbcInt.c
@@ -154,8 +154,8 @@ static EBC_ICACHE_FLUSH       mEbcICacheFlush;
 //

 // These get set via calls by the debug agent

 //

-static EFI_PERIODIC_CALLBACK  mDebugPeriodicCallback    = NULL;

-static EFI_EXCEPTION_CALLBACK mDebugExceptionCallback   = NULL;

+static EFI_PERIODIC_CALLBACK  mDebugPeriodicCallback                            = NULL;

+static EFI_EXCEPTION_CALLBACK mDebugExceptionCallback[MAX_EBC_EXCEPTION + 1] = {NULL};

 static EFI_GUID               mEfiEbcVmTestProtocolGuid = EFI_EBC_VM_TEST_PROTOCOL_GUID;

 

 EFI_STATUS

@@ -405,6 +405,13 @@ Returns:
 

 --*/

 {

+  if ((mDebugPeriodicCallback == NULL) && (PeriodicCallback == NULL)) {

+    return EFI_INVALID_PARAMETER;

+  }

+  if ((mDebugPeriodicCallback != NULL) && (PeriodicCallback != NULL)) {

+    return EFI_ALREADY_STARTED;

+  }

+	

   mDebugPeriodicCallback = PeriodicCallback;

   return EFI_SUCCESS;

 }

@@ -437,7 +444,16 @@ Returns:
 

 --*/

 {

-  mDebugExceptionCallback = ExceptionCallback;

+  if ((ExceptionType < 0) || (ExceptionType > MAX_EBC_EXCEPTION)) {

+    return EFI_INVALID_PARAMETER;

+  }

+  if ((mDebugExceptionCallback[ExceptionType] == NULL) && (ExceptionCallback == NULL)) {

+    return EFI_INVALID_PARAMETER;

+  }

+  if ((mDebugExceptionCallback[ExceptionType] != NULL) && (ExceptionCallback != NULL)) {

+    return EFI_ALREADY_STARTED;

+  }

+  mDebugExceptionCallback[ExceptionType] = ExceptionCallback;

   return EFI_SUCCESS;

 }

 

@@ -519,13 +535,15 @@ Returns:
   EbcContext.R7                   = VmPtr->R[7];

   EbcContext.Ip                   = (UINT64) (UINTN) VmPtr->Ip;

   EbcContext.Flags                = VmPtr->Flags;

+  EbcContext.ControlFlags         = 0;

   SystemContext.SystemContextEbc  = &EbcContext;

   //

   // If someone's registered for exception callbacks, then call them.

   // Otherwise report the status code via the status code API

   //

-  if (mDebugExceptionCallback != NULL) {

-    mDebugExceptionCallback (ExceptionType, SystemContext);

+  if ((ExceptionType >= 0) && (ExceptionType <= MAX_EBC_EXCEPTION) &&

+      (mDebugExceptionCallback[ExceptionType] != NULL)) {

+    mDebugExceptionCallback[ExceptionType] (ExceptionType, SystemContext);

   }

   //

   // Determine if we should report the exception. We report all of them by default,

@@ -536,50 +554,50 @@ Returns:
   Report = TRUE;

   switch (ExceptionType) {

   case EXCEPT_EBC_UNDEFINED:

-    StatusCodeValue = EFI_SOFTWARE_DXE_BS_DRIVER | EFI_SW_EC_EBC_UNDEFINED;

+    StatusCodeValue = EFI_SOFTWARE_EBC_EXCEPTION | EFI_SW_EC_EBC_UNDEFINED;

     break;

 

   case EXCEPT_EBC_DIVIDE_ERROR:

-    StatusCodeValue = EFI_SOFTWARE_DXE_BS_DRIVER | EFI_SW_EC_EBC_DIVIDE_ERROR;

+    StatusCodeValue = EFI_SOFTWARE_EBC_EXCEPTION | EFI_SW_EC_EBC_DIVIDE_ERROR;

     break;

 

   case EXCEPT_EBC_DEBUG:

-    StatusCodeValue = EFI_SOFTWARE_DXE_BS_DRIVER | EFI_SW_EC_EBC_DEBUG;

-    Report          = (BOOLEAN) ((mDebugExceptionCallback == NULL) ? TRUE : FALSE);

+    StatusCodeValue = EFI_SOFTWARE_EBC_EXCEPTION | EFI_SW_EC_EBC_DEBUG;

+    Report          = (BOOLEAN) ((mDebugExceptionCallback[ExceptionType] == NULL) ? TRUE : FALSE);

     break;

 

   case EXCEPT_EBC_BREAKPOINT:

-    StatusCodeValue = EFI_SOFTWARE_DXE_BS_DRIVER | EFI_SW_EC_EBC_BREAKPOINT;

-    Report          = (BOOLEAN) ((mDebugExceptionCallback == NULL) ? TRUE : FALSE);

+    StatusCodeValue = EFI_SOFTWARE_EBC_EXCEPTION | EFI_SW_EC_EBC_BREAKPOINT;

+    Report          = (BOOLEAN) ((mDebugExceptionCallback[ExceptionType] == NULL) ? TRUE : FALSE);

     break;

 

   case EXCEPT_EBC_INVALID_OPCODE:

-    StatusCodeValue = EFI_SOFTWARE_DXE_BS_DRIVER | EFI_SW_EC_EBC_INVALID_OPCODE;

+    StatusCodeValue = EFI_SOFTWARE_EBC_EXCEPTION | EFI_SW_EC_EBC_INVALID_OPCODE;

     break;

 

   case EXCEPT_EBC_STACK_FAULT:

-    StatusCodeValue = EFI_SOFTWARE_DXE_BS_DRIVER | EFI_SW_EC_EBC_STACK_FAULT;

+    StatusCodeValue = EFI_SOFTWARE_EBC_EXCEPTION | EFI_SW_EC_EBC_STACK_FAULT;

     break;

 

   case EXCEPT_EBC_ALIGNMENT_CHECK:

-    StatusCodeValue = EFI_SOFTWARE_DXE_BS_DRIVER | EFI_SW_EC_EBC_ALIGNMENT_CHECK;

+    StatusCodeValue = EFI_SOFTWARE_EBC_EXCEPTION | EFI_SW_EC_EBC_ALIGNMENT_CHECK;

     break;

 

   case EXCEPT_EBC_INSTRUCTION_ENCODING:

-    StatusCodeValue = EFI_SOFTWARE_DXE_BS_DRIVER | EFI_SW_EC_EBC_INSTRUCTION_ENCODING;

+    StatusCodeValue = EFI_SOFTWARE_EBC_EXCEPTION | EFI_SW_EC_EBC_INSTRUCTION_ENCODING;

     break;

 

   case EXCEPT_EBC_BAD_BREAK:

-    StatusCodeValue = EFI_SOFTWARE_DXE_BS_DRIVER | EFI_SW_EC_EBC_BAD_BREAK;

+    StatusCodeValue = EFI_SOFTWARE_EBC_EXCEPTION | EFI_SW_EC_EBC_BAD_BREAK;

     break;

 

   case EXCEPT_EBC_STEP:

-    StatusCodeValue = EFI_SOFTWARE_DXE_BS_DRIVER | EFI_SW_EC_EBC_STEP;

-    Report          = (BOOLEAN) ((mDebugExceptionCallback == NULL) ? TRUE : FALSE);

+    StatusCodeValue = EFI_SOFTWARE_EBC_EXCEPTION | EFI_SW_EC_EBC_STEP;

+    Report          = (BOOLEAN) ((mDebugExceptionCallback[ExceptionType] == NULL) ? TRUE : FALSE);

     break;

 

   default:

-    StatusCodeValue = EFI_SOFTWARE_DXE_BS_DRIVER | EFI_SW_EC_NON_SPECIFIC;

+    StatusCodeValue = EFI_SOFTWARE_EBC_EXCEPTION | EFI_SW_EC_NON_SPECIFIC;

     break;

   }

   //