From patchwork Sun Sep 13 03:22:47 2009 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: David Ellingsworth X-Patchwork-Id: 47122 X-Patchwork-Delegate: dougsland@redhat.com 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 n8D3Mqe1004988 for ; Sun, 13 Sep 2009 03:22:59 GMT Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754837AbZIMDWz (ORCPT ); Sat, 12 Sep 2009 23:22:55 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1754824AbZIMDWz (ORCPT ); Sat, 12 Sep 2009 23:22:55 -0400 Received: from qw-out-2122.google.com ([74.125.92.27]:45517 "EHLO qw-out-2122.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754741AbZIMDWy (ORCPT ); Sat, 12 Sep 2009 23:22:54 -0400 Received: by qw-out-2122.google.com with SMTP id 9so721299qwb.37 for ; Sat, 12 Sep 2009 20:22:57 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:received:sender:message-id:date:from :reply-to:user-agent:mime-version:to:subject:content-type; bh=3fnOl0ytZB1NAboaLKkfsuu7vx0u/mNhy4fVYJOs8eM=; b=c3DKYuav4qXvMP7AQpila/Q3Jt+71Nr/slARfiblfWAf98tVAx26Mu9qPWm39/f0q0 1AboyTQNTYSv7zup7jQXhnkxsd4taWoDqC8LcpeJ5cFwBcnNq0XDlt7z79o65kG+sJ6n U6MmwM96cE/qW9J6pkAO7PmdJABkVkqZa0L7Q= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=sender:message-id:date:from:reply-to:user-agent:mime-version:to :subject:content-type; b=bfVU0grzqI65TNHSXhnLgGDBJK00NLyVzKA5qvLo8UN5ykd2b60bNZ6cwz3a0aanZp y5SzcYo45SpCGFk9XsaTaL4aLp+BpvR0yaHW/hvaHUc3506sjU4B8uhjPU45BJK/wDqz aDSi6nqjIt5PDWroJfj25qyEPNnV91NrjQjEQ= Received: by 10.224.117.203 with SMTP id s11mr4074088qaq.332.1252812177692; Sat, 12 Sep 2009 20:22:57 -0700 (PDT) Received: from ?192.168.10.110? (cpe-71-72-62-249.neo.res.rr.com [71.72.62.249]) by mx.google.com with ESMTPS id 8sm3586191qwj.28.2009.09.12.20.22.56 (version=TLSv1/SSLv3 cipher=RC4-MD5); Sat, 12 Sep 2009 20:22:57 -0700 (PDT) Message-ID: <4AAC6587.7090309@gmail.com> Date: Sat, 12 Sep 2009 23:22:47 -0400 From: David Ellingsworth Reply-To: david@identd.dyndns.org User-Agent: Mozilla-Thunderbird 2.0.0.22 (X11/20090701) MIME-Version: 1.0 To: linux-media@vger.kernel.org, klimov.linux@gmail.com Subject: [RFC/RFT 10/14] radio-mr800: ensure the radio is initialized to a consistent state Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org From 8c441616f67011244cb15bc1a3dda6fd8706ecd2 Mon Sep 17 00:00:00 2001 From: David Ellingsworth Date: Sat, 12 Sep 2009 16:04:44 -0400 Subject: [PATCH 08/14] mr800: fix potential use after free Signed-off-by: David Ellingsworth --- drivers/media/radio/radio-mr800.c | 1 - 1 files changed, 0 insertions(+), 1 deletions(-) /* vidioc_querycap - query device capabilities */ From 8b5f17aeea6cf394bedd6f9029a57b85555f5815 Mon Sep 17 00:00:00 2001 From: David Ellingsworth Date: Sat, 12 Sep 2009 21:59:07 -0400 Subject: [PATCH 10/14] mr800: ensure the radio is initialized to a consistent state Signed-off-by: David Ellingsworth --- drivers/media/radio/radio-mr800.c | 34 ++++++++++++++++++++++++++++++++-- 1 files changed, 32 insertions(+), 2 deletions(-) diff --git a/drivers/media/radio/radio-mr800.c b/drivers/media/radio/radio-mr800.c index df020e8..dbf0dbb 100644 --- a/drivers/media/radio/radio-mr800.c +++ b/drivers/media/radio/radio-mr800.c @@ -85,6 +85,9 @@ MODULE_LICENSE("GPL"); #define amradio_dev_warn(dev, fmt, arg...) \ dev_warn(dev, MR800_DRIVER_NAME " - " fmt, ##arg) +#define amradio_dev_err(dev, fmt, arg...) \ + dev_err(dev, MR800_DRIVER_NAME " - " fmt, ##arg) + /* Probably USB_TIMEOUT should be modified in module parameter */ #define BUFFER_LENGTH 8 #define USB_TIMEOUT 500 @@ -137,6 +140,7 @@ struct amradio_device { int curfreq; int stereo; int muted; + int initialized; }; #define vdev_to_amradio(r) container_of(r, struct amradio_device, videodev) @@ -477,6 +481,31 @@ static int vidioc_s_input(struct file *filp, void *priv, unsigned int i) return 0; } +static int usb_amradio_init(struct amradio_device *radio) +{ + int retval; + + retval = amradio_set_mute(radio, AMRADIO_STOP); + if (retval < 0) { + amradio_dev_warn(&radio->videodev.dev, "amradio_stop failed\n"); + goto out_err; + } + + retval = amradio_set_stereo(radio, WANT_STEREO); + if (retval < 0) { + amradio_dev_warn(&radio->videodev.dev, "set stereo failed\n"); + goto out_err; + } + + radio->initialized = 1; + goto out; + +out_err: + amradio_dev_err(&radio->videodev.dev, "initialization failed\n"); +out: + return retval; +} + /* open device - amradio_start() and amradio_setfreq() */ static int usb_amradio_open(struct file *file) { @@ -492,6 +521,9 @@ static int usb_amradio_open(struct file *file) file->private_data = radio; + if (unlikely(!radio->initialized)) + retval = usb_amradio_init(radio); + unlock: mutex_unlock(&radio->lock); return retval; @@ -640,8 +672,6 @@ static int usb_amradio_probe(struct usb_interface *intf, radio->usbdev = interface_to_usbdev(intf); radio->curfreq = 95.16 * FREQ_MUL; - radio->stereo = -1; - radio->muted = 1; mutex_init(&radio->lock); -- 1.6.3.3