[PATCH] bdisp: Clean up file handle in open() error path.

2016-12-01 Thread Shailendra Verma
The File handle is not yet added in the vdev list.So no need to call v4l2_fh_del(>fh)if it fails to create control. Signed-off-by: Shailendra Verma <shailendr...@samsung.com> --- drivers/media/platform/sti/bdisp/bdisp-v4l2.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) d

[PATCH] exynos-gsc: Clean up file handle in open() error path.

2016-12-01 Thread Shailendra Verma
The File handle is not yet added in the vfd list.So no need to call v4l2_fh_del(>fh) if it fails to create control. Signed-off-by: Shailendra Verma <shailendr...@samsung.com> --- drivers/media/platform/exynos-gsc/gsc-m2m.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) d

[PATCH] exynos4-is: Clean up file handle in open() error path.

2016-12-01 Thread Shailendra Verma
The File handle is not yet added in the vfd list.So no need to call v4l2_fh_del(>fh) if it fails to create control. Signed-off-by: Shailendra Verma <shailendr...@samsung.com> --- drivers/media/platform/exynos4-is/fimc-m2m.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) d

[PATCH] Platform: Sti: Bdisp: Clean up file handle in open() error path.

2016-11-30 Thread Shailendra Verma
The File handle is not yet added in the vdev list.So no need to call v4l2_fh_del(>fh)if it fails to create control. Signed-off-by: Shailendra Verma <shailendr...@samsung.com> --- drivers/media/platform/sti/bdisp/bdisp-v4l2.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) d

[PATCH] V4l: omap4iss: Clean up file handle in open() and release().

2016-11-30 Thread Shailendra Verma
Both functions initialize the file handle with v4l2_fh_init() and thus need to call clean up with v4l2_fh_exit() as appropriate. Signed-off-by: Shailendra Verma <shailendr...@samsung.com> --- drivers/staging/media/omap4iss/iss_video.c |2 ++ 1 file changed, 2 insertions(+) diff

[PATCH] Platform: vsp1: Clean up file handle in open() error path.

2016-11-30 Thread Shailendra Verma
v4l2_fh_init is already done.So call the v4l2_fh_exit in error condition before returing from the function. Signed-off-by: Shailendra Verma <shailendr...@samsung.com> --- drivers/media/platform/vsp1/vsp1_video.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/media/platfor

[PATCH] V4l: omap3isp: Clean up file handle in open() and release().

2016-11-30 Thread Shailendra Verma
Both functions initialize the file handle with v4l2_fh_init() and thus need to call clean up with v4l2_fh_exit() as appropriate. Signed-off-by: Shailendra Verma <shailendr...@samsung.com> --- drivers/media/platform/omap3isp/ispvideo.c |2 ++ 1 file changed, 2 insertions(+) diff

[PATCH] Platform: Exynos-gsc: Clean up file handle in open() error path.

2016-11-30 Thread Shailendra Verma
The File handle is not yet added in the vfd list.So no need to call v4l2_fh_del(>fh) if it fails to create control. Signed-off-by: Shailendra Verma <shailendr...@samsung.com> --- drivers/media/platform/exynos-gsc/gsc-m2m.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) d

[PATCH] Platform: Exynos4-is: Clean up file handle in open() error path.

2016-11-30 Thread Shailendra Verma
The File handle is not yet added in the vfd list.So no need to call v4l2_fh_del(>fh) if it fails to create control. Signed-off-by: Shailendra Verma <shailendr...@samsung.com> --- drivers/media/platform/exynos4-is/fimc-m2m.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) d

[PATCH] Media: platform: vsp1: - Do not forget to call

2016-11-24 Thread Shailendra Verma
v4l2_fh_init is already done.So call the v4l2_fh_exit in error condition before returing from the function. Signed-off-by: Shailendra Verma <shailendr...@samsung.com> --- drivers/media/platform/vsp1/vsp1_video.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/media/platfor

[PATCH] Staging: media: davinci_vpfe: - Fix for memory leak if decoder initialization fails.

