search

lkrg-org / lkrg

Linux Kernel Runtime Guard
https://lkrg.org
Other
410 stars 72 forks source link

Improve x86 IDT dumping #169

Closed solardiz closed 2 years ago

solardiz commented 2 years ago

Description

This fixes #159 and cleans up the code a little.

How Has This Been Tested?

With these changes, LKRG in the old Xen PV setup appears to work fine. Previously, it crashed. Further, all of out GitHub Actions tests passed (under my personal GitHub account).

However, I did not test on 32-bit x86, and I think we lack such tests in CI. Ideally, we'd also test that, because the cleanup commit replaces a 32-bit specific piece with a new shared one.

solardiz commented 2 years ago

@Adam-pi3 Maybe it'd be quick for you to test this PR's LKRG revision in a 32-bit x86 VM you readily have for LKRG testing? I think that's a reasonable thing to do before merging. Thank you!