Module Name:    src
Committed By:   jakllsch
Date:           Sun Jul 24 20:15:09 UTC 2011

Modified Files:
        src/sys/dev/acpi: acpi_ec.c

Log Message:
Fix off-by-one in validation of EmbeddedControl OperationRegion handler.


To generate a diff of this commit:
cvs rdiff -u -r1.70 -r1.71 src/sys/dev/acpi/acpi_ec.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/sys/dev/acpi/acpi_ec.c
diff -u src/sys/dev/acpi/acpi_ec.c:1.70 src/sys/dev/acpi/acpi_ec.c:1.71
--- src/sys/dev/acpi/acpi_ec.c:1.70	Sat Feb 19 19:39:28 2011
+++ src/sys/dev/acpi/acpi_ec.c	Sun Jul 24 20:15:09 2011
@@ -1,4 +1,4 @@
-/*	$NetBSD: acpi_ec.c,v 1.70 2011/02/19 19:39:28 jruoho Exp $	*/
+/*	$NetBSD: acpi_ec.c,v 1.71 2011/07/24 20:15:09 jakllsch Exp $	*/
 
 /*-
  * Copyright (c) 2007 Joerg Sonnenberger <[email protected]>.
@@ -59,7 +59,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: acpi_ec.c,v 1.70 2011/02/19 19:39:28 jruoho Exp $");
+__KERNEL_RCSID(0, "$NetBSD: acpi_ec.c,v 1.71 2011/07/24 20:15:09 jakllsch Exp $");
 
 #include <sys/param.h>
 #include <sys/callout.h>
@@ -651,7 +651,7 @@
 	unsigned int i;
 
 	if (paddr > 0xff || width % 8 != 0 || value == NULL || arg == NULL ||
-	    paddr + width / 8 > 0xff)
+	    paddr + width / 8 > 0x100)
 		return AE_BAD_PARAMETER;
 
 	addr = paddr;

Reply via email to