Re: [PATCH] efi: get_memory_map: add sufficient slack for memory descriptors

2015-02-13 Thread Mark Rutland
On Fri, Feb 13, 2015 at 04:04:48PM +, Matt Fleming wrote: On Thu, 12 Feb, at 11:31:02PM, Ard Biesheuvel wrote: Actually, looking again at the original patch, it appears that my analysis was incorrect regarding the possibility that the loop would never terminate. The only thing that

Re: [PATCH] efi: get_memory_map: add sufficient slack for memory descriptors

2015-02-13 Thread Ard Biesheuvel
On 14 Feb 2015, at 00:04, Matt Fleming m...@codeblueprint.co.uk wrote: On Thu, 12 Feb, at 11:31:02PM, Ard Biesheuvel wrote: Actually, looking again at the original patch, it appears that my analysis was incorrect regarding the possibility that the loop would never terminate. The only

Re: [PATCH] efi: get_memory_map: add sufficient slack for memory descriptors

2015-02-13 Thread Matt Fleming
On Thu, 12 Feb, at 11:31:02PM, Ard Biesheuvel wrote: Actually, looking again at the original patch, it appears that my analysis was incorrect regarding the possibility that the loop would never terminate. The only thing that could happen if desc_size sizeof(efi_memory_desc_t) is that you