Messages in this thread | | | From | Andy Shevchenko <> | Date | Tue, 3 Aug 2021 21:30:17 +0300 | Subject | Re: [PATCH] platform/x86: intel_pmc_core: Fix potential buffer overflows |
| |
On Tue, Aug 3, 2021 at 9:26 PM Andy Shevchenko <andy.shevchenko@gmail.com> wrote: > On Tue, Aug 3, 2021 at 9:21 PM Evgeny Novikov <novikov@ispras.ru> wrote: > > > > It looks like pmc_core_get_low_power_modes() mixes up modes and > > priorities. In addition to invalid behavior, potentially this can > > cause buffer overflows since the driver reads priorities from the > > register and then it uses them as indexes for array lpm_priority > > that can contain 8 elements at most. The patch swaps modes and > > priorities. > > > > Found by Linux Driver Verification project (linuxtesting.org). > > Seems legit.
Hold on, but then it follows with another loop where actually it reads modes by priority index. Can you elaborate what exactly is the problem you think?
-- With Best Regards, Andy Shevchenko
| |