public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* Re: 2nd HttpBoot fails even after upgrading to revision 24729
@ 2016-08-24  7:39 Santhapur Naveen
  2016-08-24  8:42 ` Zhang, Lubo
  0 siblings, 1 reply; 12+ messages in thread
From: Santhapur Naveen @ 2016-08-24  7:39 UTC (permalink / raw)
  To: edk2-devel@lists.01.org

Hello,

          Revision 24729 of edk2 says 2nd HttpBoot fail issue has been fixed. So, I have taken the changes from the file NetworkPkg/HttpDxe/HttpImpl.c of the revsision 24729 and still the 2nd  HttpBoot fails. Please let me know if I have missed any.

P.S: The first http boot happens successfully though. (downloading a Shell.efi)

Thanks,
Naveen


^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: 2nd HttpBoot fails even after upgrading to revision 24729
  2016-08-24  7:39 2nd HttpBoot fails even after upgrading to revision 24729 Santhapur Naveen
@ 2016-08-24  8:42 ` Zhang, Lubo
  2016-08-24  8:52   ` Santhapur Naveen
  0 siblings, 1 reply; 12+ messages in thread
From: Zhang, Lubo @ 2016-08-24  8:42 UTC (permalink / raw)
  To: Santhapur Naveen; +Cc: edk2-devel@lists.01.org

Hi Naveen
     Could you list the patch name or Git SHA value which fixed the 2nd HttpBootfail issue. On my side, the SVN version 24729 is not exist.

Thanks 
Lubo 

-----Original Message-----
From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Santhapur Naveen
Sent: Wednesday, August 24, 2016 3:40 PM
To: edk2-devel@lists.01.org
Subject: Re: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hello,

          Revision 24729 of edk2 says 2nd HttpBoot fail issue has been fixed. So, I have taken the changes from the file NetworkPkg/HttpDxe/HttpImpl.c of the revsision 24729 and still the 2nd  HttpBoot fails. Please let me know if I have missed any.

P.S: The first http boot happens successfully though. (downloading a Shell.efi)

Thanks,
Naveen
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel


^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: 2nd HttpBoot fails even after upgrading to revision 24729
  2016-08-24  8:42 ` Zhang, Lubo
@ 2016-08-24  8:52   ` Santhapur Naveen
  2016-08-24  9:09     ` Zhang, Lubo
  2016-08-24 10:52     ` Zhang, Lubo
  0 siblings, 2 replies; 12+ messages in thread
From: Santhapur Naveen @ 2016-08-24  8:52 UTC (permalink / raw)
  To: Zhang, Lubo; +Cc: edk2-devel@lists.01.org

Hi Lubo,

	Please find the following details that may help.

NetworkPkg:Fix a bug the 2nd httpboot fail issue.

Httpboot over Ipv4 or Ipv6 stack,for both Identity and chunked transfer mode,when the last data has been parsed by HttpLib, the HttpInstance->NextMsg pointer should point a correct location.Now after the first  successful httpboot for ipv4 or ipv6,the HttpInstance->NextMsgpoint the character after the last byte, it may be a bad buffer if we don't receive another HttpHeader, so if call a 2nd httpboot, the wrong NextMsg pointer will cause the httpboot fail, so we need to check this case in HttpBodyParserCallback function in the first http boot process.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Zhang Lubo <lubo.zhang@intel.com>
Reviewed-by: Fu Siyuan <siyuan.fu@intel.com>
Reviewed-by: Ye Ting <ting.ye@intel.com>
Reviewed-by: Wu Jiaxin <jiaxin.wu@intel.com>

git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@19423 6f19259b-4bc3-4df7-8a09-765794883524


Thanks,
Naveen

-----Original Message-----
From: Zhang, Lubo [mailto:lubo.zhang@intel.com] 
Sent: Wednesday, August 24, 2016 2:13 PM
To: Santhapur Naveen
Cc: edk2-devel@lists.01.org
Subject: RE: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hi Naveen
     Could you list the patch name or Git SHA value which fixed the 2nd HttpBootfail issue. On my side, the SVN version 24729 is not exist.

Thanks 
Lubo 

-----Original Message-----
From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Santhapur Naveen
Sent: Wednesday, August 24, 2016 3:40 PM
To: edk2-devel@lists.01.org
Subject: Re: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hello,

          Revision 24729 of edk2 says 2nd HttpBoot fail issue has been fixed. So, I have taken the changes from the file NetworkPkg/HttpDxe/HttpImpl.c of the revsision 24729 and still the 2nd  HttpBoot fails. Please let me know if I have missed any.

P.S: The first http boot happens successfully though. (downloading a Shell.efi)

Thanks,
Naveen
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel


^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: 2nd HttpBoot fails even after upgrading to revision 24729
  2016-08-24  8:52   ` Santhapur Naveen
@ 2016-08-24  9:09     ` Zhang, Lubo
  2016-08-24 10:52     ` Zhang, Lubo
  1 sibling, 0 replies; 12+ messages in thread
From: Zhang, Lubo @ 2016-08-24  9:09 UTC (permalink / raw)
  To: Santhapur Naveen; +Cc: edk2-devel@lists.01.org

Thanks  Naveen. I will check it first and any progress I will inform you.

Best Regards
Lubo

-----Original Message-----
From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Santhapur Naveen
Sent: Wednesday, August 24, 2016 4:53 PM
To: Zhang, Lubo <lubo.zhang@intel.com>
Cc: edk2-devel@lists.01.org
Subject: Re: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hi Lubo,

	Please find the following details that may help.

NetworkPkg:Fix a bug the 2nd httpboot fail issue.

