Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fuzz: add x86 instructions decoder harness #162

Merged
merged 1 commit into from
Nov 27, 2023

Conversation

p4zuu
Copy link
Collaborator

@p4zuu p4zuu commented Nov 23, 2023

The x86 instructions decoder is very convenient attack surface.

For the moment, the decoder code is only reachable from CPL-0, but it will be reachable from less privileged code when CPL-3 will be supported. The decoder is currently reachable from a #VC exception.

Copy link
Member

@00xc 00xc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good overall, just a couple of suggestions.

fuzz/fuzz_targets/insn.rs Outdated Show resolved Hide resolved
fuzz/fuzz_targets/insn.rs Outdated Show resolved Hide resolved
The x86 instructions decoder is very convenient attack surface.

For the moment, the decoder code is only reachable from CPL-0, but it
will be reachable from less privileged code when CPL-3 will be
supported. The decoder is currently reachable from a #VC exception.

Signed-off-by: Thomas Leroy <thomas.leroy@suse.com>
@p4zuu p4zuu force-pushed the insn_decoder_fuzzing branch from 56c3702 to 3fbe0e2 Compare November 23, 2023 12:49
@joergroedel joergroedel merged commit 881daa2 into coconut-svsm:main Nov 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants