On Fri, Feb 19, 2021 at 3:31 PM Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote:
>
> This code does not allocate enough memory for the NUL terminator so it
> ends up putting it one character beyond the end of the buffer.
>
> Fixes: 8756828a8148 ("octeontx2-af: Add NPA aura and pool contexts to debugfs")
> Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
> ---
> drivers/net/ethernet/marvell/octeontx2/af/rvu_debugfs.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/net/ethernet/marvell/octeontx2/af/rvu_debugfs.c b/drivers/net/ethernet/marvell/octeontx2/af/rvu_debugfs.c
> index 48a84c65804c..d5f3ad660588 100644
> --- a/drivers/net/ethernet/marvell/octeontx2/af/rvu_debugfs.c
> +++ b/drivers/net/ethernet/marvell/octeontx2/af/rvu_debugfs.c
> @@ -385,7 +385,7 @@ static ssize_t rvu_dbg_qsize_write(struct file *filp,
> u16 pcifunc;
> int ret, lf;
>
> - cmd_buf = memdup_user(buffer, count);
> + cmd_buf = memdup_user(buffer, count + 1);
> if (IS_ERR(cmd_buf))
> return -ENOMEM;
>
> --
> 2.30.0
>
Thanks for the fix.