Httpboot over Ipv4 or Ipv6 stack,for both Identity and chunked transfer mode,when the last data has been parsed by HttpLib, the HttpInstance->NextMsg pointer should point a correct location.Now after the first  successful httpboot for ipv4 or ipv6,the HttpInstance->NextMsgpoint the character after the last byte, it may be a bad buffer if we don't receive another HttpHeader, so if call a 2nd httpboot, the wrong NextMsg pointer will cause the httpboot fail, so we need to check this case in HttpBodyParserCallback function in the first http boot process.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Zhang Lubo <lubo.zhang@intel.com>
Reviewed-by: Fu Siyuan <siyuan.fu@intel.com>
Reviewed-by: Ye Ting <ting.ye@intel.com>
Reviewed-by: Wu Jiaxin <jiaxin.wu@intel.com>

git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@19423 6f19259b-4bc3-4df7-8a09-765794883524


Thanks,
Naveen

-----Original Message-----
From: Zhang, Lubo [mailto:lubo.zhang@intel.com] 
Sent: Wednesday, August 24, 2016 2:13 PM
To: Santhapur Naveen
Cc: edk2-devel@lists.01.org
Subject: RE: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hi Naveen
     Could you list the patch name or Git SHA value which fixed the 2nd HttpBootfail issue. On my side, the SVN version 24729 is not exist.

Thanks 
Lubo 

-----Original Message-----
From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Santhapur Naveen
Sent: Wednesday, August 24, 2016 3:40 PM
To: edk2-devel@lists.01.org
Subject: Re: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hello,

          Revision 24729 of edk2 says 2nd HttpBoot fail issue has been fixed. So, I have taken the changes from the file NetworkPkg/HttpDxe/HttpImpl.c of the revsision 24729 and still the 2nd  HttpBoot fails. Please let me know if I have missed any.

P.S: The first http boot happens successfully though. (downloading a Shell.efi)

Thanks,
Naveen
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel


^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: 2nd HttpBoot fails even after upgrading to revision 24729
  2016-08-24  8:52   ` Santhapur Naveen
  2016-08-24  9:09     ` Zhang, Lubo
@ 2016-08-24 10:52     ` Zhang, Lubo
  2016-08-24 11:09       ` Santhapur Naveen
  1 sibling, 1 reply; 12+ messages in thread
From: Zhang, Lubo @ 2016-08-24 10:52 UTC (permalink / raw)
  To: Santhapur Naveen; +Cc: edk2-devel@lists.01.org

Hi Naveen
      I have checked the issue you mentioned. On my side, I updated the code to latest and do some test on NT32 and Denlow platform, both the first http boot and 2nd are successful. 
      Until now , we have fixed few bugs which may lead the 2nd http boot fail. As for the patch you mentioned is one case to fix the bug, So I suggest you to update the code firstly and at least make sure the HttpDxe , HttpBootDxe and DxeHttpLib are latest. 
   
     If any result, Pls let me know.

Best Regards
Lubo 
     

-----Original Message-----
From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Santhapur Naveen
Sent: Wednesday, August 24, 2016 4:53 PM
To: Zhang, Lubo <lubo.zhang@intel.com>
Cc: edk2-devel@lists.01.org
Subject: Re: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hi Lubo,

	Please find the following details that may help.

NetworkPkg:Fix a bug the 2nd httpboot fail issue.

Httpboot over Ipv4 or Ipv6 stack,for both Identity and chunked transfer mode,when the last data has been parsed by HttpLib, the HttpInstance->NextMsg pointer should point a correct location.Now after the first  successful httpboot for ipv4 or ipv6,the HttpInstance->NextMsgpoint the character after the last byte, it may be a bad buffer if we don't receive another HttpHeader, so if call a 2nd httpboot, the wrong NextMsg pointer will cause the httpboot fail, so we need to check this case in HttpBodyParserCallback function in the first http boot process.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Zhang Lubo <lubo.zhang@intel.com>
Reviewed-by: Fu Siyuan <siyuan.fu@intel.com>
Reviewed-by: Ye Ting <ting.ye@intel.com>
Reviewed-by: Wu Jiaxin <jiaxin.wu@intel.com>

git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@19423 6f19259b-4bc3-4df7-8a09-765794883524


Thanks,
Naveen

-----Original Message-----
From: Zhang, Lubo [mailto:lubo.zhang@intel.com] 
Sent: Wednesday, August 24, 2016 2:13 PM
To: Santhapur Naveen
Cc: edk2-devel@lists.01.org
Subject: RE: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hi Naveen
     Could you list the patch name or Git SHA value which fixed the 2nd HttpBootfail issue. On my side, the SVN version 24729 is not exist.

Thanks 
Lubo 

-----Original Message-----
From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Santhapur Naveen
Sent: Wednesday, August 24, 2016 3:40 PM
To: edk2-devel@lists.01.org
Subject: Re: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hello,

          Revision 24729 of edk2 says 2nd HttpBoot fail issue has been fixed. So, I have taken the changes from the file NetworkPkg/HttpDxe/HttpImpl.c of the revsision 24729 and still the 2nd  HttpBoot fails. Please let me know if I have missed any.

P.S: The first http boot happens successfully though. (downloading a Shell.efi)

Thanks,
Naveen
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel


^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: 2nd HttpBoot fails even after upgrading to revision 24729
  2016-08-24 10:52     ` Zhang, Lubo
@ 2016-08-24 11:09       ` Santhapur Naveen
  2016-08-25  1:49         ` Zhang, Lubo
  0 siblings, 1 reply; 12+ messages in thread
From: Santhapur Naveen @ 2016-08-24 11:09 UTC (permalink / raw)
  To: Zhang, Lubo; +Cc: edk2-devel@lists.01.org

Hi Lubo,

	Can you please provide me any reference revision number to download and check the same?

Thank you,
Naveen

-----Original Message-----
From: Zhang, Lubo [mailto:lubo.zhang@intel.com] 
Sent: Wednesday, August 24, 2016 4:23 PM
To: Santhapur Naveen
Cc: edk2-devel@lists.01.org
Subject: RE: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hi Naveen
      I have checked the issue you mentioned. On my side, I updated the code to latest and do some test on NT32 and Denlow platform, both the first http boot and 2nd are successful. 
      Until now , we have fixed few bugs which may lead the 2nd http boot fail. As for the patch you mentioned is one case to fix the bug, So I suggest you to update the code firstly and at least make sure the HttpDxe , HttpBootDxe and DxeHttpLib are latest. 
   
     If any result, Pls let me know.

Best Regards
Lubo 
     

-----Original Message-----
From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Santhapur Naveen
Sent: Wednesday, August 24, 2016 4:53 PM
To: Zhang, Lubo <lubo.zhang@intel.com>
Cc: edk2-devel@lists.01.org
Subject: Re: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hi Lubo,

	Please find the following details that may help.

NetworkPkg:Fix a bug the 2nd httpboot fail issue.

Httpboot over Ipv4 or Ipv6 stack,for both Identity and chunked transfer mode,when the last data has been parsed by HttpLib, the HttpInstance->NextMsg pointer should point a correct location.Now after the first  successful httpboot for ipv4 or ipv6,the HttpInstance->NextMsgpoint the character after the last byte, it may be a bad buffer if we don't receive another HttpHeader, so if call a 2nd httpboot, the wrong NextMsg pointer will cause the httpboot fail, so we need to check this case in HttpBodyParserCallback function in the first http boot process.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Zhang Lubo <lubo.zhang@intel.com>
Reviewed-by: Fu Siyuan <siyuan.fu@intel.com>
Reviewed-by: Ye Ting <ting.ye@intel.com>
Reviewed-by: Wu Jiaxin <jiaxin.wu@intel.com>

git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@19423 6f19259b-4bc3-4df7-8a09-765794883524


Thanks,
Naveen

-----Original Message-----
From: Zhang, Lubo [mailto:lubo.zhang@intel.com] 
Sent: Wednesday, August 24, 2016 2:13 PM
To: Santhapur Naveen
Cc: edk2-devel@lists.01.org
Subject: RE: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hi Naveen
     Could you list the patch name or Git SHA value which fixed the 2nd HttpBootfail issue. On my side, the SVN version 24729 is not exist.

Thanks 
Lubo 

-----Original Message-----
From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Santhapur Naveen
Sent: Wednesday, August 24, 2016 3:40 PM
To: edk2-devel@lists.01.org
Subject: Re: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hello,

          Revision 24729 of edk2 says 2nd HttpBoot fail issue has been fixed. So, I have taken the changes from the file NetworkPkg/HttpDxe/HttpImpl.c of the revsision 24729 and still the 2nd  HttpBoot fails. Please let me know if I have missed any.

P.S: The first http boot happens successfully though. (downloading a Shell.efi)

Thanks,
Naveen
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel


^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: 2nd HttpBoot fails even after upgrading to revision 24729
  2016-08-24 11:09       ` Santhapur Naveen
@ 2016-08-25  1:49         ` Zhang, Lubo
  2016-08-25  9:09           ` Santhapur Naveen
  2016-08-25  9:13           ` Santhapur Naveen
  0 siblings, 2 replies; 12+ messages in thread
From: Zhang, Lubo @ 2016-08-25  1:49 UTC (permalink / raw)
  To: Santhapur Naveen; +Cc: edk2-devel@lists.01.org

Hi Naveen
       I update the  SVN and now the revision is 22467.
       The HttpBootDxe revision is 22104,patch is - Fix IPv6 HTTPClient vendor class data
       The HttpDxe revision is 21740, patch is - NetworkPkg: Fix typos in comments.
       But I suggest you to download  the https://github.com/tianocore/edk2, it is latest edk2 code.

Best Regards
Lubo

-----Original Message-----
From: Santhapur Naveen [mailto:naveens@amiindia.co.in] 
Sent: Wednesday, August 24, 2016 7:10 PM
To: Zhang, Lubo <lubo.zhang@intel.com>
Cc: edk2-devel@lists.01.org
Subject: RE: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hi Lubo,

	Can you please provide me any reference revision number to download and check the same?

Thank you,
Naveen

-----Original Message-----
From: Zhang, Lubo [mailto:lubo.zhang@intel.com] 
Sent: Wednesday, August 24, 2016 4:23 PM
To: Santhapur Naveen
Cc: edk2-devel@lists.01.org
Subject: RE: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hi Naveen
      I have checked the issue you mentioned. On my side, I updated the code to latest and do some test on NT32 and Denlow platform, both the first http boot and 2nd are successful. 
      Until now , we have fixed few bugs which may lead the 2nd http boot fail. As for the patch you mentioned is one case to fix the bug, So I suggest you to update the code firstly and at least make sure the HttpDxe , HttpBootDxe and DxeHttpLib are latest. 
   
     If any result, Pls let me know.

Best Regards
Lubo 
     

-----Original Message-----
From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Santhapur Naveen
Sent: Wednesday, August 24, 2016 4:53 PM
To: Zhang, Lubo <lubo.zhang@intel.com>
Cc: edk2-devel@lists.01.org
Subject: Re: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hi Lubo,

	Please find the following details that may help.

NetworkPkg:Fix a bug the 2nd httpboot fail issue.

Httpboot over Ipv4 or Ipv6 stack,for both Identity and chunked transfer mode,when the last data has been parsed by HttpLib, the HttpInstance->NextMsg pointer should point a correct location.Now after the first  successful httpboot for ipv4 or ipv6,the HttpInstance->NextMsgpoint the character after the last byte, it may be a bad buffer if we don't receive another HttpHeader, so if call a 2nd httpboot, the wrong NextMsg pointer will cause the httpboot fail, so we need to check this case in HttpBodyParserCallback function in the first http boot process.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Zhang Lubo <lubo.zhang@intel.com>
Reviewed-by: Fu Siyuan <siyuan.fu@intel.com>
Reviewed-by: Ye Ting <ting.ye@intel.com>
Reviewed-by: Wu Jiaxin <jiaxin.wu@intel.com>

git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@19423 6f19259b-4bc3-4df7-8a09-765794883524


Thanks,
Naveen

-----Original Message-----
From: Zhang, Lubo [mailto:lubo.zhang@intel.com] 
Sent: Wednesday, August 24, 2016 2:13 PM
To: Santhapur Naveen
Cc: edk2-devel@lists.01.org
Subject: RE: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hi Naveen
     Could you list the patch name or Git SHA value which fixed the 2nd HttpBootfail issue. On my side, the SVN version 24729 is not exist.

Thanks 
Lubo 

-----Original Message-----
From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Santhapur Naveen
Sent: Wednesday, August 24, 2016 3:40 PM
To: edk2-devel@lists.01.org
Subject: Re: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hello,

          Revision 24729 of edk2 says 2nd HttpBoot fail issue has been fixed. So, I have taken the changes from the file NetworkPkg/HttpDxe/HttpImpl.c of the revsision 24729 and still the 2nd  HttpBoot fails. Please let me know if I have missed any.

P.S: The first http boot happens successfully though. (downloading a Shell.efi)

Thanks,
Naveen
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel


^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: 2nd HttpBoot fails even after upgrading to revision 24729
  2016-08-25  1:49         ` Zhang, Lubo
