From nobody Mon Sep 16 20:41:29 2024 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; spf=none (zoho.com: 198.145.21.10 is neither permitted nor denied by domain of lists.01.org) smtp.mailfrom=edk2-devel-bounces@lists.01.org; dmarc=fail(p=none dis=none) header.from=intel.com Return-Path: Received: from ml01.01.org (ml01.01.org [198.145.21.10]) by mx.zohomail.com with SMTPS id 1534383538079888.2774686364398; Wed, 15 Aug 2018 18:38:58 -0700 (PDT) Received: from [127.0.0.1] (localhost [IPv6:::1]) by ml01.01.org (Postfix) with ESMTP id E50DB210F2D06; Wed, 15 Aug 2018 18:38:56 -0700 (PDT) Received: from mga12.intel.com (mga12.intel.com [192.55.52.136]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 6501F210D83EF for ; Wed, 15 Aug 2018 18:38:55 -0700 (PDT) Received: from fmsmga005.fm.intel.com ([10.253.24.32]) by fmsmga106.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 15 Aug 2018 18:38:55 -0700 Received: from songpeng.ccr.corp.intel.com ([10.239.158.47]) by fmsmga005.fm.intel.com with ESMTP; 15 Aug 2018 18:38:53 -0700 X-Original-To: edk2-devel@lists.01.org Received-SPF: none (zoho.com: 198.145.21.10 is neither permitted nor denied by domain of lists.01.org) client-ip=198.145.21.10; envelope-from=edk2-devel-bounces@lists.01.org; helo=ml01.01.org; Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=192.55.52.136; helo=mga12.intel.com; envelope-from=songpeng.li@intel.com; receiver=edk2-devel@lists.01.org X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.53,245,1531810800"; d="scan'208";a="254456802" From: Songpeng Li To: edk2-devel@lists.01.org Date: Thu, 16 Aug 2018 09:37:54 +0800 Message-Id: <20180816013757.9680-3-songpeng.li@intel.com> X-Mailer: git-send-email 2.18.0.windows.1 In-Reply-To: <20180816013757.9680-1-songpeng.li@intel.com> References: <20180816013757.9680-1-songpeng.li@intel.com> Subject: [edk2] [PATCH 2/5] NetworkPkg: IpSecDxe: Remove the redundant code. X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.27 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Siyuan Fu , Jiaxin Wu , Laszlo Ersek MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Errors-To: edk2-devel-bounces@lists.01.org Sender: "edk2-devel" X-ZohoMail: RDMRC_1 RSF_4 Z_629925259 SPT_0 Content-Type: text/plain; charset="utf-8" The functions that are never called have been removed. They are Ikev2OnPacketAccepted, Ikev2SaSessionOnDeleting, Ikev2ChildSaSessionLookupByMid, Ikev2ChildSaSessionOnDeleting and Ikev2ChildExchangeRekeySpi. Cc: Jiaxin Wu Cc: Siyuan Fu Cc: Laszlo Ersek Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3D1064 Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Songpeng Li --- NetworkPkg/IpSecDxe/Ikev2/Payload.c | 18 ------- NetworkPkg/IpSecDxe/Ikev2/Utility.c | 76 ----------------------------- NetworkPkg/IpSecDxe/Ikev2/Utility.h | 67 ------------------------- 3 files changed, 161 deletions(-) diff --git a/NetworkPkg/IpSecDxe/Ikev2/Payload.c b/NetworkPkg/IpSecDxe/Ikev= 2/Payload.c index 218c26f934..1bb5e2e5e5 100644 --- a/NetworkPkg/IpSecDxe/Ikev2/Payload.c +++ b/NetworkPkg/IpSecDxe/Ikev2/Payload.c @@ -3104,24 +3104,6 @@ ON_EXIT: return Status; } =20 -/** - Save some useful payloads after accepting the Packet. - - @param[in] SessionCommon Pointer to IKEV2_SESSION_COMMON related to th= e operation. - @param[in] IkePacket Pointer to received IkePacet. - @param[in] IkeType The type used to indicate it is in IkeSa or C= hildSa or Info - exchange. - -**/ -VOID -Ikev2OnPacketAccepted ( - IN IKEV2_SESSION_COMMON *SessionCommon, - IN IKE_PACKET *IkePacket, - IN UINT8 IkeType - ) -{ - return; -} =20 /** =20 diff --git a/NetworkPkg/IpSecDxe/Ikev2/Utility.c b/NetworkPkg/IpSecDxe/Ikev= 2/Utility.c index 698aba1327..0c9c929705 100644 --- a/NetworkPkg/IpSecDxe/Ikev2/Utility.c +++ b/NetworkPkg/IpSecDxe/Ikev2/Utility.c @@ -290,21 +290,6 @@ Ikev2SaSessionRemove ( return NULL; } =20 -/** - Marking a SA session as on deleting. - - @param[in] IkeSaSession Pointer to IKEV2_SA_SESSION. - - @retval EFI_SUCCESS Find the related SA session and marked it. - -**/ -EFI_STATUS -Ikev2SaSessionOnDeleting ( - IN IKEV2_SA_SESSION *IkeSaSession - ) -{ - return EFI_SUCCESS; -} =20 /** Free specified Seession Common. The session common would belong to a IKE= SA or @@ -659,33 +644,6 @@ Ikev2ChildSaSessionReg ( return ; } =20 -/** - Find the ChildSaSession by it's MessagId. - - @param[in] SaSessionList Pointer to a ChildSaSession List. - @param[in] Mid The messageId used to search ChildSaSession. - - @return Pointer to IKEV2_CHILD_SA_SESSION or NULL. - -**/ -IKEV2_CHILD_SA_SESSION * -Ikev2ChildSaSessionLookupByMid ( - IN LIST_ENTRY *SaSessionList, - IN UINT32 Mid - ) -{ - LIST_ENTRY *Entry; - IKEV2_CHILD_SA_SESSION *ChildSaSession; - - NET_LIST_FOR_EACH (Entry, SaSessionList) { - ChildSaSession =3D IKEV2_CHILD_SA_SESSION_BY_IKE_SA (Entry); - - if (ChildSaSession->MessageId =3D=3D Mid) { - return ChildSaSession; - } - } - return NULL; -} =20 /** This function find the Child SA by the specified SPI. @@ -776,22 +734,6 @@ Ikev2ChildSaSessionRemove ( return NULL; } =20 -/** - Mark a specified Child SA Session as on deleting. - - @param[in] ChildSaSession Pointer to IKEV2_CHILD_SA_SESSION. - - @retval EFI_SUCCESS Operation is successful. - -**/ -EFI_STATUS -Ikev2ChildSaSessionOnDeleting ( - IN IKEV2_CHILD_SA_SESSION *ChildSaSession - ) -{ - return EFI_SUCCESS; -} - /** Free the memory located for the specified IKEV2_CHILD_SA_SESSION. =20 @@ -1137,24 +1079,6 @@ Ikev2ChildSaAssociateSpdEntry ( } =20 =20 -/** - This function finds the SPI from Create Child SA Exchange Packet. - - @param[in] IkePacket Pointer to IKE_PACKET to be searched. - - @retval SPI number or 0 if it is not supported. - -**/ -UINT32 -Ikev2ChildExchangeRekeySpi ( - IN IKE_PACKET *IkePacket - ) -{ - // - // Not support yet. - // - return 0; -} =20 /** Validate the IKE header of received IKE packet. diff --git a/NetworkPkg/IpSecDxe/Ikev2/Utility.h b/NetworkPkg/IpSecDxe/Ikev= 2/Utility.h index 8383ff8dd6..79def7740d 100644 --- a/NetworkPkg/IpSecDxe/Ikev2/Utility.h +++ b/NetworkPkg/IpSecDxe/Ikev2/Utility.h @@ -104,19 +104,6 @@ Ikev2SaSessionRemove ( ); =20 =20 -/** - Marking a SA session as on deleting. - - @param[in] IkeSaSession Pointer to IKEV2_SA_SESSION. - - @retval EFI_SUCCESS Find the related SA session and marked it. - -**/ -EFI_STATUS -Ikev2SaSessionOnDeleting ( - IN IKEV2_SA_SESSION *IkeSaSession - ); - /** After IKE/Child SA is estiblished, close the time event and free sent pa= cket. =20 @@ -211,20 +198,6 @@ Ikev2ChildSaSessionLookupBySpi ( IN UINT32 Spi ); =20 -/** - Find the ChildSaSession by it's MessagId. - - @param[in] SaSessionList Pointer to a ChildSaSession List. - @param[in] Mid The messageId used to search ChildSaSession. - - @return Pointer to IKEV2_CHILD_SA_SESSION. - -**/ -IKEV2_CHILD_SA_SESSION * -Ikev2ChildSaSessionLookupByMid ( - IN LIST_ENTRY *SaSessionList, - IN UINT32 Mid - ); =20 /** Insert a Child SA Session into the specified ChildSa list.. @@ -257,18 +230,6 @@ Ikev2ChildSaSessionRemove ( IN UINT8 ListType ); =20 -/** - Mark a specified Child SA Session as on deleting. - - @param[in] ChildSaSession Pointer to IKEV2_CHILD_SA_SESSION. - - @retval EFI_SUCCESS Operation is successful. - -**/ -EFI_STATUS -Ikev2ChildSaSessionOnDeleting ( - IN IKEV2_CHILD_SA_SESSION *ChildSaSession - ); =20 /** Free the memory located for the specified IKEV2_CHILD_SA_SESSION. @@ -328,19 +289,6 @@ Ikev2ChildExchangeRequestType( IN IKE_PACKET *IkePacket ); =20 -/** - This function finds the SPI from Create Child Sa Exchange Packet. - - @param[in] IkePacket Pointer to IKE_PACKET to be searched. - - @retval SPI number. - -**/ -UINT32 -Ikev2ChildExchangeRekeySpi( - IN IKE_PACKET *IkePacket - ); - =20 /** Associate a SPD selector to the Child SA Session. @@ -882,21 +830,6 @@ Ikev2DecodePacket ( IN UINTN IkeType ); =20 -/** - Save some useful payloads after accepting the Packet. - - @param[in] SessionCommon Pointer to IKEV2_SESSION_COMMON related to th= e operation. - @param[in] IkePacket Pointer to received IkePacet. - @param[in] IkeType The type used to indicate it is in IkeSa or C= hildSa or Info - exchange. - -**/ -VOID -Ikev2OnPacketAccepted ( - IN IKEV2_SESSION_COMMON *SessionCommon, - IN IKE_PACKET *IkePacket, - IN UINT8 IkeType - ); =20 /** Send out IKEV2 packet. --=20 2.18.0.windows.1 _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel