On Tue, Jan 11, 2022 at 08:55:03AM +0200, Dafna Hirschfeld wrote:
> In case we fail to allocate a transfer_buffer then we
> break the buffers creation loop and update the number of
> buffers to the number of successfully allocated which should
> be 'i' and not 'i - 1'
>
> Signed-off-by: Dafna Hirschfeld <dafna.hirschfeld@xxxxxxxxxxxxx>
Reviewed-by: Ezequiel Garcia <ezequiel@xxxxxxxxxxxxxxxxxxxx>
> ---
> drivers/media/usb/stk1160/stk1160-video.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/media/usb/stk1160/stk1160-video.c b/drivers/media/usb/stk1160/stk1160-video.c
> index 202b084f65a2..91bd6adccdd1 100644
> --- a/drivers/media/usb/stk1160/stk1160-video.c
> +++ b/drivers/media/usb/stk1160/stk1160-video.c
> @@ -511,9 +511,9 @@ int stk1160_alloc_isoc(struct stk1160 *dev)
> usb_free_urb(dev->isoc_ctl.urb[i]);
> dev->isoc_ctl.urb[i] = NULL;
>
> - stk1160_warn("%d urbs allocated. Trying to continue...\n", i - 1);
> + stk1160_warn("%d urbs allocated. Trying to continue...\n", i);
>
> - dev->isoc_ctl.num_bufs = i - 1;
> + dev->isoc_ctl.num_bufs = i;
>
> return 0;
>
> --
> 2.17.1
>
