From patchwork Fri Jun 19 20:30:04 2009 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jiri Slaby X-Patchwork-Id: 31404 Received: from vger.kernel.org (vger.kernel.org [209.132.176.167]) by demeter.kernel.org (8.14.2/8.14.2) with ESMTP id n5JKUZrN025724 for ; Fri, 19 Jun 2009 20:30:35 GMT Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756659AbZFSUa3 (ORCPT ); Fri, 19 Jun 2009 16:30:29 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1756592AbZFSUa3 (ORCPT ); Fri, 19 Jun 2009 16:30:29 -0400 Received: from smtp.wellnetcz.com ([212.24.148.102]:40504 "EHLO smtp.wellnetcz.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753476AbZFSUaX (ORCPT ); Fri, 19 Jun 2009 16:30:23 -0400 Received: from localhost.localdomain ([88.208.94.142]) by smtp.wellnetcz.com (8.14.1/8.14.1) with ESMTP id n5JLBtd5015078; Fri, 19 Jun 2009 23:11:55 +0200 From: Jiri Slaby To: mchehab@infradead.org Cc: linux-media@vger.kernel.org, linux-kernel@vger.kernel.org, Jiri Slaby Subject: [PATCH 1/4] V4L: radio-si470x, fix lock imbalance Date: Fri, 19 Jun 2009 22:30:04 +0200 Message-Id: <1245443407-17410-1-git-send-email-jirislaby@gmail.com> X-Mailer: git-send-email 1.6.3.2 Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org There is one path with omitted unlock in si470x_fops_release. Fix that. Signed-off-by: Jiri Slaby --- drivers/media/radio/radio-si470x.c | 5 ++--- 1 files changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/media/radio/radio-si470x.c b/drivers/media/radio/radio-si470x.c index 640421c..46d2163 100644 --- a/drivers/media/radio/radio-si470x.c +++ b/drivers/media/radio/radio-si470x.c @@ -1200,7 +1200,7 @@ static int si470x_fops_release(struct file *file) video_unregister_device(radio->videodev); kfree(radio->buffer); kfree(radio); - goto done; + goto unlock; } /* stop rds reception */ @@ -1213,9 +1213,8 @@ static int si470x_fops_release(struct file *file) retval = si470x_stop(radio); usb_autopm_put_interface(radio->intf); } - +unlock: mutex_unlock(&radio->disconnect_lock); - done: return retval; }