Changeset 101 for trunk/bootcode/special/lvm.asm

Timestamp:

Apr 8, 2017, 12:26:55 AM (8 years ago)

Author:

Ben Rietbroek

Message:

Prevent LVM CRC-check indicating OK on zero sectors [v1.1.1-testing]

When 'LVM_InitCRCTable' is not called yet, an LVM CRC-check on a sector
with all zeros would indicate a valid LVM CRC. Eventhough this is not
the case _after_ 'LVM_InitCRCTable' has been called, we rule out this
situation to guard agains coding bugs.

CAUTION:
This is a testbuild !
AirBoot uses the BIOS to access disks and a small coding error can trash
partition tables or other vital disk structures. You are advised to make
backups of TRACK0 and EBRs before using this testbuild. More info at:
​https://rousseaux.github.io/netlabs.air-boot/pdf/AirBoot-v1.1.0-manual.pdf

File:

• trunk/bootcode/special/lvm.asm (modified) (1 diff)

trunk/bootcode/special/lvm.asm

@@ -112 +112 @@
 ;       Out: Carry set, if LVM CRC valid
 ; Destroyed: None
-LVM_CheckSectorCRC              Proc Near   Uses ax dx
-        call    LVM_GetSectorCRC
+LVM_CheckSectorCRC              Proc Near   Uses ax bx dx
+        mov     bx, si
+        call    IsSectorZero                ; Zero sector implies bad CRC
+        jz      LVMCSCRC_BadCRC
+        call    LVM_GetSectorCRC            ; Only use after CRC table is valid
         cmp     ax, word ptr [si+LocLVM_CRC]
         jne     LVMCSCRC_BadCRC
         cmp     dx, word ptr [si+LocLVM_CRC+2]
         jne     LVMCSCRC_BadCRC
-        stc
+        stc                                 ; Indicate CRC is OK
         ret
     LVMCSCRC_BadCRC:
-        clc
+        clc                                 ; Indicate BAD CRC
         ret
 LVM_CheckSectorCRC              EndP