OvmfPkg: SataControllerDxe: SataControllerStop: remove useless null c…

…heck "ASSERT (SataPrivateData != NULL)" is just a few lines higher up. Cc: Jordan Justen <[email protected]> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Laszlo Ersek <[email protected]> Reviewed-by: Jordan Justen <[email protected]>
Wenxing-hou · Apr 26, 2016 · 81310a6 · 81310a6
1 parent dea0d6b
commit 81310a6
Showing 1 changed file with 9 additions and 11 deletions.
diff --git a/OvmfPkg/SataControllerDxe/SataController.c b/OvmfPkg/SataControllerDxe/SataController.c
@@ -602,18 +602,16 @@ SataControllerStop (
     return Status;
   }
 
-  if (SataPrivateData != NULL) {
-    if (SataPrivateData->DisqualifiedModes != NULL) {
-      FreePool (SataPrivateData->DisqualifiedModes);
-    }
-    if (SataPrivateData->IdentifyData != NULL) {
-      FreePool (SataPrivateData->IdentifyData);
-    }
-    if (SataPrivateData->IdentifyValid != NULL) {
-      FreePool (SataPrivateData->IdentifyValid);
-    }
-    FreePool (SataPrivateData);
+  if (SataPrivateData->DisqualifiedModes != NULL) {
+    FreePool (SataPrivateData->DisqualifiedModes);
   }
+  if (SataPrivateData->IdentifyData != NULL) {
+    FreePool (SataPrivateData->IdentifyData);
+  }
+  if (SataPrivateData->IdentifyValid != NULL) {
+    FreePool (SataPrivateData->IdentifyValid);
+  }
+  FreePool (SataPrivateData);
 
   //
   // Restore original PCI attributes