cvs commit: src/sys/contrib/dev/acpica exstore.c

Nate Lawson nate at root.org
Wed Jan 14 10:55:53 PST 2004


On Wed, 14 Jan 2004, Nate Lawson wrote:
>   Modified files:        (Branch: INTEL)
>     sys/contrib/dev/acpica exstore.c
>   Log:
>   When creating a package element via "Store(0xXXX, Index(ArgX, 0xXXX))",
>   be sure to increment the refcount of the argument so it is not
>   prematurely deleted.  This is a workaround and may appear in a different
>   form in ACPI-CA.  This fixes battery evaluation on Thinkpads that was
>   broken by fixing the Dell battery state.
>
>   Submitted by:   Luming Yu <luming.yu at intel.com>
>
>   Revision  Changes    Path
>   1.1.1.23  +1 -1      src/sys/contrib/dev/acpica/exstore.c
>
> --- src/sys/contrib/dev/acpica/exstore.c:1.1.1.22	Mon Dec  8 18:43:40 2003
> +++ src/sys/contrib/dev/acpica/exstore.c	Wed Jan 14 10:54:22 2004
> @@ -389,7 +389,7 @@
>
>              /* If same as the original source, add a reference */
>
> -            if (NewDesc == SourceDesc)
> +            if (NewDesc == SourceDesc || ObjDesc == NULL)
>              {
>                  AcpiUtAddReference (NewDesc);
>              }
>

Ok, done on the right branch this time.  Thanks to markm for repo work to
fix my mistake.

-Nate


More information about the cvs-all mailing list