Author: jkim
Date: Thu Apr  9 21:26:23 2015
New Revision: 281331
URL: https://svnweb.freebsd.org/changeset/base/281331

Log:
  Do not crash when RSDT/XSDT contains an empty entry.
  
  Reported by:  Slawa Olhovchenkov <s...@zxy.spb.ru>
  MFC after:    3 days

Modified:
  head/usr.sbin/acpi/acpidump/acpi.c

Modified: head/usr.sbin/acpi/acpidump/acpi.c
==============================================================================
--- head/usr.sbin/acpi/acpidump/acpi.c  Thu Apr  9 21:23:27 2015        
(r281330)
+++ head/usr.sbin/acpi/acpidump/acpi.c  Thu Apr  9 21:26:23 2015        
(r281331)
@@ -1085,17 +1085,10 @@ acpi_print_rsdt(ACPI_TABLE_HEADER *rsdp)
        for (i = 0; i < entries; i++) {
                if (i > 0)
                        printf(", ");
-               switch (addr_size) {
-               case 4:
+               if (addr_size == 4)
                        addr = le32toh(rsdt->TableOffsetEntry[i]);
-                       break;
-               case 8:
+               else
                        addr = le64toh(xsdt->TableOffsetEntry[i]);
-                       break;
-               default:
-                       addr = 0;
-               }
-               assert(addr != 0);
                printf("0x%08lx", addr);
        }
        printf(" }\n");
@@ -1342,17 +1335,12 @@ acpi_handle_rsdt(ACPI_TABLE_HEADER *rsdp
        xsdt = (ACPI_TABLE_XSDT *)rsdp;
        entries = (rsdp->Length - sizeof(ACPI_TABLE_HEADER)) / addr_size;
        for (i = 0; i < entries; i++) {
-               switch (addr_size) {
-               case 4:
+               if (addr_size == 4)
                        addr = le32toh(rsdt->TableOffsetEntry[i]);
-                       break;
-               case 8:
+               else
                        addr = le64toh(xsdt->TableOffsetEntry[i]);
-                       break;
-               default:
-                       assert((addr = 0));
-               }
-
+               if (addr == 0)
+                       continue;
                sdp = (ACPI_TABLE_HEADER *)acpi_map_sdt(addr);
                if (acpi_checksum(sdp, sdp->Length)) {
                        warnx("RSDT entry %d (sig %.4s) is corrupt", i,
@@ -1547,16 +1535,12 @@ sdt_from_rsdt(ACPI_TABLE_HEADER *rsdp, c
        xsdt = (ACPI_TABLE_XSDT *)rsdp;
        entries = (rsdp->Length - sizeof(ACPI_TABLE_HEADER)) / addr_size;
        for (i = 0; i < entries; i++) {
-               switch (addr_size) {
-               case 4:
+               if (addr_size == 4)
                        addr = le32toh(rsdt->TableOffsetEntry[i]);
-                       break;
-               case 8:
+               else
                        addr = le64toh(xsdt->TableOffsetEntry[i]);
-                       break;
-               default:
-                       assert((addr = 0));
-               }
+               if (addr == 0)
+                       continue;
                sdt = (ACPI_TABLE_HEADER *)acpi_map_sdt(addr);
                if (last != NULL) {
                        if (sdt == last)
_______________________________________________
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to