Re: [PATCH] lib/stackdepot: off by one in depot_fetch_stack()

From: Andrey Konovalov
Date: Fri Mar 01 2024 - 14:23:32 EST


On Thu, Feb 29, 2024 at 8:22 AM Dan Carpenter <dan.carpenter@linaroorg> wrote:
>
> I wrote it on top of that patch...

Ah, right, my bad.

> Backports will need to be adjusted
> to handle it, I guess. The "lib/stackdepot: fix first entry having a
> 0-handle" commit has this note:
>
> This bug has been lurking since the very beginning of stackdepot, but no
> one really cared as it seems. Because of that I am not adding a Fixes
> tag.
>
> I don't really know the code very well so I can't respond to that.

Your patch looks good to me, thank you!

Reviewed-by: Andrey Konovalov <andreyknvl@xxxxxxxxx>