Skip to content

Commit

Permalink
[PATCH] cx88-dvb oops fix
Browse files Browse the repository at this point in the history
Fixup error path, without that one the driver kills the machine by oopsing
in the IRQ handler in case the frontend initialization fails.

Signed-off-by: Gerd Knorr <kraxel@bytesex.org>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
  • Loading branch information
Gerd Knorr authored and Linus Torvalds committed May 1, 2005
1 parent f8e08a8 commit 1622c3f
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions drivers/media/video/cx88/cx88-dvb.c
Original file line number Diff line number Diff line change
Expand Up @@ -243,10 +243,8 @@ static int dvb_register(struct cx8802_dev *dev)
break;
#endif
default:
printk("%s: The frontend of your DVB/ATSC card isn't supported yet\n"
"%s: you might want to look out for patches here:\n"
"%s: http://dl.bytesex.org/patches/\n",
dev->core->name, dev->core->name, dev->core->name);
printk("%s: The frontend of your DVB/ATSC card isn't supported yet\n",
dev->core->name);
break;
}
if (NULL == dev->dvb.frontend) {
Expand Down Expand Up @@ -308,9 +306,11 @@ static int __devinit dvb_probe(struct pci_dev *pci_dev,
dev);
err = dvb_register(dev);
if (0 != err)
goto fail_free;
goto fail_fini;
return 0;

fail_fini:
cx8802_fini_common(dev);
fail_free:
kfree(dev);
fail_core:
Expand Down

0 comments on commit 1622c3f

Please sign in to comment.