[PATCH 4.9 41/41] media: v4l: ioctl: Fix memory leak in video_usercopy

2021-03-05 Thread Greg Kroah-Hartman
From: Sakari Ailus commit fb18802a338b36f675a388fc03d2aa504a0d0899 upstream. When an IOCTL with argument size larger than 128 that also used array arguments were handled, two memory allocations were made but alas, only the latter one of them was released. This happened because there was only a s

[PATCH 4.4 30/30] media: v4l: ioctl: Fix memory leak in video_usercopy

2021-03-05 Thread Greg Kroah-Hartman
From: Sakari Ailus commit fb18802a338b36f675a388fc03d2aa504a0d0899 upstream. When an IOCTL with argument size larger than 128 that also used array arguments were handled, two memory allocations were made but alas, only the latter one of them was released. This happened because there was only a s

[PATCH 4.14 39/39] media: v4l: ioctl: Fix memory leak in video_usercopy

2021-03-05 Thread Greg Kroah-Hartman
From: Sakari Ailus commit fb18802a338b36f675a388fc03d2aa504a0d0899 upstream. When an IOCTL with argument size larger than 128 that also used array arguments were handled, two memory allocations were made but alas, only the latter one of them was released. This happened because there was only a s

[PATCH 4.19 50/52] media: v4l: ioctl: Fix memory leak in video_usercopy

2021-03-05 Thread Greg Kroah-Hartman
From: Sakari Ailus commit fb18802a338b36f675a388fc03d2aa504a0d0899 upstream. When an IOCTL with argument size larger than 128 that also used array arguments were handled, two memory allocations were made but alas, only the latter one of them was released. This happened because there was only a s

[PATCH 5.4 69/72] media: v4l: ioctl: Fix memory leak in video_usercopy

2021-03-05 Thread Greg Kroah-Hartman
From: Sakari Ailus commit fb18802a338b36f675a388fc03d2aa504a0d0899 upstream. When an IOCTL with argument size larger than 128 that also used array arguments were handled, two memory allocations were made but alas, only the latter one of them was released. This happened because there was only a s

[PATCH 5.10 099/102] media: v4l: ioctl: Fix memory leak in video_usercopy

2021-03-05 Thread Greg Kroah-Hartman
From: Sakari Ailus commit fb18802a338b36f675a388fc03d2aa504a0d0899 upstream. When an IOCTL with argument size larger than 128 that also used array arguments were handled, two memory allocations were made but alas, only the latter one of them was released. This happened because there was only a s

[PATCH 5.11 666/775] media: v4l: ioctl: Fix memory leak in video_usercopy

2021-03-01 Thread Greg Kroah-Hartman
From: Sakari Ailus commit fb18802a338b36f675a388fc03d2aa504a0d0899 upstream. When an IOCTL with argument size larger than 128 that also used array arguments were handled, two memory allocations were made but alas, only the latter one of them was released. This happened because there was only a s

Re: [PATCH v2 1/1] v4l: ioctl: Fix memory leak in video_usercopy

2021-01-13 Thread Bingbu Cao
On 1/14/21 12:50 PM, Bingbu Cao wrote: > Sakari, > > On 12/21/20 4:11 AM, Sakari Ailus wrote: >> When an IOCTL with argument size larger than 128 that also used array >> arguments were handled, two memory allocations were made but alas, only >> the latter one of them was released. This happened

Re: [PATCH v2 1/1] v4l: ioctl: Fix memory leak in video_usercopy

2021-01-13 Thread Bingbu Cao
Sakari, On 12/21/20 4:11 AM, Sakari Ailus wrote: > When an IOCTL with argument size larger than 128 that also used array > arguments were handled, two memory allocations were made but alas, only > the latter one of them was released. This happened because there was only > a single local variable t

[PATCH v2 1/1] v4l: ioctl: Fix memory leak in video_usercopy

2020-12-20 Thread Sakari Ailus
When an IOCTL with argument size larger than 128 that also used array arguments were handled, two memory allocations were made but alas, only the latter one of them was released. This happened because there was only a single local variable to hold such a temporary allocation. Fix this by adding se

Re: [PATCH 1/1] v4l: ioctl: Fix memory leak in video_usercopy

2020-12-20 Thread Sakari Ailus
Hi Laurent, On Sun, Dec 20, 2020 at 04:15:07PM +0200, Laurent Pinchart wrote: > Hi Sakari, > > Thank you for the patch. > > On Sun, Dec 20, 2020 at 01:06:51PM +0200, Sakari Ailus wrote: > > When an IOCTL with argument size larger than 128 that also used array > > arguments were handled, two memo

Re: [PATCH 1/1] v4l: ioctl: Fix memory leak in video_usercopy

2020-12-20 Thread Laurent Pinchart
Hi Sakari, Thank you for the patch. On Sun, Dec 20, 2020 at 01:06:51PM +0200, Sakari Ailus wrote: > When an IOCTL with argument size larger than 128 that also used array > arguments were handled, two memory allocations were made but alas, only > the latter one of them was released. Alas, this fi

Re: [PATCH 1/1] v4l: ioctl: Fix memory leak in video_usercopy

2020-12-20 Thread Sakari Ailus
On Sun, Dec 20, 2020 at 12:36:23PM +0100, Arnd Bergmann wrote: > On Sun, Dec 20, 2020 at 12:06 PM Sakari Ailus > wrote: > > > > When an IOCTL with argument size larger than 128 that also used array > > arguments were handled, two memory allocations were made but alas, only > > the latter one of th

Re: [PATCH 1/1] v4l: ioctl: Fix memory leak in video_usercopy

2020-12-20 Thread Arnd Bergmann
On Sun, Dec 20, 2020 at 12:06 PM Sakari Ailus wrote: > > When an IOCTL with argument size larger than 128 that also used array > arguments were handled, two memory allocations were made but alas, only > the latter one of them was released. This happened because there was only > a single local vari

Re: [PATCH 1/1] v4l: ioctl: Fix memory leak in video_usercopy

2020-12-20 Thread Hans Verkuil
On 20/12/2020 12:06, Sakari Ailus wrote: > When an IOCTL with argument size larger than 128 that also used array > arguments were handled, two memory allocations were made but alas, only > the latter one of them was released. This happened because there was only > a single local variable to hold su

[PATCH 1/1] v4l: ioctl: Fix memory leak in video_usercopy

2020-12-20 Thread Sakari Ailus
When an IOCTL with argument size larger than 128 that also used array arguments were handled, two memory allocations were made but alas, only the latter one of them was released. This happened because there was only a single local variable to hold such a temporary allocation. Fix this by adding se

Re: memory leak in video_usercopy

2020-12-19 Thread Sakari Ailus
Hi Arnd, On Sat, Dec 19, 2020 at 03:08:14PM +0100, Arnd Bergmann wrote: > ,On Sat, Dec 19, 2020 at 2:15 PM syzbot > wrote: > > > > Hello, > > > > syzbot found the following issue on: > > > > HEAD commit:a409ed15 Merge tag 'gpio-v5.11-1' of git://git.kernel.org/.. > > git tree: upstream

Re: memory leak in video_usercopy

2020-12-19 Thread Arnd Bergmann
,On Sat, Dec 19, 2020 at 2:15 PM syzbot wrote: > > Hello, > > syzbot found the following issue on: > > HEAD commit:a409ed15 Merge tag 'gpio-v5.11-1' of git://git.kernel.org/.. > git tree: upstream > console output: https://syzkaller.appspot.com/x/log.txt?x=10a5880f50 > kernel config:

memory leak in video_usercopy

2020-12-19 Thread syzbot
Hello, syzbot found the following issue on: HEAD commit:a409ed15 Merge tag 'gpio-v5.11-1' of git://git.kernel.org/.. git tree: upstream console output: https://syzkaller.appspot.com/x/log.txt?x=10a5880f50 kernel config: https://syzkaller.appspot.com/x/.config?x=37c889fb8b2761af das