Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756862AbcLBEpi (ORCPT ); Thu, 1 Dec 2016 23:45:38 -0500 Received: from mailout3.samsung.com ([203.254.224.33]:37576 "EHLO mailout3.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751963AbcLBEpf (ORCPT ); Thu, 1 Dec 2016 23:45:35 -0500 X-AuditID: cbfee61a-f79916d0000062de-b5-5840fc6dcc3d From: Shailendra Verma To: Mauro Carvalho Chehab , Kukjin Kim , Krzysztof Kozlowski , Javier Martinez Canillas , Kyungmin Park , Sylwester Nawrocki , linux-media@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org, Shailendra Verma , Shailendra Verma Cc: vidushi.koul@samsung.com Subject: [PATCH] exynos4-is: Clean up file handle in open() error path. Date: Fri, 02 Dec 2016 10:13:05 +0530 Message-id: <1480653785-6713-1-git-send-email-shailendra.v@samsung.com> X-Mailer: git-send-email 1.7.9.5 X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFvrOLMWRmVeSWpSXmKPExsVy+t9jAd3cPw4RBrsaZSzevF3DZNH/+DWz xfnzG9gtzja9YbfY9Pgaq8XlXXPYLHo2bGW1mHF+H5PFsk1/mCwOv2lntbg1byabxaJvr5gt /l68xObA67Fz1l12j02rOtk8Ni+p99jSD+T1bVnF6PF5k1wAW5SbTUZqYkpqkUJqXnJ+SmZe uq1SaIibroWSQl5ibqqtUoSub0iQkkJZYk4pkGdkgAYcnAPcg5X07RLcMs6emspasIit4v79 5ywNjOtYuxg5OSQETCQ6T8xnh7DFJC7cW8/WxcjFISSwlFFi9exvTBDOT0aJiWu2MYJUsQkY Srzu+wuWEBH4yCzx4fJesHZmARmJfRPbmEBsYQF3ib9HD4E1sAioSnRt3cACYvMCxa+v/cnc xcgBtE5BYs4kmwmM3AsYGVYxSqQWJBcUJ6XnGuallusVJ+YWl+al6yXn525iBAf6M6kdjAd3 uR9iFOBgVOLhfWHsECHEmlhWXJl7iFGCg1lJhLfvF1CINyWxsiq1KD++qDQntfgQoynQ/onM UqLJ+cAozCuJNzQxNzE3NrAwt7Q0MVIS522c/SxcSCA9sSQ1OzW1ILUIpo+Jg1OqgVHiY0JS ffhUnnnN7+W0xS49zZ32RPvE9J+PPENqGm74LBIx/sbjb9N74vgpZ7XqFdZSe9VMQvWOneXY GrL1THRQ5UvHLt6nv3JjZr/p3u62+aPXsu8mlb6Cj84z5zH926z/YNniiqqPyye+2HHv+Bqh thmPjwU7lbwvZNeudXz9o8GNP/rW6ntKLMUZiYZazEXFiQDcqchZigIAAA== X-MTR: 20000000000000000@CPGS Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 776 Lines: 24 The File handle is not yet added in the vfd list.So no need to call v4l2_fh_del(&ctx->fh) if it fails to create control. Signed-off-by: Shailendra Verma --- drivers/media/platform/exynos4-is/fimc-m2m.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/platform/exynos4-is/fimc-m2m.c b/drivers/media/platform/exynos4-is/fimc-m2m.c index 6028e4f..d8724fe 100644 --- a/drivers/media/platform/exynos4-is/fimc-m2m.c +++ b/drivers/media/platform/exynos4-is/fimc-m2m.c @@ -663,8 +663,8 @@ static int fimc_m2m_open(struct file *file) v4l2_m2m_ctx_release(ctx->fh.m2m_ctx); error_c: fimc_ctrls_delete(ctx); -error_fh: v4l2_fh_del(&ctx->fh); +error_fh: v4l2_fh_exit(&ctx->fh); kfree(ctx); unlock: -- 1.7.9.5