NetworkPkg/IScsiDxe/IScsiDxe.inf | 1 - NetworkPkg/IScsiDxe/IScsiProto.c | 33 ------------ NetworkPkg/IpSecDxe/Ikev2/Payload.c | 18 ------- NetworkPkg/IpSecDxe/Ikev2/Utility.c | 76 ---------------------------- NetworkPkg/IpSecDxe/Ikev2/Utility.h | 67 ------------------------ NetworkPkg/TcpDxe/TcpOption.c | 30 ----------- NetworkPkg/TcpDxe/TcpOption.h | 16 ------ NetworkPkg/TlsDxe/TlsDxe.inf | 1 - NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.c | 23 --------- NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.h | 10 ---- 10 files changed, 275 deletions(-)
Cc: Jiaxin Wu <jiaxin.wu@intel.com>
Cc: Siyuan Fu <siyuan.fu@intel.com>
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1064
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Songpeng Li <songpeng.li@intel.com>
---
NetworkPkg/IScsiDxe/IScsiDxe.inf | 1 -
NetworkPkg/IScsiDxe/IScsiProto.c | 33 ------------
NetworkPkg/IpSecDxe/Ikev2/Payload.c | 18 -------
NetworkPkg/IpSecDxe/Ikev2/Utility.c | 76 ----------------------------
NetworkPkg/IpSecDxe/Ikev2/Utility.h | 67 ------------------------
NetworkPkg/TcpDxe/TcpOption.c | 30 -----------
NetworkPkg/TcpDxe/TcpOption.h | 16 ------
NetworkPkg/TlsDxe/TlsDxe.inf | 1 -
NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.c | 23 ---------
NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.h | 10 ----
10 files changed, 275 deletions(-)
diff --git a/NetworkPkg/IScsiDxe/IScsiDxe.inf b/NetworkPkg/IScsiDxe/IScsiDxe.inf
index 2d96611b44..60737c96ab 100644
--- a/NetworkPkg/IScsiDxe/IScsiDxe.inf
+++ b/NetworkPkg/IScsiDxe/IScsiDxe.inf
@@ -124,7 +124,6 @@
gEfiIfrTianoGuid ## SOMETIMES_PRODUCES ## UNDEFINED
gEfiAcpiTableGuid ## SOMETIMES_CONSUMES ## SystemTable
gEfiAcpi10TableGuid ## SOMETIMES_CONSUMES ## SystemTable
- gEfiAcpi20TableGuid ## SOMETIMES_CONSUMES ## SystemTable
gEfiAdapterInfoNetworkBootGuid ## SOMETIMES_CONSUMES ## UNDEFINED
gEfiAdapterInfoUndiIpv6SupportGuid ## SOMETIMES_CONSUMES ## GUID
diff --git a/NetworkPkg/IScsiDxe/IScsiProto.c b/NetworkPkg/IScsiDxe/IScsiProto.c
index 7619360568..f4a49c677a 100644
--- a/NetworkPkg/IScsiDxe/IScsiProto.c
+++ b/NetworkPkg/IScsiDxe/IScsiProto.c
@@ -2096,39 +2096,6 @@ IScsiDelTcb (
}
-/**
- Find the task control block by the initator task tag.
-
- @param[in] TcbList The tcb list.
- @param[in] InitiatorTaskTag The initiator task tag.
-
- @return The task control block found.
- @retval NULL The task control block cannot be found.
-
-**/
-ISCSI_TCB *
-IScsiFindTcbByITT (
- IN LIST_ENTRY *TcbList,
- IN UINT32 InitiatorTaskTag
- )
-{
- ISCSI_TCB *Tcb;
- LIST_ENTRY *Entry;
-
- Tcb = NULL;
-
- NET_LIST_FOR_EACH (Entry, TcbList) {
- Tcb = NET_LIST_USER_STRUCT (Entry, ISCSI_TCB, Link);
-
- if (Tcb->InitiatorTaskTag == InitiatorTaskTag) {
- break;
- }
- }
-
- return Tcb;
-}
-
-
/**
Create a data segment, pad it, and calculate the CRC if needed.
diff --git a/NetworkPkg/IpSecDxe/Ikev2/Payload.c b/NetworkPkg/IpSecDxe/Ikev2/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;
}
-/**
- Save some useful payloads after accepting the Packet.
-
- @param[in] SessionCommon Pointer to IKEV2_SESSION_COMMON related to the operation.
- @param[in] IkePacket Pointer to received IkePacet.
- @param[in] IkeType The type used to indicate it is in IkeSa or ChildSa or Info
- exchange.
-
-**/
-VOID
-Ikev2OnPacketAccepted (
- IN IKEV2_SESSION_COMMON *SessionCommon,
- IN IKE_PACKET *IkePacket,
- IN UINT8 IkeType
- )
-{
- return;
-}
/**
diff --git a/NetworkPkg/IpSecDxe/Ikev2/Utility.c b/NetworkPkg/IpSecDxe/Ikev2/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;
}
-/**
- 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;
-}
/**
Free specified Seession Common. The session common would belong to a IKE SA or
@@ -659,33 +644,6 @@ Ikev2ChildSaSessionReg (
return ;
}
-/**
- 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 = IKEV2_CHILD_SA_SESSION_BY_IKE_SA (Entry);
-
- if (ChildSaSession->MessageId == Mid) {
- return ChildSaSession;
- }
- }
- return NULL;
-}
/**
This function find the Child SA by the specified SPI.
@@ -776,22 +734,6 @@ Ikev2ChildSaSessionRemove (
return NULL;
}
-/**
- 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.
@@ -1137,24 +1079,6 @@ Ikev2ChildSaAssociateSpdEntry (
}
-/**
- 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;
-}
/**
Validate the IKE header of received IKE packet.
diff --git a/NetworkPkg/IpSecDxe/Ikev2/Utility.h b/NetworkPkg/IpSecDxe/Ikev2/Utility.h
index 8383ff8dd6..79def7740d 100644
--- a/NetworkPkg/IpSecDxe/Ikev2/Utility.h
+++ b/NetworkPkg/IpSecDxe/Ikev2/Utility.h
@@ -104,19 +104,6 @@ Ikev2SaSessionRemove (
);
-/**
- 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 packet.
@@ -211,20 +198,6 @@ Ikev2ChildSaSessionLookupBySpi (
IN UINT32 Spi
);
-/**
- 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
- );
/**
Insert a Child SA Session into the specified ChildSa list..
@@ -257,18 +230,6 @@ Ikev2ChildSaSessionRemove (
IN UINT8 ListType
);
-/**
- 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
- );
/**
Free the memory located for the specified IKEV2_CHILD_SA_SESSION.
@@ -328,19 +289,6 @@ Ikev2ChildExchangeRequestType(
IN IKE_PACKET *IkePacket
);
-/**
- 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
- );
-
/**
Associate a SPD selector to the Child SA Session.
@@ -882,21 +830,6 @@ Ikev2DecodePacket (
IN UINTN IkeType
);
-/**
- Save some useful payloads after accepting the Packet.
-
- @param[in] SessionCommon Pointer to IKEV2_SESSION_COMMON related to the operation.
- @param[in] IkePacket Pointer to received IkePacet.
- @param[in] IkeType The type used to indicate it is in IkeSa or ChildSa or Info
- exchange.
-
-**/
-VOID
-Ikev2OnPacketAccepted (
- IN IKEV2_SESSION_COMMON *SessionCommon,
- IN IKE_PACKET *IkePacket,
- IN UINT8 IkeType
- );
/**
Send out IKEV2 packet.
diff --git a/NetworkPkg/TcpDxe/TcpOption.c b/NetworkPkg/TcpDxe/TcpOption.c
index bacce1070d..73d781729f 100644
--- a/NetworkPkg/TcpDxe/TcpOption.c
+++ b/NetworkPkg/TcpDxe/TcpOption.c
@@ -342,33 +342,3 @@ TcpParseOption (
return 0;
}
-
-/**
- Check the segment against PAWS.
-
- @param[in] Tcb Pointer to the TCP_CB of this TCP instance.
- @param[in] TSVal The timestamp value.
-
- @retval 1 The segment passed the PAWS check.
- @retval 0 The segment failed to pass the PAWS check.
-
-**/
-UINT32
-TcpPawsOK (
- IN TCP_CB *Tcb,
- IN UINT32 TSVal
- )
-{
- //
- // PAWS as defined in RFC1323, buggy...
- //
- if (TCP_TIME_LT (TSVal, Tcb->TsRecent) &&
- TCP_TIME_LT (Tcb->TsRecentAge + TCP_PAWS_24DAY, mTcpTick)
- ) {
-
- return 0;
-
- }
-
- return 1;
-}
diff --git a/NetworkPkg/TcpDxe/TcpOption.h b/NetworkPkg/TcpDxe/TcpOption.h
index 0ccadb9536..9d8581b669 100644
--- a/NetworkPkg/TcpDxe/TcpOption.h
+++ b/NetworkPkg/TcpDxe/TcpOption.h
@@ -126,20 +126,4 @@ TcpParseOption (
IN OUT TCP_OPTION *Option
);
-/**
- Check the segment against PAWS.
-
- @param[in] Tcb Pointer to the TCP_CB of this TCP instance.
- @param[in] TSVal The timestamp value.
-
- @retval 1 The segment passed the PAWS check.
- @retval 0 The segment failed to pass the PAWS check.
-
-**/
-UINT32
-TcpPawsOK (
- IN TCP_CB *Tcb,
- IN UINT32 TSVal
- );
-
#endif
diff --git a/NetworkPkg/TlsDxe/TlsDxe.inf b/NetworkPkg/TlsDxe/TlsDxe.inf
index 907feb735b..bbe8c554a2 100644
--- a/NetworkPkg/TlsDxe/TlsDxe.inf
+++ b/NetworkPkg/TlsDxe/TlsDxe.inf
@@ -52,7 +52,6 @@
BaseLib
UefiLib
DebugLib
- NetLib
BaseCryptLib
TlsLib
diff --git a/NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.c b/NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.c
index c06b0242bb..5bbc1922a6 100644
--- a/NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.c
+++ b/NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.c
@@ -202,29 +202,6 @@ PxeBcCacheDhcp6Packet (
return EFI_SUCCESS;
}
-
-/**
- Free all the nodes in the list for boot file.
-
- @param[in] Head The pointer to the head of list.
-
-**/
-VOID
-PxeBcFreeBootFileOption (
- IN LIST_ENTRY *Head
- )
-{
- LIST_ENTRY *Entry;
- LIST_ENTRY *NextEntry;
- PXEBC_DHCP6_OPTION_NODE *Node;
-
- NET_LIST_FOR_EACH_SAFE (Entry, NextEntry, Head) {
- Node = NET_LIST_USER_STRUCT (Entry, PXEBC_DHCP6_OPTION_NODE, Link);
- RemoveEntryList (Entry);
- FreePool (Node);
- }
-}
-
/**
Retrieve the boot server address using the EFI_DNS6_PROTOCOL.
diff --git a/NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.h b/NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.h
index f3dcd89743..d6e6745f0d 100644
--- a/NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.h
+++ b/NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.h
@@ -116,16 +116,6 @@ typedef struct {
} PXEBC_DHCP6_PACKET_CACHE;
-/**
- Free all the nodes in the boot file list.
-
- @param[in] Head The pointer to the head of the list.
-
-**/
-VOID
-PxeBcFreeBootFileOption (
- IN LIST_ENTRY *Head
- );
/**
--
2.18.0.windows.1
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
Hi, Songpeng The patch is good with me, please remember to update the copyright year when commit it. Reviewed-by: Fu Siyuan <siyuan.fu@intel.com> > -----Original Message----- > From: Li, Songpeng > Sent: Monday, August 13, 2018 10:45 AM > To: edk2-devel@lists.01.org > Cc: Wu, Jiaxin <jiaxin.wu@intel.com>; Fu, Siyuan <siyuan.fu@intel.com> > Subject: [PATCH] NetworkPkg: Remove the redundant code and definition. > > Cc: Jiaxin Wu <jiaxin.wu@intel.com> > Cc: Siyuan Fu <siyuan.fu@intel.com> > Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1064 > Contributed-under: TianoCore Contribution Agreement 1.1 > Signed-off-by: Songpeng Li <songpeng.li@intel.com> > --- > NetworkPkg/IScsiDxe/IScsiDxe.inf | 1 - > NetworkPkg/IScsiDxe/IScsiProto.c | 33 ------------ > NetworkPkg/IpSecDxe/Ikev2/Payload.c | 18 ------- > NetworkPkg/IpSecDxe/Ikev2/Utility.c | 76 ---------------------------- > NetworkPkg/IpSecDxe/Ikev2/Utility.h | 67 ------------------------ > NetworkPkg/TcpDxe/TcpOption.c | 30 ----------- > NetworkPkg/TcpDxe/TcpOption.h | 16 ------ > NetworkPkg/TlsDxe/TlsDxe.inf | 1 - > NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.c | 23 --------- > NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.h | 10 ---- > 10 files changed, 275 deletions(-) > > diff --git a/NetworkPkg/IScsiDxe/IScsiDxe.inf > b/NetworkPkg/IScsiDxe/IScsiDxe.inf > index 2d96611b44..60737c96ab 100644 > --- a/NetworkPkg/IScsiDxe/IScsiDxe.inf > +++ b/NetworkPkg/IScsiDxe/IScsiDxe.inf > @@ -124,7 +124,6 @@ > gEfiIfrTianoGuid ## SOMETIMES_PRODUCES ## > UNDEFINED > gEfiAcpiTableGuid ## SOMETIMES_CONSUMES ## > SystemTable > gEfiAcpi10TableGuid ## SOMETIMES_CONSUMES ## > SystemTable > - gEfiAcpi20TableGuid ## SOMETIMES_CONSUMES ## > SystemTable > gEfiAdapterInfoNetworkBootGuid ## SOMETIMES_CONSUMES ## > UNDEFINED > gEfiAdapterInfoUndiIpv6SupportGuid ## SOMETIMES_CONSUMES ## > GUID > > diff --git a/NetworkPkg/IScsiDxe/IScsiProto.c > b/NetworkPkg/IScsiDxe/IScsiProto.c > index 7619360568..f4a49c677a 100644 > --- a/NetworkPkg/IScsiDxe/IScsiProto.c > +++ b/NetworkPkg/IScsiDxe/IScsiProto.c > @@ -2096,39 +2096,6 @@ IScsiDelTcb ( > } > > > -/** > - Find the task control block by the initator task tag. > - > - @param[in] TcbList The tcb list. > - @param[in] InitiatorTaskTag The initiator task tag. > - > - @return The task control block found. > - @retval NULL The task control block cannot be found. > - > -**/ > -ISCSI_TCB * > -IScsiFindTcbByITT ( > - IN LIST_ENTRY *TcbList, > - IN UINT32 InitiatorTaskTag > - ) > -{ > - ISCSI_TCB *Tcb; > - LIST_ENTRY *Entry; > - > - Tcb = NULL; > - > - NET_LIST_FOR_EACH (Entry, TcbList) { > - Tcb = NET_LIST_USER_STRUCT (Entry, ISCSI_TCB, Link); > - > - if (Tcb->InitiatorTaskTag == InitiatorTaskTag) { > - break; > - } > - } > - > - return Tcb; > -} > - > - > /** > Create a data segment, pad it, and calculate the CRC if needed. > > diff --git a/NetworkPkg/IpSecDxe/Ikev2/Payload.c > b/NetworkPkg/IpSecDxe/Ikev2/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; > } > > -/** > - Save some useful payloads after accepting the Packet. > - > - @param[in] SessionCommon Pointer to IKEV2_SESSION_COMMON related to > the operation. > - @param[in] IkePacket Pointer to received IkePacet. > - @param[in] IkeType The type used to indicate it is in IkeSa or > ChildSa or Info > - exchange. > - > -**/ > -VOID > -Ikev2OnPacketAccepted ( > - IN IKEV2_SESSION_COMMON *SessionCommon, > - IN IKE_PACKET *IkePacket, > - IN UINT8 IkeType > - ) > -{ > - return; > -} > > /** > > diff --git a/NetworkPkg/IpSecDxe/Ikev2/Utility.c > b/NetworkPkg/IpSecDxe/Ikev2/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; > } > > -/** > - 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; > -} > > /** > Free specified Seession Common. The session common would belong to a > IKE SA or > @@ -659,33 +644,6 @@ Ikev2ChildSaSessionReg ( > return ; > } > > -/** > - 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 = IKEV2_CHILD_SA_SESSION_BY_IKE_SA (Entry); > - > - if (ChildSaSession->MessageId == Mid) { > - return ChildSaSession; > - } > - } > - return NULL; > -} > > /** > This function find the Child SA by the specified SPI. > @@ -776,22 +734,6 @@ Ikev2ChildSaSessionRemove ( > return NULL; > } > > -/** > - 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. > > @@ -1137,24 +1079,6 @@ Ikev2ChildSaAssociateSpdEntry ( > } > > > -/** > - 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; > -} > > /** > Validate the IKE header of received IKE packet. > diff --git a/NetworkPkg/IpSecDxe/Ikev2/Utility.h > b/NetworkPkg/IpSecDxe/Ikev2/Utility.h > index 8383ff8dd6..79def7740d 100644 > --- a/NetworkPkg/IpSecDxe/Ikev2/Utility.h > +++ b/NetworkPkg/IpSecDxe/Ikev2/Utility.h > @@ -104,19 +104,6 @@ Ikev2SaSessionRemove ( > ); > > > -/** > - 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 > packet. > > @@ -211,20 +198,6 @@ Ikev2ChildSaSessionLookupBySpi ( > IN UINT32 Spi > ); > > -/** > - 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 > - ); > > /** > Insert a Child SA Session into the specified ChildSa list.. > @@ -257,18 +230,6 @@ Ikev2ChildSaSessionRemove ( > IN UINT8 ListType > ); > > -/** > - 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 > - ); > > /** > Free the memory located for the specified IKEV2_CHILD_SA_SESSION. > @@ -328,19 +289,6 @@ Ikev2ChildExchangeRequestType( > IN IKE_PACKET *IkePacket > ); > > -/** > - 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 > - ); > - > > /** > Associate a SPD selector to the Child SA Session. > @@ -882,21 +830,6 @@ Ikev2DecodePacket ( > IN UINTN IkeType > ); > > -/** > - Save some useful payloads after accepting the Packet. > - > - @param[in] SessionCommon Pointer to IKEV2_SESSION_COMMON related to > the operation. > - @param[in] IkePacket Pointer to received IkePacet. > - @param[in] IkeType The type used to indicate it is in IkeSa or > ChildSa or Info > - exchange. > - > -**/ > -VOID > -Ikev2OnPacketAccepted ( > - IN IKEV2_SESSION_COMMON *SessionCommon, > - IN IKE_PACKET *IkePacket, > - IN UINT8 IkeType > - ); > > /** > Send out IKEV2 packet. > diff --git a/NetworkPkg/TcpDxe/TcpOption.c b/NetworkPkg/TcpDxe/TcpOption.c > index bacce1070d..73d781729f 100644 > --- a/NetworkPkg/TcpDxe/TcpOption.c > +++ b/NetworkPkg/TcpDxe/TcpOption.c > @@ -342,33 +342,3 @@ TcpParseOption ( > > return 0; > } > - > -/** > - Check the segment against PAWS. > - > - @param[in] Tcb Pointer to the TCP_CB of this TCP instance. > - @param[in] TSVal The timestamp value. > - > - @retval 1 The segment passed the PAWS check. > - @retval 0 The segment failed to pass the PAWS check. > - > -**/ > -UINT32 > -TcpPawsOK ( > - IN TCP_CB *Tcb, > - IN UINT32 TSVal > - ) > -{ > - // > - // PAWS as defined in RFC1323, buggy... > - // > - if (TCP_TIME_LT (TSVal, Tcb->TsRecent) && > - TCP_TIME_LT (Tcb->TsRecentAge + TCP_PAWS_24DAY, mTcpTick) > - ) { > - > - return 0; > - > - } > - > - return 1; > -} > diff --git a/NetworkPkg/TcpDxe/TcpOption.h b/NetworkPkg/TcpDxe/TcpOption.h > index 0ccadb9536..9d8581b669 100644 > --- a/NetworkPkg/TcpDxe/TcpOption.h > +++ b/NetworkPkg/TcpDxe/TcpOption.h > @@ -126,20 +126,4 @@ TcpParseOption ( > IN OUT TCP_OPTION *Option > ); > > -/** > - Check the segment against PAWS. > - > - @param[in] Tcb Pointer to the TCP_CB of this TCP instance. > - @param[in] TSVal The timestamp value. > - > - @retval 1 The segment passed the PAWS check. > - @retval 0 The segment failed to pass the PAWS check. > - > -**/ > -UINT32 > -TcpPawsOK ( > - IN TCP_CB *Tcb, > - IN UINT32 TSVal > - ); > - > #endif > diff --git a/NetworkPkg/TlsDxe/TlsDxe.inf b/NetworkPkg/TlsDxe/TlsDxe.inf > index 907feb735b..bbe8c554a2 100644 > --- a/NetworkPkg/TlsDxe/TlsDxe.inf > +++ b/NetworkPkg/TlsDxe/TlsDxe.inf > @@ -52,7 +52,6 @@ > BaseLib > UefiLib > DebugLib > - NetLib > BaseCryptLib > TlsLib > > diff --git a/NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.c > b/NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.c > index c06b0242bb..5bbc1922a6 100644 > --- a/NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.c > +++ b/NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.c > @@ -202,29 +202,6 @@ PxeBcCacheDhcp6Packet ( > return EFI_SUCCESS; > } > > - > -/** > - Free all the nodes in the list for boot file. > - > - @param[in] Head The pointer to the head of list. > - > -**/ > -VOID > -PxeBcFreeBootFileOption ( > - IN LIST_ENTRY *Head > - ) > -{ > - LIST_ENTRY *Entry; > - LIST_ENTRY *NextEntry; > - PXEBC_DHCP6_OPTION_NODE *Node; > - > - NET_LIST_FOR_EACH_SAFE (Entry, NextEntry, Head) { > - Node = NET_LIST_USER_STRUCT (Entry, PXEBC_DHCP6_OPTION_NODE, Link); > - RemoveEntryList (Entry); > - FreePool (Node); > - } > -} > - > /** > Retrieve the boot server address using the EFI_DNS6_PROTOCOL. > > diff --git a/NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.h > b/NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.h > index f3dcd89743..d6e6745f0d 100644 > --- a/NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.h > +++ b/NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.h > @@ -116,16 +116,6 @@ typedef struct { > } PXEBC_DHCP6_PACKET_CACHE; > > > -/** > - Free all the nodes in the boot file list. > - > - @param[in] Head The pointer to the head of the list. > - > -**/ > -VOID > -PxeBcFreeBootFileOption ( > - IN LIST_ENTRY *Head > - ); > > > /** > -- > 2.18.0.windows.1 _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel
On 08/13/18 04:45, Songpeng Li wrote: > Cc: Jiaxin Wu <jiaxin.wu@intel.com> > Cc: Siyuan Fu <siyuan.fu@intel.com> > Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1064 > Contributed-under: TianoCore Contribution Agreement 1.1 > Signed-off-by: Songpeng Li <songpeng.li@intel.com> > --- > NetworkPkg/IScsiDxe/IScsiDxe.inf | 1 - > NetworkPkg/IScsiDxe/IScsiProto.c | 33 ------------ > NetworkPkg/IpSecDxe/Ikev2/Payload.c | 18 ------- > NetworkPkg/IpSecDxe/Ikev2/Utility.c | 76 ---------------------------- > NetworkPkg/IpSecDxe/Ikev2/Utility.h | 67 ------------------------ > NetworkPkg/TcpDxe/TcpOption.c | 30 ----------- > NetworkPkg/TcpDxe/TcpOption.h | 16 ------ > NetworkPkg/TlsDxe/TlsDxe.inf | 1 - > NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.c | 23 --------- > NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.h | 10 ---- > 10 files changed, 275 deletions(-) Nacked-by: Laszlo Ersek <lersek@redhat.com> Please fragment the patch to a series so that each patch modify only one module under NetworkPkg. References: http://mid.mail-archive.com/4A89E2EF3DFEDB4C8BFDE51014F606A14E2C6745@SHSMSX104.ccr.corp.intel.com http://mid.mail-archive.com/20180808084712.34696-1-shenglei.zhang@intel.com Laszlo _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel
Looks good to me. Reviewed-by: Jiaxin Wu <jiaxin.wu@intel.com> > -----Original Message----- > From: Li, Songpeng > Sent: Monday, August 13, 2018 10:45 AM > To: edk2-devel@lists.01.org > Cc: Wu, Jiaxin <jiaxin.wu@intel.com>; Fu, Siyuan <siyuan.fu@intel.com> > Subject: [PATCH] NetworkPkg: Remove the redundant code and definition. > > Cc: Jiaxin Wu <jiaxin.wu@intel.com> > Cc: Siyuan Fu <siyuan.fu@intel.com> > Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1064 > Contributed-under: TianoCore Contribution Agreement 1.1 > Signed-off-by: Songpeng Li <songpeng.li@intel.com> > --- > NetworkPkg/IScsiDxe/IScsiDxe.inf | 1 - > NetworkPkg/IScsiDxe/IScsiProto.c | 33 ------------ > NetworkPkg/IpSecDxe/Ikev2/Payload.c | 18 ------- > NetworkPkg/IpSecDxe/Ikev2/Utility.c | 76 ---------------------------- > NetworkPkg/IpSecDxe/Ikev2/Utility.h | 67 ------------------------ > NetworkPkg/TcpDxe/TcpOption.c | 30 ----------- > NetworkPkg/TcpDxe/TcpOption.h | 16 ------ > NetworkPkg/TlsDxe/TlsDxe.inf | 1 - > NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.c | 23 --------- > NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.h | 10 ---- > 10 files changed, 275 deletions(-) > > diff --git a/NetworkPkg/IScsiDxe/IScsiDxe.inf > b/NetworkPkg/IScsiDxe/IScsiDxe.inf > index 2d96611b44..60737c96ab 100644 > --- a/NetworkPkg/IScsiDxe/IScsiDxe.inf > +++ b/NetworkPkg/IScsiDxe/IScsiDxe.inf > @@ -124,7 +124,6 @@ > gEfiIfrTianoGuid ## SOMETIMES_PRODUCES ## UNDEFINED > gEfiAcpiTableGuid ## SOMETIMES_CONSUMES ## > SystemTable > gEfiAcpi10TableGuid ## SOMETIMES_CONSUMES ## > SystemTable > - gEfiAcpi20TableGuid ## SOMETIMES_CONSUMES ## > SystemTable > gEfiAdapterInfoNetworkBootGuid ## SOMETIMES_CONSUMES ## > UNDEFINED > gEfiAdapterInfoUndiIpv6SupportGuid ## SOMETIMES_CONSUMES ## > GUID > > diff --git a/NetworkPkg/IScsiDxe/IScsiProto.c > b/NetworkPkg/IScsiDxe/IScsiProto.c > index 7619360568..f4a49c677a 100644 > --- a/NetworkPkg/IScsiDxe/IScsiProto.c > +++ b/NetworkPkg/IScsiDxe/IScsiProto.c > @@ -2096,39 +2096,6 @@ IScsiDelTcb ( > } > > > -/** > - Find the task control block by the initator task tag. > - > - @param[in] TcbList The tcb list. > - @param[in] InitiatorTaskTag The initiator task tag. > - > - @return The task control block found. > - @retval NULL The task control block cannot be found. > - > -**/ > -ISCSI_TCB * > -IScsiFindTcbByITT ( > - IN LIST_ENTRY *TcbList, > - IN UINT32 InitiatorTaskTag > - ) > -{ > - ISCSI_TCB *Tcb; > - LIST_ENTRY *Entry; > - > - Tcb = NULL; > - > - NET_LIST_FOR_EACH (Entry, TcbList) { > - Tcb = NET_LIST_USER_STRUCT (Entry, ISCSI_TCB, Link); > - > - if (Tcb->InitiatorTaskTag == InitiatorTaskTag) { > - break; > - } > - } > - > - return Tcb; > -} > - > - > /** > Create a data segment, pad it, and calculate the CRC if needed. > > diff --git a/NetworkPkg/IpSecDxe/Ikev2/Payload.c > b/NetworkPkg/IpSecDxe/Ikev2/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; > } > > -/** > - Save some useful payloads after accepting the Packet. > - > - @param[in] SessionCommon Pointer to IKEV2_SESSION_COMMON > related to the operation. > - @param[in] IkePacket Pointer to received IkePacet. > - @param[in] IkeType The type used to indicate it is in IkeSa or ChildSa or > Info > - exchange. > - > -**/ > -VOID > -Ikev2OnPacketAccepted ( > - IN IKEV2_SESSION_COMMON *SessionCommon, > - IN IKE_PACKET *IkePacket, > - IN UINT8 IkeType > - ) > -{ > - return; > -} > > /** > > diff --git a/NetworkPkg/IpSecDxe/Ikev2/Utility.c > b/NetworkPkg/IpSecDxe/Ikev2/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; > } > > -/** > - 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; > -} > > /** > Free specified Seession Common. The session common would belong to a > IKE SA or > @@ -659,33 +644,6 @@ Ikev2ChildSaSessionReg ( > return ; > } > > -/** > - 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 = IKEV2_CHILD_SA_SESSION_BY_IKE_SA (Entry); > - > - if (ChildSaSession->MessageId == Mid) { > - return ChildSaSession; > - } > - } > - return NULL; > -} > > /** > This function find the Child SA by the specified SPI. > @@ -776,22 +734,6 @@ Ikev2ChildSaSessionRemove ( > return NULL; > } > > -/** > - 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. > > @@ -1137,24 +1079,6 @@ Ikev2ChildSaAssociateSpdEntry ( > } > > > -/** > - 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; > -} > > /** > Validate the IKE header of received IKE packet. > diff --git a/NetworkPkg/IpSecDxe/Ikev2/Utility.h > b/NetworkPkg/IpSecDxe/Ikev2/Utility.h > index 8383ff8dd6..79def7740d 100644 > --- a/NetworkPkg/IpSecDxe/Ikev2/Utility.h > +++ b/NetworkPkg/IpSecDxe/Ikev2/Utility.h > @@ -104,19 +104,6 @@ Ikev2SaSessionRemove ( > ); > > > -/** > - 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 packet. > > @@ -211,20 +198,6 @@ Ikev2ChildSaSessionLookupBySpi ( > IN UINT32 Spi > ); > > -/** > - 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 > - ); > > /** > Insert a Child SA Session into the specified ChildSa list.. > @@ -257,18 +230,6 @@ Ikev2ChildSaSessionRemove ( > IN UINT8 ListType > ); > > -/** > - 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 > - ); > > /** > Free the memory located for the specified IKEV2_CHILD_SA_SESSION. > @@ -328,19 +289,6 @@ Ikev2ChildExchangeRequestType( > IN IKE_PACKET *IkePacket > ); > > -/** > - 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 > - ); > - > > /** > Associate a SPD selector to the Child SA Session. > @@ -882,21 +830,6 @@ Ikev2DecodePacket ( > IN UINTN IkeType > ); > > -/** > - Save some useful payloads after accepting the Packet. > - > - @param[in] SessionCommon Pointer to IKEV2_SESSION_COMMON > related to the operation. > - @param[in] IkePacket Pointer to received IkePacet. > - @param[in] IkeType The type used to indicate it is in IkeSa or ChildSa or > Info > - exchange. > - > -**/ > -VOID > -Ikev2OnPacketAccepted ( > - IN IKEV2_SESSION_COMMON *SessionCommon, > - IN IKE_PACKET *IkePacket, > - IN UINT8 IkeType > - ); > > /** > Send out IKEV2 packet. > diff --git a/NetworkPkg/TcpDxe/TcpOption.c > b/NetworkPkg/TcpDxe/TcpOption.c > index bacce1070d..73d781729f 100644 > --- a/NetworkPkg/TcpDxe/TcpOption.c > +++ b/NetworkPkg/TcpDxe/TcpOption.c > @@ -342,33 +342,3 @@ TcpParseOption ( > > return 0; > } > - > -/** > - Check the segment against PAWS. > - > - @param[in] Tcb Pointer to the TCP_CB of this TCP instance. > - @param[in] TSVal The timestamp value. > - > - @retval 1 The segment passed the PAWS check. > - @retval 0 The segment failed to pass the PAWS check. > - > -**/ > -UINT32 > -TcpPawsOK ( > - IN TCP_CB *Tcb, > - IN UINT32 TSVal > - ) > -{ > - // > - // PAWS as defined in RFC1323, buggy... > - // > - if (TCP_TIME_LT (TSVal, Tcb->TsRecent) && > - TCP_TIME_LT (Tcb->TsRecentAge + TCP_PAWS_24DAY, mTcpTick) > - ) { > - > - return 0; > - > - } > - > - return 1; > -} > diff --git a/NetworkPkg/TcpDxe/TcpOption.h > b/NetworkPkg/TcpDxe/TcpOption.h > index 0ccadb9536..9d8581b669 100644 > --- a/NetworkPkg/TcpDxe/TcpOption.h > +++ b/NetworkPkg/TcpDxe/TcpOption.h > @@ -126,20 +126,4 @@ TcpParseOption ( > IN OUT TCP_OPTION *Option > ); > > -/** > - Check the segment against PAWS. > - > - @param[in] Tcb Pointer to the TCP_CB of this TCP instance. > - @param[in] TSVal The timestamp value. > - > - @retval 1 The segment passed the PAWS check. > - @retval 0 The segment failed to pass the PAWS check. > - > -**/ > -UINT32 > -TcpPawsOK ( > - IN TCP_CB *Tcb, > - IN UINT32 TSVal > - ); > - > #endif > diff --git a/NetworkPkg/TlsDxe/TlsDxe.inf b/NetworkPkg/TlsDxe/TlsDxe.inf > index 907feb735b..bbe8c554a2 100644 > --- a/NetworkPkg/TlsDxe/TlsDxe.inf > +++ b/NetworkPkg/TlsDxe/TlsDxe.inf > @@ -52,7 +52,6 @@ > BaseLib > UefiLib > DebugLib > - NetLib > BaseCryptLib > TlsLib > > diff --git a/NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.c > b/NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.c > index c06b0242bb..5bbc1922a6 100644 > --- a/NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.c > +++ b/NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.c > @@ -202,29 +202,6 @@ PxeBcCacheDhcp6Packet ( > return EFI_SUCCESS; > } > > - > -/** > - Free all the nodes in the list for boot file. > - > - @param[in] Head The pointer to the head of list. > - > -**/ > -VOID > -PxeBcFreeBootFileOption ( > - IN LIST_ENTRY *Head > - ) > -{ > - LIST_ENTRY *Entry; > - LIST_ENTRY *NextEntry; > - PXEBC_DHCP6_OPTION_NODE *Node; > - > - NET_LIST_FOR_EACH_SAFE (Entry, NextEntry, Head) { > - Node = NET_LIST_USER_STRUCT (Entry, PXEBC_DHCP6_OPTION_NODE, > Link); > - RemoveEntryList (Entry); > - FreePool (Node); > - } > -} > - > /** > Retrieve the boot server address using the EFI_DNS6_PROTOCOL. > > diff --git a/NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.h > b/NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.h > index f3dcd89743..d6e6745f0d 100644 > --- a/NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.h > +++ b/NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.h > @@ -116,16 +116,6 @@ typedef struct { > } PXEBC_DHCP6_PACKET_CACHE; > > > -/** > - Free all the nodes in the boot file list. > - > - @param[in] Head The pointer to the head of the list. > - > -**/ > -VOID > -PxeBcFreeBootFileOption ( > - IN LIST_ENTRY *Head > - ); > > > /** > -- > 2.18.0.windows.1 _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel
© 2016 - 2023 Red Hat, Inc.