Re: isdn: isdnloop support crashes kernel when compiled in

Kai Germaschewski (kai@tp1.ruhr-uni-bochum.de)
Sun, 11 Nov 2001 13:56:04 +0100 (CET)


On Sat, 10 Nov 2001, Mario 'BitKoenig' Holbe wrote:

> The isdnloop support for the ISDN subsystem crashes the kernel when
> compiled in the kernel directly (null pointer reference).
> I guess, this is because parameters are only given as MODULE_PARM().
>
> I found that in the 2.4.12 kernel, if it's fixed already in higher
> kernels, please excuse me :)
>
> If this is a feature and not a bug, should'nt it then be forced
> to be configured as a module in the Config.in?

It's a bug. The appended patch should fix it.

--Kai

diff -ur linux-2.4.15-pre2.patches/drivers/isdn/isdnloop/isdnloop.c linux-2.4.15-pre2.work/drivers/isdn/isdnloop/isdnloop.c
--- linux-2.4.15-pre2.patches/drivers/isdn/isdnloop/isdnloop.c Sun Oct 21 00:17:11 2001
+++ linux-2.4.15-pre2.work/drivers/isdn/isdnloop/isdnloop.c Sun Nov 11 13:37:36 2001
@@ -1542,7 +1542,11 @@
} else
strcpy(rev, " ??? ");
printk(KERN_NOTICE "isdnloop-ISDN-driver Rev%s\n", rev);
- return (isdnloop_addcard(isdnloop_id));
+
+ if (isdnloop_id)
+ return (isdnloop_addcard(isdnloop_id));
+
+ return 0;
}

static void __exit

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/