V4L/DVB (13547): radio: add trivial checks on the tuner and type args.

[platform/kernel/linux-starfive.git] / drivers / media / radio / radio-mr800.c

diff --git a/drivers/media/radio/radio-mr800.c b/drivers/media/radio/radio-mr800.c
index 5506fac..02a9cef 100644 (file)
--- a/drivers/media/radio/radio-mr800.c
+++ b/drivers/media/radio/radio-mr800.c
@@ -28,7 +28,7 @@
  * http://av-usbradio.sourceforge.net/index.php
  * http://sourceforge.net/projects/av-usbradio/
  * Latest release of theirs project was in 2005.
- * Probably, this driver could be improved trough using their
+ * Probably, this driver could be improved through using their
  * achievements (specifications given).
  * Also, Faidon Liambotis <paravoid@debian.org> wrote nice driver for this radio
  * in 2007. He allowed to use his driver to improve current mr800 radio driver.
@@ -132,6 +132,7 @@ static int usb_amradio_resume(struct usb_interface *intf);
 struct amradio_device {
        /* reference to USB and video device */
        struct usb_device *usbdev;
+       struct usb_interface *intf;
        struct video_device videodev;
        struct v4l2_device v4l2_dev;
 
@@ -163,7 +164,7 @@ static struct usb_driver usb_amradio_driver = {
        .resume                 = usb_amradio_resume,
        .reset_resume           = usb_amradio_resume,
        .id_table               = usb_amradio_device_table,
-       .supports_autosuspend   = 0,
+       .supports_autosuspend   = 1,
 };
 
 /* switch on/off the radio. Send 8 bytes to device */
@@ -235,6 +236,7 @@ static int amradio_setfreq(struct amradio_device *radio, int freq)
        if (retval < 0 || size != BUFFER_LENGTH)
                goto out_err;
 
+       radio->curfreq = freq;
        goto out;
 
 out_err:
@@ -371,13 +373,10 @@ static int vidioc_s_frequency(struct file *file, void *priv,
                                struct v4l2_frequency *f)
 {
        struct amradio_device *radio = file->private_data;
-       int retval = 0;
-
-       radio->curfreq = f->frequency;
-
-       retval = amradio_setfreq(radio, radio->curfreq);
 
-       return retval;
+       if (f->tuner != 0 || f->type != V4L2_TUNER_RADIO)
+               return -EINVAL;
+       return amradio_setfreq(radio, f->frequency);
 }
 
 /* vidioc_g_frequency - get tuner radio frequency */
@@ -386,6 +385,8 @@ static int vidioc_g_frequency(struct file *file, void *priv,
 {
        struct amradio_device *radio = file->private_data;
 
+       if (f->tuner != 0)
+               return -EINVAL;
        f->type = V4L2_TUNER_RADIO;
        f->frequency = radio->curfreq;
 
@@ -510,9 +511,15 @@ static int usb_amradio_open(struct file *file)
        }
 
        file->private_data = radio;
+       retval = usb_autopm_get_interface(radio->intf);
+       if (retval)
+               goto unlock;
 
-       if (unlikely(!radio->initialized))
+       if (unlikely(!radio->initialized)) {
                retval = usb_amradio_init(radio);
+               if (retval)
+                       usb_autopm_put_interface(radio->intf);
+       }
 
 unlock:
        mutex_unlock(&radio->lock);
@@ -529,6 +536,8 @@ static int usb_amradio_close(struct file *file)
 
        if (!radio->usbdev)
                retval = -EIO;
+       else
+               usb_autopm_put_interface(radio->intf);
 
        mutex_unlock(&radio->lock);
        return retval;
@@ -670,6 +679,7 @@ static int usb_amradio_probe(struct usb_interface *intf,
        radio->videodev.release = usb_amradio_video_device_release;
 
        radio->usbdev = interface_to_usbdev(intf);
+       radio->intf = intf;
        radio->curfreq = 95.16 * FREQ_MUL;
 
        mutex_init(&radio->lock);