Changeset 101 for trunk/bootcode/regular/other.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/regular/other.asm (modified) (1 diff)

trunk/bootcode/regular/other.asm

@@ -562 +562 @@
         ret
 MBR_TeletypeSyncPos     EndP
+
+; Check if a memory block is all zeros
+; IN  : BX pointer to memblock
+;       CX length to check, zero length is interpreted as block is zero
+; OUT : ZF=1 block if all zeros
+; NOTE: Segment used is DS, which should be the same as ES
+IsMemBlockZero  Proc    Near    Uses ax di
+        push    es          ; Save ES just to be sure
+        push    ds          ; Segment to use
+        pop     es          ; Pop in ES because ES is required for scasb
+        mov     di, bx      ; Pointer to memblock
+        xor     al, al      ; Compare to zero
+        repe    scasb       ; Scan the block, will leave ZF=1 if all zeros
+        mov     bx, di      ; Update pointer (points past last byte compared)
+        pop     es          ; Restore ES
+        ret
+IsMemBlockZero  EndP
+
+; Check if a loaded sector is all zeros
+; IN  : BX pointer to memblock
+; OUT : ZF=1 block if all zeros
+; NOTE: Segment used is DS, which should be the same as ES
+IsSectorZero    Proc    Near    Uses cx
+        mov     cx, sector_size ; Normal size of a sector (512 bytes)
+        call    IsMemBlockZero  ; Check the memory block
+        ret
+IsSectorZero    EndP