media: bdisp: fix memleak on release
commit11609a7e21
upstream. If a process is interrupted while accessing the video device and the device lock is contended, release() could return early and fail to free related resources. Note that the return value of the v4l2 release file operation is ignored. Fixes:28ffeebbb7
("[media] bdisp: 2D blitter driver using v4l2 mem2mem framework") Cc: stable <stable@vger.kernel.org> # 4.2 Signed-off-by: Johan Hovold <johan@kernel.org> Reviewed-by: Fabien Dessenne <fabien.dessenne@st.com> Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl> Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
parent
7c5aabf080
commit
733c4d12e9
|
@ -651,8 +651,7 @@ static int bdisp_release(struct file *file)
|
||||||
|
|
||||||
dev_dbg(bdisp->dev, "%s\n", __func__);
|
dev_dbg(bdisp->dev, "%s\n", __func__);
|
||||||
|
|
||||||
if (mutex_lock_interruptible(&bdisp->lock))
|
mutex_lock(&bdisp->lock);
|
||||||
return -ERESTARTSYS;
|
|
||||||
|
|
||||||
v4l2_m2m_ctx_release(ctx->fh.m2m_ctx);
|
v4l2_m2m_ctx_release(ctx->fh.m2m_ctx);
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue