Thank you for catching this up. I had this fixed already, but due to hassle caused by having multiple versions of the driver this bug somehow made it unnoticed to mainline. Acked-by: Sylwester Nawrocki <s.nawrocki@xxxxxxxxxxx> > -----Original Message----- > From: Dan Carpenter [mailto:error27@xxxxxxxxx] > Sent: Thursday, August 12, 2010 9:42 AM > To: Mauro Carvalho Chehab > Cc: Pawel Osciak; Kyungmin Park; Sylwester Nawrocki; linux- > media@xxxxxxxxxxxxxxx; kernel-janitors@xxxxxxxxxxxxxxx > Subject: [patch] V4L/DVB: unlock on error path > > If we return directly here then we miss out on some mutex_unlock()s > > Signed-off-by: Dan Carpenter <error27@xxxxxxxxx> > > diff --git a/drivers/media/video/s5p-fimc/fimc-core.c > b/drivers/media/video/s5p-fimc/fimc-core.c > index b151c7b..1beb226 100644 > --- a/drivers/media/video/s5p-fimc/fimc-core.c > +++ b/drivers/media/video/s5p-fimc/fimc-core.c > @@ -822,7 +822,8 @@ static int fimc_m2m_s_fmt(struct file *file, void > *priv, struct v4l2_format *f) > } else { > v4l2_err(&ctx->fimc_dev->m2m.v4l2_dev, > "Wrong buffer/video queue type (%d)\n", f->type); > - return -EINVAL; > + ret = -EINVAL; > + goto s_fmt_out; > } > > pix = &f->fmt.pix; -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
