On Wed, Oct 23, 2013 at 06:06:46AM +0000, Zheng, Lv wrote: > The last resource should always be ACPI_RESOURCE_TYPE_END_TAG and we > have nsrepair mechanism to ensure that this entry is always there for > resource based properties (there are still 2 kinds of properties > haven't been fixed yet). > > If we do not want to rely on this logic, then we may need to use > buffer->length and pass this one to the acpi_rs_get_aml_length to > avoid potential infinite loop. > I'll draft an ACPICA patch to achieve the latter. Actually couldn't we just change the loop to: while (resource->type != ACPI_RESOURCE_TYPE_END_TAG) { That would be my preference (as an ignorant third party). regards, dan carpenter -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html