-
Notifications
You must be signed in to change notification settings - Fork 479
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
Sysinject hacks #1283
base: dev
Are you sure you want to change the base?
Sysinject hacks #1283
Conversation
let abe_callback = Callback::new(); | ||
abe_callback.after_block_exec(move |cpu, _, _| { | ||
if panda::current_asid(cpu) == injected_asid { | ||
if !panda::in_kernel_mode(cpu) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ASID//PC pair?```suggestion
if !panda::in_kernel_mode(cpu) {
panda/plugins/sysinject_rs/README.md
Outdated
`sysinject` allows for the injection of syscalls into the guest at arbitrary points. | ||
|
||
The function `inject_syscall` takes 4 arguments | ||
1) `cpu`, the cpu state. This is standard for panda plugins. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Check formatting on github.
Check on MIPS issues |
|
… mess around with typecasting their args
panda/plugins/sysinject/src/lib.rs
Outdated
|
||
#[panda::init] | ||
fn init(_: &mut PluginHandle) -> bool { | ||
#[cfg(any(feature = "x86_64", feature = "aarch64", feature = "mips64", feature = "mipsl"))]{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
mipsel
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this work for mipsel?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Classic mipsel misspell, oops
Everything should be in order for 32 bit arm and x86 support. I left in the broken x86-64 code incase we want to do that in the future, so we have something to work off of, but I can remove it for neatness if need be.