Skip to content

Commit d8c94df

Browse files
Haoxiang Ligregkh
authored andcommitted
media: imx: fix a potential memory leak in imx_media_csc_scaler_device_init()
commit fc5f8ae upstream. Add video_device_release() in label 'err_m2m' to release the memory allocated by video_device_alloc() and prevent potential memory leaks. Remove the reduntant code in label 'err_m2m'. Fixes: a8ef048 ("media: imx: add csc/scaler mem2mem device") Cc: stable@vger.kernel.org Signed-off-by: Haoxiang Li <haoxiang_li2024@163.com> Reviewed-by: Dan Carpenter <dan.carpenter@linaro.org> Signed-off-by: Nicolas Dufresne <nicolas.dufresne@collabora.com> Signed-off-by: Hans Verkuil <hverkuil@xs4all.nl> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
1 parent 1d60df4 commit d8c94df

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

drivers/staging/media/imx/imx-media-csc-scaler.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -912,7 +912,7 @@ imx_media_csc_scaler_device_init(struct imx_media_dev *md)
912912
return &priv->vdev;
913913

914914
err_m2m:
915-
video_set_drvdata(vfd, NULL);
915+
video_device_release(vfd);
916916
err_vfd:
917917
kfree(priv);
918918
return ERR_PTR(ret);

0 commit comments

Comments
 (0)