Fix error handling in spihal and debugenabled/mmconfig modules

Signed-off-by: Nathaniel Mitchell <nathaniel.p.mitchell@intel.com>

Author: npmitche

chipsec/hal/intel/spi.py

@@ -51,7 +51,7 @@
 from chipsec.library.logger import print_buffer_bytes
 from chipsec.hal import hal_base
 from chipsec.library.spi_jedec_ids import JEDEC_ID
-from chipsec.library.exceptions import SpiRuntimeError
+from chipsec.library.exceptions import SpiRuntimeError, HALInitializationError
 from chipsec.library.intel.spi import SPI_REGION, SPI_FLA_SHIFT, SPI_FLA_PAGE_MASK, SPI_REGION_NAMES, SPI_REGION_tuple, print_SPI_Flash_Regions
 
 SPI_READ_WRITE_MAX_DBC = 64

chipsec/modules/common/debugenabled.py

@@ -77,6 +77,9 @@ def check_dci(self) -> int:
         self.logger.log('')
         self.logger.log('[*] Checking DCI register status')
         ectrl = self.cs.register.get_list_by_name('ECTRL')
+        if not ectrl:
+            self.logger.log_important('ECTRL register not found. Skipping DCI check.')
+            return ModuleResult.WARNING
         ectrl.read_and_verbose_print()
         hdcien_mask = ectrl[0].get_field_mask('ENABLE', True)
 
@@ -95,6 +98,9 @@ def check_cpu_debug_enable(self) -> int:
         self.logger.log('[*] Checking IA32_DEBUG_INTERFACE MSR status')
         TestFail = ModuleResult.PASSED
         dbg_regs = self.cs.register.get_list_by_name('IA32_DEBUG_INTERFACE')
+        if not dbg_regs:
+            self.logger.log_important('IA32_DEBUG_INTERFACE MSR not found. Skipping CPU debug enable check.')
+            return ModuleResult.WARNING
         dbg_regs.read_and_verbose_print()
 
 

chipsec/modules/common/memconfig.py

@@ -94,6 +94,10 @@ def check_memmap_locks(self) -> int:
                 self.logger.log_important(f'{reg}.{self.memmap_registers[reg]} not defined for platform.  Skipping register.')
                 continue
             reglist = self.cs.register.get_list_by_name(reg)
+            if not reglist:
+                all_locked = False
+                self.logger.log_important(f'{reg} register not found. Unable to verify lock state.')
+                continue
             description = reglist[0].desc
             reglist.read_and_verbose_print()
             if reglist.is_all_field_value(1, self.memmap_registers[reg]):