On 14/01/2019 16:07, Julien Grall wrote: > Hi Andrew, > > On 14/01/2019 15:17, Andrew Cooper wrote: >>> diff --git a/xen/arch/arm/efi/efi-boot.h b/xen/arch/arm/efi/efi-boot.h >>> index ca655ff..22a86ec 100644 >>> --- a/xen/arch/arm/efi/efi-boot.h >>> +++ b/xen/arch/arm/efi/efi-boot.h >>> @@ -212,7 +212,7 @@ EFI_STATUS __init >>> fdt_add_uefi_nodes(EFI_SYSTEM_TABLE *sys_table, >>> break; >>> type = fdt_getprop(fdt, node, "device_type", &len); >>> - if ( type && strncmp(type, "memory", len) == 0 ) >>> + if ( type && len == 6 && strncmp(type, "memory", 6) == 0 ) > > string property terminates with NUL and is included in the len. So I > don't think this change is correct.
Are you saying that len is 7 here then? ~Andrew _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel