On Thu, Jul 21, 2016 at 02:40:35AM +0000, Zhang, Lubo wrote: > Yes, the wrong length definition of the ClassIdentifier lead to a wrong > location CopyMem for system architecture when we build the vendor class > option. Thanks for your correction. > Do you have access to check in the code?
No. I don't. I also do not know other alternatives like sending pull request to the git repo instead of sending patch to the list. I am absoutely fine to follow any instrctions here to get the fix committed, just tell me (could via the private mails or so) for any other procedures I should take. > > Best Regards > Lubo > > -----Original Message----- > From: edk2-devel [mailto:[email protected]] On Behalf Of > Michael Chang > Sent: Thursday, July 21, 2016 9:59 AM > To: [email protected] > Subject: [edk2] [PATCH] Fix IPv6 HTTPClient vendor class data > > The size of the ClassIdentifier is apparently wrong in the structure. In my > testing it caused the vendor class data to be displayed as bogus > "HTTPClient:Arch00016x:UNDI00300". After correcting the size the vendor class > data is "HTTPClient:Arch00016:UNDI003000" which looks good to me. Gary reminded me missed out the necessary tags contributing the patch to the project so here it is. :) Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Michael Chang <[email protected]> > > --- > NetworkPkg/HttpBootDxe/HttpBootDhcp6.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/NetworkPkg/HttpBootDxe/HttpBootDhcp6.h > b/NetworkPkg/HttpBootDxe/HttpBootDhcp6.h > index f9eec9d..14d6db0 100644 > --- a/NetworkPkg/HttpBootDxe/HttpBootDhcp6.h > +++ b/NetworkPkg/HttpBootDxe/HttpBootDhcp6.h > @@ -50,7 +50,7 @@ typedef struct { > } HTTP_BOOT_DHCP6_OPTION_ARCH; > > typedef struct { > - UINT8 ClassIdentifier[10]; > + UINT8 ClassIdentifier[11]; > UINT8 ArchitecturePrefix[5]; > UINT8 ArchitectureType[5]; > UINT8 Lit3[1]; > -- > 2.6.6 > > _______________________________________________ > edk2-devel mailing list > [email protected] > https://lists.01.org/mailman/listinfo/edk2-devel _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel
