1 files changed, 24 insertions, 11 deletions

diff --git a/sound/oss/msnd_pinnacle.c b/sound/oss/msnd_pinnacle.c
index a1e3f9671bea..2e48b17667d0 100644
--- a/sound/oss/msnd_pinnacle.c
+++ b/sound/oss/msnd_pinnacle.c
@@ -639,21 +639,26 @@ static int mixer_ioctl(unsigned int cmd, unsigned long arg)
         return -EINVAL;
 }
 
-static int dev_ioctl(struct inode *inode, struct file *file, unsigned int cmd, unsigned long arg)
+static long dev_ioctl(struct file *file, unsigned int cmd, unsigned long arg)
 {
-        int minor = iminor(inode);
+        int minor = iminor(file->f_path.dentry->d_inode);
+        int ret;
 
         if (cmd == OSS_GETVERSION) {
                 int sound_version = SOUND_VERSION;
                 return put_user(sound_version, (int __user *)arg);
         }
 
+        ret = -EINVAL;
+
+        lock_kernel();
         if (minor == dev.dsp_minor)
-                return dsp_ioctl(file, cmd, arg);
+                ret = dsp_ioctl(file, cmd, arg);
         else if (minor == dev.mixer_minor)
-                return mixer_ioctl(cmd, arg);
+                ret = mixer_ioctl(cmd, arg);
+        unlock_kernel();
 
-        return -EINVAL;
+        return ret;
 }
 
 static void dsp_write_flush(void)
@@ -756,12 +761,15 @@ static int dev_open(struct inode *inode, struct file *file)
         int minor = iminor(inode);
         int err = 0;
 
+        lock_kernel();
         if (minor == dev.dsp_minor) {
                 if ((file->f_mode & FMODE_WRITE &&
                      test_bit(F_AUDIO_WRITE_INUSE, &dev.flags)) ||
                     (file->f_mode & FMODE_READ &&
-                     test_bit(F_AUDIO_READ_INUSE, &dev.flags)))
-                        return -EBUSY;
+                     test_bit(F_AUDIO_READ_INUSE, &dev.flags))) {
+                        err = -EBUSY;
+                        goto out;
+                }
 
                 if ((err = dsp_open(file)) >= 0) {
                         dev.nresets = 0;
@@ -782,7 +790,8 @@ static int dev_open(struct inode *inode, struct file *file)
                 /* nothing */
         } else
                 err = -EINVAL;
-
+out:
+        unlock_kernel();
         return err;
 }
 
@@ -1105,7 +1114,7 @@ static const struct file_operations dev_fileops = {
         .owner          = THIS_MODULE,
         .read           = dev_read,
         .write          = dev_write,
-        .ioctl          = dev_ioctl,
+        .unlocked_ioctl = dev_ioctl,
         .open           = dev_open,
         .release        = dev_release,
 };
@@ -1391,9 +1400,13 @@ static int __init attach_multisound(void)
                 printk(KERN_ERR LOGNAME ": Couldn't grab IRQ %d\n", dev.irq);
                 return err;
         }
-        request_region(dev.io, dev.numio, dev.name);
+        if (request_region(dev.io, dev.numio, dev.name) == NULL) {
+                free_irq(dev.irq, &dev);
+                return -EBUSY;
+        }
 
-        if ((err = dsp_full_reset()) < 0) {
+        err = dsp_full_reset();
+        if (err < 0) {
                 release_region(dev.io, dev.numio);
                 free_irq(dev.irq, &dev);
                 return err;