Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 27 Dec 2008 20:50:55 -0800
From:      Andrew Thompson <thompsa@FreeBSD.org>
To:        Stanislav Sedov <stas@FreeBSD.org>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org
Subject:   Re: svn commit: r186529 - head/sys/dev/acpi_support
Message-ID:  <20081228045055.GA81182@citylink.fud.org.nz>
In-Reply-To: <200812272048.mBRKmBKo082102@svn.freebsd.org>
References:  <200812272048.mBRKmBKo082102@svn.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Sat, Dec 27, 2008 at 08:48:11PM +0000, Stanislav Sedov wrote:
> Author: stas
> Date: Sat Dec 27 20:48:11 2008
> New Revision: 186529
> URL: http://svn.freebsd.org/changeset/base/186529
> 
> Log:
>   - Fix incorrect array declaration that was causing the stack overflow
>     on some (most?) Asus laptops.
>   
>   Discussed with:	rpaulo
>   Approved by:	kib (mentor)
>   MFC after:	2 weeks
> 
> Modified:
>   head/sys/dev/acpi_support/acpi_asus.c
> 
> Modified: head/sys/dev/acpi_support/acpi_asus.c
> ==============================================================================
> --- head/sys/dev/acpi_support/acpi_asus.c	Sat Dec 27 20:37:53 2008	(r186528)
> +++ head/sys/dev/acpi_support/acpi_asus.c	Sat Dec 27 20:48:11 2008	(r186529)
> @@ -976,7 +976,7 @@ acpi_asus_sysctl_set(struct acpi_asus_so
>  {
>  	ACPI_STATUS		status = AE_OK;
>  	ACPI_OBJECT_LIST 	acpiargs;
> -	ACPI_OBJECT		acpiarg[0];
> +	ACPI_OBJECT		acpiarg[1];
>  
>  	ACPI_FUNCTION_TRACE((char *)(uintptr_t)__func__);
>  	ACPI_SERIAL_ASSERT(asus);

Thanks! I have had panics that I think are related to this.

Andrew



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20081228045055.GA81182>