@ 2016-08-25  9:09           ` Santhapur Naveen
  2016-08-25  9:48             ` Zhang, Lubo
  2016-08-25  9:13           ` Santhapur Naveen
  1 sibling, 1 reply; 12+ messages in thread
From: Santhapur Naveen @ 2016-08-25  9:09 UTC (permalink / raw)
  To: Zhang, Lubo; +Cc: edk2-devel@lists.01.org

Hi Lubo,

	I believe the way we refer the version numbers seem different. I use tortoise SVN to get the details of the edk2. When I checked the revision numbers 22104 and 21740 for HttpBootDxe and HttpDxe drivers respectively, I couldn't find anything. Please refer the attachments. By any means is it possible for you to provide the way you refer the revision details?

	However, I've downloaded the latest source from the URL you have provided and I'm trying to verify the same now. Will update you my result.

Thank you,
Naveen

-----Original Message-----
From: Zhang, Lubo [mailto:lubo.zhang@intel.com] 
Sent: Thursday, August 25, 2016 7:19 AM
To: Santhapur Naveen
Cc: edk2-devel@lists.01.org
Subject: RE: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hi Naveen
       I update the  SVN and now the revision is 22467.
       The HttpBootDxe revision is 22104,patch is - Fix IPv6 HTTPClient vendor class data
       The HttpDxe revision is 21740, patch is - NetworkPkg: Fix typos in comments.
       But I suggest you to download  the https://github.com/tianocore/edk2, it is latest edk2 code.

Best Regards
Lubo

-----Original Message-----
From: Santhapur Naveen [mailto:naveens@amiindia.co.in] 
Sent: Wednesday, August 24, 2016 7:10 PM
To: Zhang, Lubo <lubo.zhang@intel.com>
Cc: edk2-devel@lists.01.org
Subject: RE: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hi Lubo,

	Can you please provide me any reference revision number to download and check the same?

Thank you,
Naveen

-----Original Message-----
From: Zhang, Lubo [mailto:lubo.zhang@intel.com] 
Sent: Wednesday, August 24, 2016 4:23 PM
To: Santhapur Naveen
Cc: edk2-devel@lists.01.org
Subject: RE: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hi Naveen
      I have checked the issue you mentioned. On my side, I updated the code to latest and do some test on NT32 and Denlow platform, both the first http boot and 2nd are successful. 
      Until now , we have fixed few bugs which may lead the 2nd http boot fail. As for the patch you mentioned is one case to fix the bug, So I suggest you to update the code firstly and at least make sure the HttpDxe , HttpBootDxe and DxeHttpLib are latest. 
   
     If any result, Pls let me know.

Best Regards
Lubo 
     

-----Original Message-----
From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Santhapur Naveen
Sent: Wednesday, August 24, 2016 4:53 PM
To: Zhang, Lubo <lubo.zhang@intel.com>
Cc: edk2-devel@lists.01.org
Subject: Re: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hi Lubo,

	Please find the following details that may help.

NetworkPkg:Fix a bug the 2nd httpboot fail issue.

Httpboot over Ipv4 or Ipv6 stack,for both Identity and chunked transfer mode,when the last data has been parsed by HttpLib, the HttpInstance->NextMsg pointer should point a correct location.Now after the first  successful httpboot for ipv4 or ipv6,the HttpInstance->NextMsgpoint the character after the last byte, it may be a bad buffer if we don't receive another HttpHeader, so if call a 2nd httpboot, the wrong NextMsg pointer will cause the httpboot fail, so we need to check this case in HttpBodyParserCallback function in the first http boot process.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Zhang Lubo <lubo.zhang@intel.com>
Reviewed-by: Fu Siyuan <siyuan.fu@intel.com>
Reviewed-by: Ye Ting <ting.ye@intel.com>
Reviewed-by: Wu Jiaxin <jiaxin.wu@intel.com>

git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@19423 6f19259b-4bc3-4df7-8a09-765794883524


Thanks,
Naveen

-----Original Message-----
From: Zhang, Lubo [mailto:lubo.zhang@intel.com] 
Sent: Wednesday, August 24, 2016 2:13 PM
To: Santhapur Naveen
Cc: edk2-devel@lists.01.org
Subject: RE: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hi Naveen
     Could you list the patch name or Git SHA value which fixed the 2nd HttpBootfail issue. On my side, the SVN version 24729 is not exist.

Thanks 
Lubo 

-----Original Message-----
From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Santhapur Naveen
Sent: Wednesday, August 24, 2016 3:40 PM
To: edk2-devel@lists.01.org
Subject: Re: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hello,

          Revision 24729 of edk2 says 2nd HttpBoot fail issue has been fixed. So, I have taken the changes from the file NetworkPkg/HttpDxe/HttpImpl.c of the revsision 24729 and still the 2nd  HttpBoot fails. Please let me know if I have missed any.

P.S: The first http boot happens successfully though. (downloading a Shell.efi)

Thanks,
Naveen
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel


^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: 2nd HttpBoot fails even after upgrading to revision 24729
  2016-08-25  1:49         ` Zhang, Lubo
  2016-08-25  9:09           ` Santhapur Naveen
@ 2016-08-25  9:13           ` Santhapur Naveen
  1 sibling, 0 replies; 12+ messages in thread
