mirrors/acpica
1
0
Fork 0
mirror of https://github.com/acpica/acpica/ synced 2025-03-15 10:42:55 +03:00
Code Issues Packages Projects Releases Wiki Activity

Fix to handle zero-length buffers and invalid EndTags (too short)

Browse Source
This commit is contained in:
rmoore1 2006-01-26 21:47:54 +00:00
parent 89ed3bf052
commit f14c19ce5e
1 changed files with 19 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
source/components/utilities/utresrc.c
View File

@ -1,7 +1,7 @@
/*******************************************************************************
*
* Module Name: utresrc - Resource managment utilities
* $Revision: 1.8 $
* $Revision: 1.9 $
*
******************************************************************************/
@ -631,6 +631,7 @@ AcpiUtGetDescriptorLength (
* RETURN: Status, pointer to the end tag
*
* DESCRIPTION: Find the EndTag resource descriptor in an AML resource template
* Note: allows a buffer length of zero.
*
******************************************************************************/
@ -652,6 +653,14 @@ AcpiUtGetResourceEndTag (
Aml = ObjDesc->Buffer.Pointer;
EndAml = Aml + ObjDesc->Buffer.Length;
/* Allow a buffer length of zero */
if (!ObjDesc->Buffer.Length)
{
*EndTag = Aml;
return_ACPI_STATUS (AE_OK);
}
/* Walk the resource template, one descriptor per iteration */
while (Aml < EndAml)
@ -668,6 +677,15 @@ AcpiUtGetResourceEndTag (
if (AcpiUtGetResourceType (Aml) == ACPI_RESOURCE_NAME_END_TAG)
{
/*
* There must be at least one more byte in the buffer for
* the 2nd byte of the EndTag
*/
if ((Aml + 1) >= EndAml)
{
return_ACPI_STATUS (AE_AML_NO_RESOURCE_END_TAG);
}
/* Return the pointer to the EndTag */
*EndTag = Aml;