Re: [stable] [PATCH] Fix oops in asus_acpi.c on Samsung P30/P35 Laptops

[Greg KH <greg@kroah.com>]

On Tue, Dec 06, 2005 at 07:32:37PM +0100, Carl-Daniel Hailfinger wrote:
> Hi,
> 
> on insmod of asus_acpi on my Samsung P35 laptop I get the following
> Oops (perfectly reproducible):
> 
> Asus Laptop ACPI Extras version 0.29
> Unable to handle kernel NULL pointer dereference at virtual address 00000000
>  printing eip:
> e1dfc362
> *pde = 00000000
> Oops: 0000 [#1]
> Modules linked in: asus_acpi thermal processor fan button battery ac 
> snd_pcm_oss snd_mixer_oss snd_intel8x0 snd_ac97_codec snd_ac97_bus snd_pcm 
> snd_timer snd soundcore
> snd_page_alloc ipt_TOS ipt_LOG ipt_limit ipt_pkttype pcmcia firmware_class 
> ipt_state ip6t_REJECT ipt_REJECT iptable_mangle iptable_nat iptable_filter 
> ip6table_mangle
> ip_nat_ftp ip_nat ip_conntrack_ftp ip_conntrack nfnetlink ip_tables 
> ip6table_filter ip6_tables ipv6 evdev sg sd_mod sr_mod scsi_mod intel_agp 
> agpgart ohci1394 ieee1394
> yenta_socket rsrc_nonstatic pcmcia_core ehci_hcd uhci_hcd i2c_i801 joydev 
> dm_mod usbcore 8139too mii reiserfs ide_cd cdrom ide_disk piix ide_core
> CPU:    0
> EIP:    0060:[<e1dfc362>]    Not tainted VLI
> EFLAGS: 00010203   (2.6.15-rc5)
> EIP is at asus_hotk_get_info+0x17f/0x76c [asus_acpi]
> eax: def75000   ebx: de8aaf54   ecx: 00000002   edx: 00000003
> esi: 00000000   edi: e1e82a9c   ebp: dde2fea0   esp: de8aaf48
> ds: 007b   es: 007b   ss: 0068
> Process modprobe (pid: 6566, threadinfo=de8aa000 task=ddac05b0)
> Stack: 00000000 00005105 deef8000 00000010 dde2fea0 dfeddc00 e1e83196 
> dfeddc84
>        dfedd820 e1dfc982 e1dfca11 dfeddc00 e1e849e0 00000000 c021c2fa 
>        dfeddc00
>        e1e849e0 c021c39e e1e84b00 0805bc08 00000028 de8aa000 e1dfcb20 
>        c0133b32
> Call Trace:
>  [<e1dfc982>] asus_hotk_check+0x33/0x34 [asus_acpi]
>  [<e1dfca11>] asus_hotk_add+0x8e/0x148 [asus_acpi]
>  [<c021c2fa>] acpi_bus_driver_init+0x2e/0x57
>  [<c021c39e>] acpi_driver_attach+0x3e/0x63
>  [<e1dfcb20>] asus_acpi_init+0x55/0x7d [asus_acpi]
>  [<c0133b32>] sys_init_module+0xf2/0x180
>  [<c0102e6f>] sysenter_past_esp+0x54/0x75
> Code: 08 68 7f 30 e8 e1 e8 0e f2 31 de 58 5a a1 10 4d e8 e1 ba 03 00 00 00 
> bf 9c 2a e8 e1 89 d1 c7 40 14 12 00 00 00 8b 75 08 49 78 08 <ac> ae 75 08 
> 84 c0 75 f5 31 c0 eb 04
> 19 c0 0c 01 85 c0 75 11 a1
> 
> 
> This oops affects all kernels since 2.6.12. Patch follows.
> Please apply.

Is this patch accepted by the acpi maintainers yet?

> +	/* INIT on Samsung's P35 returns an integer, possible return
> +	 * values are tested below */
> +	if (model->type == ACPI_TYPE_INTEGER) {
> +		if (model->integer.value == -1 ||
> +			model->integer.value == 0x58 ||
> +			model->integer.value == 0x38) {
> +			hotk->model = P30;
> +			printk(KERN_NOTICE
> +				       "  Samsung P35 detected, 
> supported\n");

Linewrapped :(

Are you sure that if logic is correct?

thanks,

greg k-h
-
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/