Re: [PATCH v5 1/2] lib/mpi: Fix buffer overrun when SG is too long

From: Roberto Sassu
Date: Mon Jan 16 2023 - 04:00:20 EST


On Fri, 2023-01-06 at 23:18 +0800, Herbert Xu wrote:
> On Tue, Dec 27, 2022 at 03:27:39PM +0100, Roberto Sassu wrote:
> > From: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx>
> >
> > The helper mpi_read_raw_from_sgl sets the number of entries in
> > the SG list according to nbytes. However, if the last entry
> > in the SG list contains more data than nbytes, then it may overrun
> > the buffer because it only allocates enough memory for nbytes.
> >
> > Fixes: 2d4d1eea540b ("lib/mpi: Add mpi sgl helpers")
> > Reported-by: Roberto Sassu <roberto.sassu@xxxxxxxxxxxxxxx>
> > Signed-off-by: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx>
> > ---
> > lib/mpi/mpicoder.c | 3 ++-
> > 1 file changed, 2 insertions(+), 1 deletion(-)
>
> Patch applied. Thanks.

Hi Herbert

will you take also the second patch?

Thanks

Roberto