Commit d4e64319 authored by Rusty Russell's avatar Rusty Russell Committed by Linus Torvalds

[PATCH] missing release_region in drivers_cdrom_cm206.c

From:  Scott Russell <scott@pantastik.com>

  - missing release_region (reported by kbugs.org)
  - removed extraneous return (Randy Dunlap)
parent 2b803a1b
...@@ -1447,7 +1447,7 @@ int __init cm206_init(void) ...@@ -1447,7 +1447,7 @@ int __init cm206_init(void)
request_region(cm206_base, 16, "cm206"); request_region(cm206_base, 16, "cm206");
cd = (struct cm206_struct *) kmalloc(size, GFP_KERNEL); cd = (struct cm206_struct *) kmalloc(size, GFP_KERNEL);
if (!cd) if (!cd)
return -EIO; goto out_base;
/* Now we have found the adaptor card, try to reset it. As we have /* Now we have found the adaptor card, try to reset it. As we have
* found out earlier, this process generates an interrupt as well, * found out earlier, this process generates an interrupt as well,
* so we might just exploit that fact for irq probing! */ * so we might just exploit that fact for irq probing! */
...@@ -1456,7 +1456,6 @@ int __init cm206_init(void) ...@@ -1456,7 +1456,6 @@ int __init cm206_init(void)
if (cm206_irq <= 0) { if (cm206_irq <= 0) {
printk("can't find IRQ!\n"); printk("can't find IRQ!\n");
goto out_probe; goto out_probe;
return -EIO;
} else } else
printk(" IRQ %d found\n", cm206_irq); printk(" IRQ %d found\n", cm206_irq);
#else #else
...@@ -1530,6 +1529,7 @@ int __init cm206_init(void) ...@@ -1530,6 +1529,7 @@ int __init cm206_init(void)
free_irq(cm206_irq, NULL); free_irq(cm206_irq, NULL);
out_probe: out_probe:
kfree(cd); kfree(cd);
out_base:
release_region(cm206_base, 16); release_region(cm206_base, 16);
return -EIO; return -EIO;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment