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