MdeModulePkg/TerminalDxe: Remove unnecessary NULL pointer check

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com>
Cc: Star Zeng <star.zeng@intel.com>
Reviewed-by: Feng Tian <feng.tian@intel.com>

1 files changed, 2 insertions, 7 deletions

diff --git a/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c b/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
index b8dcf0c..604bda7 100644
--- a/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
+++ b/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
@@ -1356,19 +1356,14 @@ TerminalDriverBindingStop (
               );

       } else {

 

-        if (TerminalDevice->ControllerNameTable != NULL) {

-          FreeUnicodeStringTable (TerminalDevice->ControllerNameTable);

-        }

-

+        FreeUnicodeStringTable (TerminalDevice->ControllerNameTable);

         StopTerminalStateMachine (TerminalDevice);

         gBS->CloseEvent (TerminalDevice->SimpleInput.WaitForKey);

         gBS->CloseEvent (TerminalDevice->SimpleInputEx.WaitForKeyEx);

         gBS->CloseEvent (TerminalDevice->KeyNotifyProcessEvent);

         TerminalFreeNotifyList (&TerminalDevice->NotifyList);

         FreePool (TerminalDevice->DevicePath);

-        if (TerminalDevice->TerminalConsoleModeData != NULL) {

-          FreePool (TerminalDevice->TerminalConsoleModeData);

-        }

+        FreePool (TerminalDevice->TerminalConsoleModeData);

         FreePool (TerminalDevice);

       }

     }