From: Santhapur Naveen @ 2016-08-25  9:13 UTC (permalink / raw)
  To: Zhang, Lubo; +Cc: edk2-devel@lists.01.org

Hi Lubo,

	I believe the way we refer the version numbers seem different. I use tortoise SVN to get the details of the edk2. When I checked the revision numbers 22104 and 21740 for HttpBootDxe and HttpDxe drivers respectively, I couldn't find anything. Please refer the attachment. By any means is it possible for you to provide the way you refer the revision details?

	However, I've downloaded the latest source from the URL you have provided and I'm trying to verify the same now. Will update you my result.

Thank you,
Naveen

-----Original Message-----
From: Zhang, Lubo [mailto:lubo.zhang@intel.com] 
Sent: Thursday, August 25, 2016 7:19 AM
To: Santhapur Naveen
Cc: edk2-devel@lists.01.org
Subject: RE: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hi Naveen
       I update the  SVN and now the revision is 22467.
       The HttpBootDxe revision is 22104,patch is - Fix IPv6 HTTPClient vendor class data
       The HttpDxe revision is 21740, patch is - NetworkPkg: Fix typos in comments.
       But I suggest you to download  the https://github.com/tianocore/edk2, it is latest edk2 code.

Best Regards
Lubo

-----Original Message-----
From: Santhapur Naveen [mailto:naveens@amiindia.co.in] 
Sent: Wednesday, August 24, 2016 7:10 PM
To: Zhang, Lubo <lubo.zhang@intel.com>
Cc: edk2-devel@lists.01.org
Subject: RE: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hi Lubo,

	Can you please provide me any reference revision number to download and check the same?

Thank you,
Naveen

-----Original Message-----
From: Zhang, Lubo [mailto:lubo.zhang@intel.com] 
Sent: Wednesday, August 24, 2016 4:23 PM
To: Santhapur Naveen
Cc: edk2-devel@lists.01.org
Subject: RE: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hi Naveen
      I have checked the issue you mentioned. On my side, I updated the code to latest and do some test on NT32 and Denlow platform, both the first http boot and 2nd are successful. 
      Until now , we have fixed few bugs which may lead the 2nd http boot fail. As for the patch you mentioned is one case to fix the bug, So I suggest you to update the code firstly and at least make sure the HttpDxe , HttpBootDxe and DxeHttpLib are latest. 
   
     If any result, Pls let me know.

Best Regards
Lubo 
     

-----Original Message-----
From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Santhapur Naveen
Sent: Wednesday, August 24, 2016 4:53 PM
To: Zhang, Lubo <lubo.zhang@intel.com>
Cc: edk2-devel@lists.01.org
Subject: Re: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hi Lubo,

	Please find the following details that may help.

NetworkPkg:Fix a bug the 2nd httpboot fail issue.

Httpboot over Ipv4 or Ipv6 stack,for both Identity and chunked transfer mode,when the last data has been parsed by HttpLib, the HttpInstance->NextMsg pointer should point a correct location.Now after the first  successful httpboot for ipv4 or ipv6,the HttpInstance->NextMsgpoint the character after the last byte, it may be a bad buffer if we don't receive another HttpHeader, so if call a 2nd httpboot, the wrong NextMsg pointer will cause the httpboot fail, so we need to check this case in HttpBodyParserCallback function in the first http boot process.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Zhang Lubo <lubo.zhang@intel.com>
Reviewed-by: Fu Siyuan <siyuan.fu@intel.com>
Reviewed-by: Ye Ting <ting.ye@intel.com>
Reviewed-by: Wu Jiaxin <jiaxin.wu@intel.com>

git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@19423 6f19259b-4bc3-4df7-8a09-765794883524


Thanks,
Naveen

-----Original Message-----
From: Zhang, Lubo [mailto:lubo.zhang@intel.com] 
Sent: Wednesday, August 24, 2016 2:13 PM
To: Santhapur Naveen
Cc: edk2-devel@lists.01.org
Subject: RE: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hi Naveen
     Could you list the patch name or Git SHA value which fixed the 2nd HttpBootfail issue. On my side, the SVN version 24729 is not exist.

Thanks 
Lubo 

-----Original Message-----
From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Santhapur Naveen
Sent: Wednesday, August 24, 2016 3:40 PM
To: edk2-devel@lists.01.org
Subject: Re: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hello,

          Revision 24729 of edk2 says 2nd HttpBoot fail issue has been fixed. So, I have taken the changes from the file NetworkPkg/HttpDxe/HttpImpl.c of the revsision 24729 and still the 2nd  HttpBoot fails. Please let me know if I have missed any.

P.S: The first http boot happens successfully though. (downloading a Shell.efi)

Thanks,
Naveen
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel


