Laszlo Ersek <ler...@redhat.com> writes: > On 06/06/13 18:27, Markus Armbruster wrote: >> Signed-off-by: Markus Armbruster <arm...@redhat.com> >> --- >> arch_init.c | 3 ++- >> hw/nvram/fw_cfg.c | 2 +- >> include/sysemu/sysemu.h | 2 +- >> 3 files changed, 4 insertions(+), 3 deletions(-) >> >> diff --git a/arch_init.c b/arch_init.c >> index 5d71870..aa24660 100644 >> --- a/arch_init.c >> +++ b/arch_init.c >> @@ -1029,7 +1029,8 @@ int qemu_uuid_parse(const char *str, uint8_t *uuid) >> return -1; >> } >> #ifdef TARGET_I386 >> - smbios_add_field(1, offsetof(struct smbios_type_1, uuid), 16, uuid); >> + smbios_add_field(1, offsetof(struct smbios_type_1, uuid), >> + sizeof(uuid), uuid); >> #endif >> return 0; >> } > > I believe this is wrong, "uuid" is not an array here but a pointer. I > guess you mistyped "sizeof(qemu_uuid)" as "sizeof(uuid)" in the third arg.
Rats! You're right. Thanks!
