Re: [PATCH 7/8] test: Fix pointer overrun in dm_test_devm_regmap()

Simon Glass
2022-04-11T18:41:33Z

On Sun, 3 Apr 2022 at 04:39, Andrew Scull <ascull@google.com> wrote:
>
> This tests calls regmap_read() which takes a uint pointer as an output
> parameter. The test was passing a pointer to a u16 which resulted in an
> overflow when the output was written. Fix this by following the
> regmap_read() API and passing a uint pointer instead.
>
> Signed-off-by: Andrew Scull <ascull@google.com>
> Cc: Simon Glass <sjg@chromium.org>
> Cc: Heinrich Schuchardt <xypron.glpk@gmx.de>
> Cc: Jean-Jacques Hiblot <jjhiblot@ti.com>
> Cc: Pratyush Yadav <p.yadav@ti.com>
> ---
> test/dm/regmap.c | 9 ++++-----
> 1 file changed, 4 insertions(+), 5 deletions(-)

Reviewed-by: Simon Glass <sjg@chromium.org>