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?
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. --- 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
