On Mon, Nov 05, 2012 at 03:26:41PM +0000, Jan Beulich wrote: > Header length should be validated for all ACPI tables before accessing > any non-header field. > > The valid flags should also be check, as with it clear there's no point > in trying to go through the rest of the code (and there's no guarantee > that the other table contents are valid/consistent in that case). > > Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx> The length check seems reasonable. However, Matthew Garrett (already CCed) previously suggested to me that this code should not check the "valid" bit, and should instead present the information to userspace if otherwise valid (such as having image_address != 0). Matthew? Out of curiosity, did you encounter a system that requires this patch? - Josh Triplett -- To unsubscribe from this list: send the line "unsubscribe linux-efi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html