diff options
author | Shailendra Verma <shailendra.v@samsung.com> | 2016-12-01 23:48:01 -0500 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@s-opensource.com> | 2017-03-22 08:50:02 -0400 |
commit | 389e3f0d57e6ab50c207bee5ea2c4a5982a029c2 (patch) | |
tree | dde478297c6f167bd83c3e768d3abd70282b1194 | |
parent | 6e2e0eea072f6f82ac0afbcfa8dc38dcc3a29fa4 (diff) |
[media] bdisp: Clean up file handle in open() error path
The File handle is not yet added in the vdev list.So no need to call
v4l2_fh_del(&ctx->fh)if it fails to create control.
Signed-off-by: Shailendra Verma <shailendra.v@samsung.com>
Reviewed-by: Fabien Dessenne <fabien.dessenne@st.com>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
-rw-r--r-- | drivers/media/platform/sti/bdisp/bdisp-v4l2.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/media/platform/sti/bdisp/bdisp-v4l2.c b/drivers/media/platform/sti/bdisp/bdisp-v4l2.c index 823608112d89..7918b928f058 100644 --- a/drivers/media/platform/sti/bdisp/bdisp-v4l2.c +++ b/drivers/media/platform/sti/bdisp/bdisp-v4l2.c | |||
@@ -632,8 +632,8 @@ static int bdisp_open(struct file *file) | |||
632 | 632 | ||
633 | error_ctrls: | 633 | error_ctrls: |
634 | bdisp_ctrls_delete(ctx); | 634 | bdisp_ctrls_delete(ctx); |
635 | error_fh: | ||
636 | v4l2_fh_del(&ctx->fh); | 635 | v4l2_fh_del(&ctx->fh); |
636 | error_fh: | ||
637 | v4l2_fh_exit(&ctx->fh); | 637 | v4l2_fh_exit(&ctx->fh); |
638 | bdisp_hw_free_nodes(ctx); | 638 | bdisp_hw_free_nodes(ctx); |
639 | mem_ctx: | 639 | mem_ctx: |