On Tue, Nov 03, 2015 at 04:13:27PM +0800, Zhang Lubo wrote:
> v2:
> *Rewrite the logic of setting ipv6 gateway in httpboot driver and update some 
> comment.
> 
> Fix a potential bug which can cause assert when setting ipv6 Gateway. If 
> there is no
> router in network environment, it will fail to find out the gateway address 
> which can
> route the message to Http Server Ip . When it check the IP6 route table again 
> 1 seconds
> later, it will assert while the router table structure had been freed last 
> time without
> retrieve new relevant data form ip6 driver.
> 

Hi Lubo,

I applied the patch and it worked as expected except the IP expressed
url. I checked the code and found the function(*) to parse the url couldn't
handle IPv6 and just treat the colon as a separator of the host name and port.
Is this a known issue?

Cheers,

Gary Lin

(*) MdeModulePkg/Library/DxeHttpLib/DxeHttpLib.c::NetHttpParseAuthority()

> Cc: Fu Siyuan <siyuan...@intel.com>
> Cc: Ye Ting <ting...@intel.com>
> CC: Wu Jiaxin <jiaxin...@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Zhang Lubo <lubo.zh...@intel.com>
> ---
>  NetworkPkg/HttpBootDxe/HttpBootClient.c        |  234 ++++-
>  NetworkPkg/HttpBootDxe/HttpBootComponentName.c |    5 +-
>  NetworkPkg/HttpBootDxe/HttpBootDhcp4.c         |   12 +-
>  NetworkPkg/HttpBootDxe/HttpBootDhcp4.h         |   11 +
>  NetworkPkg/HttpBootDxe/HttpBootDhcp6.c         |  984 +++++++++++++++++++
>  NetworkPkg/HttpBootDxe/HttpBootDhcp6.h         |  198 ++++
>  NetworkPkg/HttpBootDxe/HttpBootDxe.c           |  903 +++++++++++++++---
>  NetworkPkg/HttpBootDxe/HttpBootDxe.h           |  159 +++-
>  NetworkPkg/HttpBootDxe/HttpBootDxe.inf         |    9 +
>  NetworkPkg/HttpBootDxe/HttpBootImpl.c          |  109 ++-
>  NetworkPkg/HttpBootDxe/HttpBootImpl.h          |    2 +-
>  NetworkPkg/HttpBootDxe/HttpBootSupport.c       |  292 +++++-
>  NetworkPkg/HttpBootDxe/HttpBootSupport.h       |   70 ++
>  NetworkPkg/HttpDxe/HttpDns.c                   |  207 +++-
>  NetworkPkg/HttpDxe/HttpDns.h                   |   20 +
>  NetworkPkg/HttpDxe/HttpDriver.c                |  806 +++++++++++-----
>  NetworkPkg/HttpDxe/HttpDriver.h                |  143 ++-
>  NetworkPkg/HttpDxe/HttpDxe.inf                 |    5 +
>  NetworkPkg/HttpDxe/HttpImpl.c                  |  358 +++----
>  NetworkPkg/HttpDxe/HttpImpl.h                  |    1 -
>  NetworkPkg/HttpDxe/HttpProto.c                 | 1210 
> ++++++++++++++++++++----
>  NetworkPkg/HttpDxe/HttpProto.h                 |  182 +++-
>  22 files changed, 5017 insertions(+), 903 deletions(-)
>  create mode 100644 NetworkPkg/HttpBootDxe/HttpBootDhcp6.c
>  create mode 100644 NetworkPkg/HttpBootDxe/HttpBootDhcp6.h
> 
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to