^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: 2nd HttpBoot fails even after upgrading to revision 24729
  2016-08-25  9:09           ` Santhapur Naveen
@ 2016-08-25  9:48             ` Zhang, Lubo
  2016-08-29  6:26               ` Santhapur Naveen
  0 siblings, 1 reply; 12+ messages in thread
From: Zhang, Lubo @ 2016-08-25  9:48 UTC (permalink / raw)
  To: Santhapur Naveen; +Cc: edk2-devel@lists.01.org

  Yes, the SVN version number I provide is not consistent with you. 
  The attachment is latest patch log details of the GitHub, you can make a  contrast when you download the code. And then do some test.
  
Best Regards
Lubo
 
  

-----Original Message-----
From: Santhapur Naveen [mailto:naveens@amiindia.co.in] 
Sent: Thursday, August 25, 2016 5:09 PM
To: Zhang, Lubo <lubo.zhang@intel.com>
Cc: edk2-devel@lists.01.org
Subject: RE: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hi Lubo,

	I believe the way we refer the version numbers seem different. I use tortoise SVN to get the details of the edk2. When I checked the revision numbers 22104 and 21740 for HttpBootDxe and HttpDxe drivers respectively, I couldn't find anything. Please refer the attachments. By any means is it possible for you to provide the way you refer the revision details?

	However, I've downloaded the latest source from the URL you have provided and I'm trying to verify the same now. Will update you my result.

Thank you,
Naveen

-----Original Message-----
From: Zhang, Lubo [mailto:lubo.zhang@intel.com] 
Sent: Thursday, August 25, 2016 7:19 AM
To: Santhapur Naveen
Cc: edk2-devel@lists.01.org
Subject: RE: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hi Naveen
       I update the  SVN and now the revision is 22467.
       The HttpBootDxe revision is 22104,patch is - Fix IPv6 HTTPClient vendor class data
       The HttpDxe revision is 21740, patch is - NetworkPkg: Fix typos in comments.
       But I suggest you to download  the https://github.com/tianocore/edk2, it is latest edk2 code.

Best Regards
Lubo

-----Original Message-----
From: Santhapur Naveen [mailto:naveens@amiindia.co.in] 
Sent: Wednesday, August 24, 2016 7:10 PM
To: Zhang, Lubo <lubo.zhang@intel.com>
Cc: edk2-devel@lists.01.org
Subject: RE: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hi Lubo,

	Can you please provide me any reference revision number to download and check the same?

Thank you,
Naveen

-----Original Message-----
From: Zhang, Lubo [mailto:lubo.zhang@intel.com] 
Sent: Wednesday, August 24, 2016 4:23 PM
To: Santhapur Naveen
Cc: edk2-devel@lists.01.org
Subject: RE: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hi Naveen
      I have checked the issue you mentioned. On my side, I updated the code to latest and do some test on NT32 and Denlow platform, both the first http boot and 2nd are successful. 
      Until now , we have fixed few bugs which may lead the 2nd http boot fail. As for the patch you mentioned is one case to fix the bug, So I suggest you to update the code firstly and at least make sure the HttpDxe , HttpBootDxe and DxeHttpLib are latest. 
   
     If any result, Pls let me know.

Best Regards
Lubo 
     

-----Original Message-----
From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Santhapur Naveen
Sent: Wednesday, August 24, 2016 4:53 PM
To: Zhang, Lubo <lubo.zhang@intel.com>
Cc: edk2-devel@lists.01.org
Subject: Re: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hi Lubo,

	Please find the following details that may help.

NetworkPkg:Fix a bug the 2nd httpboot fail issue.

Httpboot over Ipv4 or Ipv6 stack,for both Identity and chunked transfer mode,when the last data has been parsed by HttpLib, the HttpInstance->NextMsg pointer should point a correct location.Now after the first  successful httpboot for ipv4 or ipv6,the HttpInstance->NextMsgpoint the character after the last byte, it may be a bad buffer if we don't receive another HttpHeader, so if call a 2nd httpboot, the wrong NextMsg pointer will cause the httpboot fail, so we need to check this case in HttpBodyParserCallback function in the first http boot process.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Zhang Lubo <lubo.zhang@intel.com>
Reviewed-by: Fu Siyuan <siyuan.fu@intel.com>
Reviewed-by: Ye Ting <ting.ye@intel.com>
Reviewed-by: Wu Jiaxin <jiaxin.wu@intel.com>

git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@19423 6f19259b-4bc3-4df7-8a09-765794883524


Thanks,
Naveen

-----Original Message-----
From: Zhang, Lubo [mailto:lubo.zhang@intel.com] 
Sent: Wednesday, August 24, 2016 2:13 PM
To: Santhapur Naveen
Cc: edk2-devel@lists.01.org
Subject: RE: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hi Naveen
     Could you list the patch name or Git SHA value which fixed the 2nd HttpBootfail issue. On my side, the SVN version 24729 is not exist.

Thanks 
Lubo 

-----Original Message-----
From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Santhapur Naveen
Sent: Wednesday, August 24, 2016 3:40 PM
To: edk2-devel@lists.01.org
Subject: Re: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hello,

          Revision 24729 of edk2 says 2nd HttpBoot fail issue has been fixed. So, I have taken the changes from the file NetworkPkg/HttpDxe/HttpImpl.c of the revsision 24729 and still the 2nd  HttpBoot fails. Please let me know if I have missed any.

P.S: The first http boot happens successfully though. (downloading a Shell.efi)

Thanks,
Naveen
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel


^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: 2nd HttpBoot fails even after upgrading to revision 24729
  2016-08-25  9:48             ` Zhang, Lubo
@ 2016-08-29  6:26               ` Santhapur Naveen
  2016-08-29  7:18                 ` Zhang, Lubo
  0 siblings, 1 reply; 12+ messages in thread
From: Santhapur Naveen @ 2016-08-29  6:26 UTC (permalink / raw)
  To: Zhang, Lubo; +Cc: edk2-devel@lists.01.org

Hi Lubo,

	I've taken the source from the location you have specified and checked. But I still get the same failure. When I select the same HTTP boot option the 2nd time, control goes to next boot option available. I guess the problem could be on my side only. I'll look into it.

	Thank you very much for your time and effort.

Thanks,
Naveen

-----Original Message-----
From: Zhang, Lubo [mailto:lubo.zhang@intel.com] 
Sent: Thursday, August 25, 2016 3:19 PM
To: Santhapur Naveen
Cc: edk2-devel@lists.01.org
Subject: RE: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

  Yes, the SVN version number I provide is not consistent with you. 
  The attachment is latest patch log details of the GitHub, you can make a  contrast when you download the code. And then do some test.
  
Best Regards
Lubo
 
  

