MdeModulePkg\Universal\Network\Tcp4Dxe has the same issue, I will provide 
another patch to fix it also.

Thanks,
Jiaxin

> -----Original Message-----
> From: edk2-devel [mailto:[email protected]] On Behalf Of
> Wu, Jiaxin
> Sent: Friday, May 20, 2016 8:31 AM
> To: Gary Lin <[email protected]>; [email protected]
> Cc: Fu, Siyuan <[email protected]>
> Subject: Re: [edk2] [PATCH 2/2] NetworkPkg/TcpDxe: Remove the status
> check of SockProcessRcvToken
> 
> Reviewed-By: Wu Jiaxin <[email protected]>
> 
> 
> > -----Original Message-----
> > From: edk2-devel [mailto:[email protected]] On Behalf Of
> > Gary Lin
> > Sent: Thursday, May 19, 2016 11:49 AM
> > To: [email protected]
> > Cc: Fu, Siyuan <[email protected]>; Wu, Jiaxin <[email protected]>
> > Subject: [edk2] [PATCH 2/2] NetworkPkg/TcpDxe: Remove the status
> check
> > of SockProcessRcvToken
> >
> > SockProcessRcvToken only returns the number of the received bytes, not
> > an EFI Status.
> >
> > Cc: "Siyuan Fu" <[email protected]>
> > Cc: "Jiaxin Wu" <[email protected]>
> > Contributed-under: TianoCore Contribution Agreement 1.0
> > Signed-off-by: Gary Lin <[email protected]>
> > ---
> >  NetworkPkg/TcpDxe/SockInterface.c | 6 +-----
> >  1 file changed, 1 insertion(+), 5 deletions(-)
> >
> > diff --git a/NetworkPkg/TcpDxe/SockInterface.c
> > b/NetworkPkg/TcpDxe/SockInterface.c
> > index 4abda74..1f3524b 100644
> > --- a/NetworkPkg/TcpDxe/SockInterface.c
> > +++ b/NetworkPkg/TcpDxe/SockInterface.c
> > @@ -724,11 +724,7 @@ SockRcv (
> >    }
> >
> >    if (RcvdBytes != 0) {
> > -    Status = SockProcessRcvToken (Sock, RcvToken);
> > -
> > -    if (EFI_ERROR (Status)) {
> > -      goto Exit;
> > -    }
> > +    SockProcessRcvToken (Sock, RcvToken);
> >
> >      Status = Sock->ProtoHandler (Sock, SOCK_CONSUMED, NULL);
> >    } else {
> > --
> > 2.8.2
> >
> > _______________________________________________
> > edk2-devel mailing list
> > [email protected]
> > https://lists.01.org/mailman/listinfo/edk2-devel
> _______________________________________________
> edk2-devel mailing list
> [email protected]
> https://lists.01.org/mailman/listinfo/edk2-devel
_______________________________________________
edk2-devel mailing list
[email protected]
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to