Re: [PATCH] clk: ux500: fix a possible off-by-one in u8500_prcc_reset_base()

From: Linus Walleij
Date: Wed May 18 2022 - 09:39:35 EST

Next message: Miguel Ojeda: "Re: [PATCH v2] binder: convert `BINDER_*` ioctl `#define`s into an `enum`"
Previous message: Eli Cohen: "[PATCH v8 6/6] vdpa/mlx5: Use readers/writers semaphore instead of mutex"
In reply to: Hangyu Hua: "[PATCH] clk: ux500: fix a possible off-by-one in u8500_prcc_reset_base()"
Next in thread: Stephen Boyd: "Re: [PATCH] clk: ux500: fix a possible off-by-one in u8500_prcc_reset_base()"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

On Wed, May 18, 2022 at 8:25 AM Hangyu Hua <hbh25y@xxxxxxxxx> wrote:

> Off-by-one will happen when index == ARRAY_SIZE(ur->base).
>
> Fixes: b14cbdfd467d ("clk: ux500: Add driver for the reset portions of PRCC")
> Signed-off-by: Hangyu Hua <hbh25y@xxxxxxxxx>

Good catch!
Reviewed-by: Linus Walleij <linus.walleij@xxxxxxxxxx>

Yours,
Linus Walleij

Next message: Miguel Ojeda: "Re: [PATCH v2] binder: convert `BINDER_*` ioctl `#define`s into an `enum`"
Previous message: Eli Cohen: "[PATCH v8 6/6] vdpa/mlx5: Use readers/writers semaphore instead of mutex"
In reply to: Hangyu Hua: "[PATCH] clk: ux500: fix a possible off-by-one in u8500_prcc_reset_base()"
Next in thread: Stephen Boyd: "Re: [PATCH] clk: ux500: fix a possible off-by-one in u8500_prcc_reset_base()"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]