2016-11-24 Thread Shailendra Verma
Fix to avoid possible memory leak if the decoder initialization got failed.Free the allocated memory for file handle object before return in case decoder initialization fails. Signed-off-by: Shailendra Verma <shailendr...@samsung.com> --- drivers/staging/media/davinci_vpfe/vpfe_video.c

[PATCH] Media: Platform: Exynos-gsc: - Fix for error handling

2016-11-24 Thread Shailendra Verma
The File handle is not yet added in the vfd list.So no need to call v4l2_fh_del(>fh) if it fails to create control. Signed-off-by: Shailendra Verma <shailendr...@samsung.com> --- drivers/media/platform/exynos-gsc/gsc-m2m.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) d

[PATCH] Staging: media: platform: davinci: - Fix for memory leak

2016-11-24 Thread Shailendra Verma
Fix to avoid possible memory leak if the decoder initialization got failed. Free the allocated memory for file handle object before return in case decoder initialization fails. Signed-off-by: Shailendra Verma <shailendr...@samsung.com> --- drivers/media/platform/davinci/vpfe_capture.c

[PATCH] Staging: Media: Omap4iss: Do not forget to call

2016-11-24 Thread Shailendra Verma
v4l2_fh_init is already done.So call the v4l2_fh_exit in error condition before returing from the function. Signed-off-by: Shailendra Verma <shailendr...@samsung.com> --- drivers/staging/media/omap4iss/iss_video.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging

[PATCH] Media: Platform: Sti: Bdisp: - Fix for error handling

2016-11-24 Thread Shailendra Verma
The File handle is not yet added in the vdev list.So no need to call v4l2_fh_del(>fh)if it fails to create control. Signed-off-by: Shailendra Verma <shailendr...@samsung.com> --- drivers/media/platform/sti/bdisp/bdisp-v4l2.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) d

[PATCH] Media: Platform: Omap3isp: Do not forget to call

2016-11-24 Thread Shailendra Verma
v4l2_fh_init is already done.So call the v4l2_fh_exit in error condition before returing from the function. Signed-off-by: Shailendra Verma <shailendr...@samsung.com> --- drivers/media/platform/omap3isp/ispvideo.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/media/pl

[PATCH] Media: Platform: Exynos4-is: - Fix for error handling

2016-11-24 Thread Shailendra Verma
The File handle is not yet added in the vfd list.So no need to call v4l2_fh_del(>fh) if it fails to create control. Signed-off-by: Shailendra Verma <shailendr...@samsung.com> --- drivers/media/platform/exynos4-is/fimc-m2m.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) d

[PATCH] Staging: media: davinci_vpfe: - Fix for memory leak if

2016-11-11 Thread Shailendra Verma
From: "Shailendra Verma" <shailendr...@samsung.com> Fix to avoid possible memory leak if the decoder initialization got failed.Free the allocated memory for file handle object before return in case decoder initialization fails. Signed-off-by: Shailendra Verma <shailendra.

[PATCH] Staging: Media: Lirc - Improvement in code readability

2016-11-10 Thread Shailendra Verma
From: "Shailendra Verma" <shailendr...@samsung.com> There is no need to call kfree() if memdup_user() fails, as no memory was allocated and the error in the error-valued pointer should be returned. Signed-off-by: Shailendra Verma <shailendr...@samsung.com> --- driv

[PATCH 2/2] Staging: Media: Lirc - Improvement in code readability

2016-11-10 Thread Shailendra Verma
From: "Shailendra Verma" <shailendr...@samsung.com> There is no need to call kfree() if memdup_user() fails, as no memory was allocated and the error in the error-valued pointer should be returned. Signed-off-by: Shailendra Verma <shailendr...@samsung.com> --- driv

Staging: Media: Lirc - Fix possible ERR_PTR() dereferencing.

2016-09-20 Thread Shailendra Verma
This is of course wrong to call kfree() if memdup_user() fails, no memory was allocated and the error in the error-valued pointer should be returned. Reviewed-by: Ravikant Sharma <ravikant...@samsung.com> Signed-off-by: Shailendra Verma <shailendr...@samsung.com> --- drivers/stagin