Return the error status from gBS->OpenProtocol() since there are cases when the input ControllerHandle or ChildHandle is not a valid EFI_HANDLE. We should return EFI_INVALID_PARAMETER instead of EFI_UNSUPPORTED for Driver Diagnostic (2) protocol implementation.

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

1 files changed, 2 insertions, 2 deletions

diff --git a/IntelFrameworkModulePkg/Bus/Pci/IdeBusDxe/DriverDiagnostics.c b/IntelFrameworkModulePkg/Bus/Pci/IdeBusDxe/DriverDiagnostics.c
index 1502268c20..f9f0b2252f 100644
--- a/IntelFrameworkModulePkg/Bus/Pci/IdeBusDxe/DriverDiagnostics.c
+++ b/IntelFrameworkModulePkg/Bus/Pci/IdeBusDxe/DriverDiagnostics.c
@@ -163,7 +163,7 @@ IDEBusDriverDiagnosticsRunDiagnostics (
                     EFI_OPEN_PROTOCOL_TEST_PROTOCOL

                     );

     if (EFI_ERROR (Status)) {

-      return EFI_UNSUPPORTED;

+      return Status;

     }

 

     Status = gBS->OpenProtocol (

@@ -202,7 +202,7 @@ IDEBusDriverDiagnosticsRunDiagnostics (
                   EFI_OPEN_PROTOCOL_GET_PROTOCOL

                   );

   if (EFI_ERROR (Status)) {

-    return EFI_UNSUPPORTED;

+    return Status;

   }

 

   IdeBlkIoDevice = IDE_BLOCK_IO_DEV_FROM_THIS (BlkIo);