-----Original Message-----
From: Santhapur Naveen [mailto:naveens@amiindia.co.in] 
Sent: Thursday, August 25, 2016 5:09 PM
To: Zhang, Lubo <lubo.zhang@intel.com>
Cc: edk2-devel@lists.01.org
Subject: RE: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hi Lubo,

	I believe the way we refer the version numbers seem different. I use tortoise SVN to get the details of the edk2. When I checked the revision numbers 22104 and 21740 for HttpBootDxe and HttpDxe drivers respectively, I couldn't find anything. Please refer the attachments. By any means is it possible for you to provide the way you refer the revision details?

	However, I've downloaded the latest source from the URL you have provided and I'm trying to verify the same now. Will update you my result.

Thank you,
Naveen

-----Original Message-----
From: Zhang, Lubo [mailto:lubo.zhang@intel.com] 
Sent: Thursday, August 25, 2016 7:19 AM
To: Santhapur Naveen
Cc: edk2-devel@lists.01.org
Subject: RE: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hi Naveen
       I update the  SVN and now the revision is 22467.
       The HttpBootDxe revision is 22104,patch is - Fix IPv6 HTTPClient vendor class data
       The HttpDxe revision is 21740, patch is - NetworkPkg: Fix typos in comments.
       But I suggest you to download  the https://github.com/tianocore/edk2, it is latest edk2 code.

Best Regards
Lubo

-----Original Message-----
From: Santhapur Naveen [mailto:naveens@amiindia.co.in] 
Sent: Wednesday, August 24, 2016 7:10 PM
To: Zhang, Lubo <lubo.zhang@intel.com>
Cc: edk2-devel@lists.01.org
Subject: RE: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hi Lubo,

	Can you please provide me any reference revision number to download and check the same?

Thank you,
Naveen

-----Original Message-----
From: Zhang, Lubo [mailto:lubo.zhang@intel.com] 
Sent: Wednesday, August 24, 2016 4:23 PM
To: Santhapur Naveen
Cc: edk2-devel@lists.01.org
Subject: RE: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hi Naveen
      I have checked the issue you mentioned. On my side, I updated the code to latest and do some test on NT32 and Denlow platform, both the first http boot and 2nd are successful. 
      Until now , we have fixed few bugs which may lead the 2nd http boot fail. As for the patch you mentioned is one case to fix the bug, So I suggest you to update the code firstly and at least make sure the HttpDxe , HttpBootDxe and DxeHttpLib are latest. 
   
     If any result, Pls let me know.

Best Regards
Lubo 
     

-----Original Message-----
From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Santhapur Naveen
Sent: Wednesday, August 24, 2016 4:53 PM
To: Zhang, Lubo <lubo.zhang@intel.com>
Cc: edk2-devel@lists.01.org
Subject: Re: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hi Lubo,

	Please find the following details that may help.

NetworkPkg:Fix a bug the 2nd httpboot fail issue.

Httpboot over Ipv4 or Ipv6 stack,for both Identity and chunked transfer mode,when the last data has been parsed by HttpLib, the HttpInstance->NextMsg pointer should point a correct location.Now after the first  successful httpboot for ipv4 or ipv6,the HttpInstance->NextMsgpoint the character after the last byte, it may be a bad buffer if we don't receive another HttpHeader, so if call a 2nd httpboot, the wrong NextMsg pointer will cause the httpboot fail, so we need to check this case in HttpBodyParserCallback function in the first http boot process.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Zhang Lubo <lubo.zhang@intel.com>
Reviewed-by: Fu Siyuan <siyuan.fu@intel.com>
Reviewed-by: Ye Ting <ting.ye@intel.com>
Reviewed-by: Wu Jiaxin <jiaxin.wu@intel.com>

git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@19423 6f19259b-4bc3-4df7-8a09-765794883524


Thanks,
Naveen

-----Original Message-----
From: Zhang, Lubo [mailto:lubo.zhang@intel.com] 
Sent: Wednesday, August 24, 2016 2:13 PM
To: Santhapur Naveen
Cc: edk2-devel@lists.01.org
Subject: RE: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hi Naveen
     Could you list the patch name or Git SHA value which fixed the 2nd HttpBootfail issue. On my side, the SVN version 24729 is not exist.

Thanks 
Lubo 

-----Original Message-----
From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Santhapur Naveen
Sent: Wednesday, August 24, 2016 3:40 PM
To: edk2-devel@lists.01.org
Subject: Re: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hello,

          Revision 24729 of edk2 says 2nd HttpBoot fail issue has been fixed. So, I have taken the changes from the file NetworkPkg/HttpDxe/HttpImpl.c of the revsision 24729 and still the 2nd  HttpBoot fails. Please let me know if I have missed any.

P.S: The first http boot happens successfully though. (downloading a Shell.efi)

Thanks,
Naveen
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel


^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: 2nd HttpBoot fails even after upgrading to revision 24729
  2016-08-29  6:26               ` Santhapur Naveen
@ 2016-08-29  7:18                 ` Zhang, Lubo
  0 siblings, 0 replies; 12+ messages in thread
From: Zhang, Lubo @ 2016-08-29  7:18 UTC (permalink / raw)
  To: Santhapur Naveen; +Cc: edk2-devel@lists.01.org

What do you mean control goes to next boot option?  Will it go to PXE boot when you select the 2nd http boot. If was, you just need to update the whole edk2 code. This issue related to the BDS and we already fixed long before,

Best regards
Lubo 

-----Original Message-----
From: Santhapur Naveen [mailto:naveens@amiindia.co.in] 
Sent: Monday, August 29, 2016 2:27 PM
To: Zhang, Lubo <lubo.zhang@intel.com>
Cc: edk2-devel@lists.01.org
Subject: RE: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hi Lubo,

	I've taken the source from the location you have specified and checked. But I still get the same failure. When I select the same HTTP boot option the 2nd time, control goes to next boot option available. I guess the problem could be on my side only. I'll look into it.

	Thank you very much for your time and effort.

Thanks,
Naveen

