* 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 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
* 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
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