Re: PR/43504 CVS commit: src/sys/dev/acpi/wmi

[David DEMELIER <demelier.david%gmail.com@localhost>]

2010/6/30 Jukka Ruohonen <jruoho%netbsd.org@localhost>:
> The following reply was made to PR kern/43504; it has been noted by GNATS.
>
> From: Jukka Ruohonen <jruoho%netbsd.org@localhost>
> To: gnats-bugs%gnats.NetBSD.org@localhost
> Cc:
> Subject: PR/43504 CVS commit: src/sys/dev/acpi/wmi
> Date: Wed, 30 Jun 2010 07:58:11 +0000
>
> ÂModule Name: Â src
> ÂCommitted By: Âjruoho
> ÂDate: Â Â Â Â ÂWed Jun 30 07:58:11 UTC 2010
>
> ÂModified Files:
> Â Â Â Âsrc/sys/dev/acpi/wmi: wmi_dell.c wmi_hp.c
>
> ÂLog Message:
> ÂInitialize pointers to NULL. Otherwise we may try to free garbage.
> ÂTries to address a panic reported by David Demelier in PR kern/43504.
>
>
> ÂTo generate a diff of this commit:
> Âcvs rdiff -u -r1.3 -r1.4 src/sys/dev/acpi/wmi/wmi_dell.c
> Âcvs rdiff -u -r1.1 -r1.2 src/sys/dev/acpi/wmi/wmi_hp.c
>
> ÂPlease note that diffs are not public domain; they are subject to the
> Âcopyright notices on the relevant files.
>

Tested, it does not panic anymore but it doesn't print any output and
does not adjust brightness. I guess this laptop is impossible to
maintain.

-- 
Demelier David