-----Original Message-----
From: Zhang, Lubo [mailto:lubo.zhang@intel.com] 
Sent: Thursday, August 25, 2016 3:19 PM
To: Santhapur Naveen
Cc: edk2-devel@lists.01.org
Subject: RE: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

  Yes, the SVN version number I provide is not consistent with you. 
  The attachment is latest patch log details of the GitHub, you can make a  contrast when you download the code. And then do some test.
  
Best Regards
Lubo
 
  

-----Original Message-----
From: Santhapur Naveen [mailto:naveens@amiindia.co.in] 
Sent: Thursday, August 25, 2016 5:09 PM
To: Zhang, Lubo <lubo.zhang@intel.com>
Cc: edk2-devel@lists.01.org
Subject: RE: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hi Lubo,

	I believe the way we refer the version numbers seem different. I use tortoise SVN to get the details of the edk2. When I checked the revision numbers 22104 and 21740 for HttpBootDxe and HttpDxe drivers respectively, I couldn't find anything. Please refer the attachments. By any means is it possible for you to provide the way you refer the revision details?

	However, I've downloaded the latest source from the URL you have provided and I'm trying to verify the same now. Will update you my result.

Thank you,
Naveen

-----Original Message-----
From: Zhang, Lubo [mailto:lubo.zhang@intel.com] 
Sent: Thursday, August 25, 2016 7:19 AM
To: Santhapur Naveen
Cc: edk2-devel@lists.01.org
Subject: RE: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hi Naveen
       I update the  SVN and now the revision is 22467.
       The HttpBootDxe revision is 22104,patch is - Fix IPv6 HTTPClient vendor class data
       The HttpDxe revision is 21740, patch is - NetworkPkg: Fix typos in comments.
       But I suggest you to download  the https://github.com/tianocore/edk2, it is latest edk2 code.

Best Regards
Lubo

-----Original Message-----
From: Santhapur Naveen [mailto:naveens@amiindia.co.in] 
Sent: Wednesday, August 24, 2016 7:10 PM
To: Zhang, Lubo <lubo.zhang@intel.com>
Cc: edk2-devel@lists.01.org
Subject: RE: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hi Lubo,

	Can you please provide me any reference revision number to download and check the same?

Thank you,
Naveen

-----Original Message-----
From: Zhang, Lubo [mailto:lubo.zhang@intel.com] 
Sent: Wednesday, August 24, 2016 4:23 PM
To: Santhapur Naveen
Cc: edk2-devel@lists.01.org
Subject: RE: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hi Naveen
      I have checked the issue you mentioned. On my side, I updated the code to latest and do some test on NT32 and Denlow platform, both the first http boot and 2nd are successful. 
      Until now , we have fixed few bugs which may lead the 2nd http boot fail. As for the patch you mentioned is one case to fix the bug, So I suggest you to update the code firstly and at least make sure the HttpDxe , HttpBootDxe and DxeHttpLib are latest. 
   
     If any result, Pls let me know.

Best Regards
Lubo 
     

-----Original Message-----
From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Santhapur Naveen
Sent: Wednesday, August 24, 2016 4:53 PM
To: Zhang, Lubo <lubo.zhang@intel.com>
Cc: edk2-devel@lists.01.org
Subject: Re: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hi Lubo,

	Please find the following details that may help.

NetworkPkg:Fix a bug the 2nd httpboot fail issue.

Httpboot over Ipv4 or Ipv6 stack,for both Identity and chunked transfer mode,when the last data has been parsed by HttpLib, the HttpInstance->NextMsg pointer should point a correct location.Now after the first  successful httpboot for ipv4 or ipv6,the HttpInstance->NextMsgpoint the character after the last byte, it may be a bad buffer if we don't receive another HttpHeader, so if call a 2nd httpboot, the wrong NextMsg pointer will cause the httpboot fail, so we need to check this case in HttpBodyParserCallback function in the first http boot process.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Zhang Lubo <lubo.zhang@intel.com>
Reviewed-by: Fu Siyuan <siyuan.fu@intel.com>
Reviewed-by: Ye Ting <ting.ye@intel.com>
Reviewed-by: Wu Jiaxin <jiaxin.wu@intel.com>

git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@19423 6f19259b-4bc3-4df7-8a09-765794883524


Thanks,
Naveen

-----Original Message-----
From: Zhang, Lubo [mailto:lubo.zhang@intel.com] 
Sent: Wednesday, August 24, 2016 2:13 PM
To: Santhapur Naveen
Cc: edk2-devel@lists.01.org
Subject: RE: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hi Naveen
     Could you list the patch name or Git SHA value which fixed the 2nd HttpBootfail issue. On my side, the SVN version 24729 is not exist.

Thanks 
Lubo 

-----Original Message-----
From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Santhapur Naveen
Sent: Wednesday, August 24, 2016 3:40 PM
To: edk2-devel@lists.01.org
Subject: Re: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729

Hello,

          Revision 24729 of edk2 says 2nd HttpBoot fail issue has been fixed. So, I have taken the changes from the file NetworkPkg/HttpDxe/HttpImpl.c of the revsision 24729 and still the 2nd  HttpBoot fails. Please let me know if I have missed any.

P.S: The first http boot happens successfully though. (downloading a Shell.efi)

Thanks,
Naveen
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel


^ permalink raw reply	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2016-08-29  7:18 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-08-24  7:39 2nd HttpBoot fails even after upgrading to revision 24729 Santhapur Naveen
2016-08-24  8:42 ` Zhang, Lubo
2016-08-24  8:52   ` Santhapur Naveen
2016-08-24  9:09     ` Zhang, Lubo
2016-08-24 10:52     ` Zhang, Lubo
2016-08-24 11:09       ` Santhapur Naveen
2016-08-25  1:49         ` Zhang, Lubo
2016-08-25  9:09           ` Santhapur Naveen
2016-08-25  9:48             ` Zhang, Lubo
2016-08-29  6:26               ` Santhapur Naveen
2016-08-29  7:18                 ` Zhang, Lubo
2016-08-25  9:13           ` Santhapur Naveen

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox