HookId trait and types in libafl_qemu

[rbran]

fix: #1795

This created HookType and add it to HookId, what allows the function remove_hook to call the proper libafl_qemu_sys::libafl_qemu_remove*_hook function for the received hook.

[andreafioraldi]

Hi, thank you, hookid is WIP, the plan is to have a type for each hook (InstructionHookId, CmpHookId, PreSyscallHookId, etc...) implementing a trait with a removal method, something like HookId::remove(emu: &Emulator).
In this way, hooks id remain just a wrapper around u64 without taking additional space and they are much better for the type system.
If you want to adapt this PR in this way, I'll be happy to merge it.

[rbran]

Hi, thank you, hookid is WIP, the plan is to have a type for each hook (InstructionHookId, CmpHookId, PreSyscallHookId, etc...) implementing a trait with a removal method, something like HookId::remove(emu: &Emulator). In this way, hooks id remain just a wrapper around u64 without taking additional space and they are much better for the type system. If you want to adapt this PR in this way, I'll be happy to merge it.

I adapted the PR, now it uses a trait and specialized structs. NOTE the trait function is HookId::remove(&self, invalidate_block: bool) -> bool, the emulator reference is not required.

[andreafioraldi]

ty!

[domenukk]

This needs a cargo fmt.
@andreafioraldi merge?