Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758294AbcLAEmz (ORCPT ); Wed, 30 Nov 2016 23:42:55 -0500 Received: from mailout2.samsung.com ([203.254.224.25]:33046 "EHLO mailout2.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752151AbcLAEmw (ORCPT ); Wed, 30 Nov 2016 23:42:52 -0500 X-AuditID: cbfee61b-f796f6d000004092-76-583faa4950f7 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] Platform: Exynos4-is: Clean up file handle in open() error path. Date: Thu, 01 Dec 2016 10:10:22 +0530 Message-id: <1480567222-12829-1-git-send-email-shailendra.v@samsung.com> X-Mailer: git-send-email 1.7.9.5 X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFvrGLMWRmVeSWpSXmKPExsVy+t9jQV3PVfYRBgsPKVu8ebuGyaL/8Wtm i/PnN7BbnG16w26x6fE1VovLu+awWfRs2MpqMeP8PiaLZZv+MFkcftPOanFr3kw2i0XfXjFb /L14ic2B12PnrLvsHptWdbJ5bF5S77GlH8jr27KK0ePzJrkAtig3m4zUxJTUIoXUvOT8lMy8 dFul0BA3XQslhbzE3FRbpQhd35AgJYWyxJxSIM/IAA04OAe4Byvp2yW4ZZw9NZW1YBFbxf37 z1kaGNexdjFyckgImEi035nIDGGLSVy4t56ti5GLQ0hgFqPEt/8LmCCcn4wSD1r62UCq2AQM JV73/QVLiAh8ZJb4cHkvO0iCWUBGYt/ENqAEB4ewQLDEm1uWIGEWAVWJX/ffg/XyCnhIrP21 gw2kREJAQWLOJJsJjNwLGBlWMUqkFiQXFCel5xrlpZbrFSfmFpfmpesl5+duYgSH+TPpHYyH d7kfYhTgYFTi4d1x1y5CiDWxrLgy9xCjBAezkgiv7Ur7CCHelMTKqtSi/Pii0pzU4kOMpkD7 JzJLiSbnA2MwryTe0MTcxNzYwMLc0tLESEmct3H2s3AhgfTEktTs1NSC1CKYPiYOTqkGRn73 mmeNHPfct5XF1eyUfWi95PCN9/ceTeE/8MnVjTdC5muQyo/aJ6suF6+72Sn73LRdU0ZRMy+m 0tJzT3Y1+7b8uTs1Ds22vbnqdcJRjmcyEyTCuxdq322bGfS91bIqcv6NxdNL9x5xuz3B631G WnFJg9yKD1Fcf4yUqv21Xym9zWy/G6I5X4mlOCPRUIu5qDgRABdWuUaJAgAA 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