On Tue, Jun 27, 2017 at 11:59:17AM -0700, Life is hard, and then you die wrote: > On Tue, Jun 27, 2017 at 11:24:25AM +0200, Lukas Wunner wrote: > > On Tue, Jun 27, 2017 at 01:19:31AM +0000, Zheng, Lv wrote: > > > > On Mon, Jun 26, 2017 at 10:38 AM, Lukas Wunner <lukas@xxxxxxxxx> wrote: > > > > > ACPICA commit c8eac101 (= Linux commit a83019eb9f1f, "ACPICA: Update > > > > > resource descriptor handling") is causing a regression when parsing > > > > > an empty ResourceTemplate: > > > > > > Can it be fixed by using this logic: > > > if (AmlLength < sizeof (AML_RESOURCE_END_TAG) && AmlLength) > > > Instead of the logic introduced in commit a83019eb9f1f? > > > if (AmlLength < sizeof (AML_RESOURCE_END_TAG)) > > > > Unfortunately not, AmlLength is not 0 but 2 in this case > > (= sizeof(struct aml_resource_end_tag)). > > Actually, that will work here (the use of '<' ensures that). Ugh, right, I had missed that the = was gone in Lv's suggestion, sorry. On Tue, Jun 27, 2017 at 05:31:52PM +0200, Rafael J. Wysocki wrote: > So I will drop this commit for now and I will add it again along with > the next release material to avoid triggering regressions in the > meantime. Okay, thanks for the speedy response Rafael and Robert! Lukas -- 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
