From f3fa35a00233b6f2e7653b3b8c3e2b28b8ecbe7f Mon Sep 17 00:00:00 2001
From: Thomas Huth <thuth@redhat.com>
Date: Thu, 19 Jan 2017 17:37:31 +0800
Subject: [PATCH] NetworkPkg: Remove superfluous return statement.

If the code eventually returns "Status" anyway, it does not make
sense to explicitely return "Status" in case of an error, too.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Thomas Huth <thuth@redhat.com>
Reviewed-by: Wu Jiaxin <jiaxin.wu@intel.com>
---
 NetworkPkg/HttpDxe/HttpsSupport.c | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/NetworkPkg/HttpDxe/HttpsSupport.c b/NetworkPkg/HttpDxe/HttpsSupport.c
index 77e5371242..f0077dd4b8 100644
--- a/NetworkPkg/HttpDxe/HttpsSupport.c
+++ b/NetworkPkg/HttpDxe/HttpsSupport.c
@@ -1294,10 +1294,6 @@ TlsCloseSession (
   FreePool (BufferOut);
   NetbufFree (PacketOut);
 
-  if (EFI_ERROR (Status)) {
-    return Status;
-  }
-
   return Status;
 }
 
-- 
2.39.2