linux-kernel - Re: isapnp: broken API

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20070223220502.GA5599@martell.zuzino.mipt.ru>
Date:	Sat, 24 Feb 2007 01:05:02 +0300
From:	Alexey Dobriyan <adobriyan@...il.com>
To:	Sacher Khoudari <Sacher.Khoudari@...d.uni-heidelberg.de>
Cc:	linux-kernel@...r.kernel.org
Subject: Re: isapnp: broken API

On Fri, Feb 23, 2007 at 04:25:05PM +0100, Sacher Khoudari wrote:
> I've lately tried to run a ISA card (HP 82341, with drivers from the
> Linux GPIB project, latest version), but the module failed to get
> loaded. Modprobe said it couldn't find a symbol (Undefined symbol
> 'isapnp_read_byte'), although it was present in /boot/System.map and
> /proc/kallsyms. After some googleing, trying to compile a new kernel,
> looking into code, etc, I found out what the problem was. The function
> "isapnp_read_byte" was no longer been exported. You can find the line
> in question here [1] (note the #if 0). The patch that did this change
> was this [2] one.
>
> I'm not sure, but as I understand it, it was not intended to comment
> this EXPORT_SYMBOLS(isapnp_read_byte) out. So it probably just
> happened by accident?

No. It was unexported because no modular code in mainline was using it.

> [1] http://www.gelato.unsw.edu.au/lxr/source/drivers/pnp/isapnp/core.c#L948
> [2] http://lkml.org/lkml/2005/6/20/307

> --- linux-2.6.20.1/drivers/pnp/isapnp/core.c.old
> +++ linux-2.6.20.1/drivers/pnp/isapnp/core.c
> @@ -943,9 +943,7 @@
>  EXPORT_SYMBOL(isapnp_present);
>  EXPORT_SYMBOL(isapnp_cfg_begin);
>  EXPORT_SYMBOL(isapnp_cfg_end);
> -#if 0
>  EXPORT_SYMBOL(isapnp_read_byte);
> -#endif
>  EXPORT_SYMBOL(isapnp_write_byte);

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