Re: [PATCH 3/3] ACPI: fix acpi_parse_entries_array() so it reports overflow correctly

2016-07-01 Thread Rafael J. Wysocki
On Sat, Jul 2, 2016 at 12:38 AM, Al Stone wrote: > On 07/01/2016 03:54 PM, Rafael J. Wysocki wrote: >> On Fri, Jul 1, 2016 at 11:44 PM, Al Stone wrote: >>> On 07/01/2016 03:40 PM, Rafael J. Wysocki wrote: On Fri, Jul 1, 2016 at 11:21 PM, Al Stone wrote: > The function acpi_parse_entries

Re: [PATCH 3/3] ACPI: fix acpi_parse_entries_array() so it reports overflow correctly

2016-07-01 Thread Al Stone
On 07/01/2016 03:54 PM, Rafael J. Wysocki wrote: > On Fri, Jul 1, 2016 at 11:44 PM, Al Stone wrote: >> On 07/01/2016 03:40 PM, Rafael J. Wysocki wrote: >>> On Fri, Jul 1, 2016 at 11:21 PM, Al Stone wrote: The function acpi_parse_entries_array() has a limiting parameter, max_entries, whi

Re: [PATCH 3/3] ACPI: fix acpi_parse_entries_array() so it reports overflow correctly

2016-07-01 Thread Rafael J. Wysocki
On Fri, Jul 1, 2016 at 11:44 PM, Al Stone wrote: > On 07/01/2016 03:40 PM, Rafael J. Wysocki wrote: >> On Fri, Jul 1, 2016 at 11:21 PM, Al Stone wrote: >>> The function acpi_parse_entries_array() has a limiting parameter, >>> max_entries, which tells the function to stop looking at subtables >>>

Re: [PATCH 3/3] ACPI: fix acpi_parse_entries_array() so it reports overflow correctly

2016-07-01 Thread Al Stone
On 07/01/2016 03:40 PM, Rafael J. Wysocki wrote: > On Fri, Jul 1, 2016 at 11:21 PM, Al Stone wrote: >> The function acpi_parse_entries_array() has a limiting parameter, >> max_entries, which tells the function to stop looking at subtables >> once that limit has been reached. Further, if the limit

Re: [PATCH 3/3] ACPI: fix acpi_parse_entries_array() so it reports overflow correctly

2016-07-01 Thread Rafael J. Wysocki
On Fri, Jul 1, 2016 at 11:21 PM, Al Stone wrote: > The function acpi_parse_entries_array() has a limiting parameter, > max_entries, which tells the function to stop looking at subtables > once that limit has been reached. Further, if the limit is reached, > it is reported. However, the logic is

[PATCH 3/3] ACPI: fix acpi_parse_entries_array() so it reports overflow correctly

2016-07-01 Thread Al Stone
The function acpi_parse_entries_array() has a limiting parameter, max_entries, which tells the function to stop looking at subtables once that limit has been reached. Further, if the limit is reached, it is reported. However, the logic is incorrect in that the loop to examine all subtables will a