Add C callback in r_bp

[radare]

Used for tracing, breakpointing etc, and running code from inside the api. this will make the debugger loop async and we will require to add other callbacks to handle other exceptions instead of checking right after the waitpid() thing

[XVilka]

@jduck your thoughts?

[jduck]

I'll take this into consideration surely.

[jduck]

After thinking some more about the issue, I think it's quite doable to have a callback function that gets called whenever a breakpoint is hit. That said, such a thing is only useful when using the r_debug API directly via C or bindings, right? In all likelihood the code would be executed right before "cmd.bp" is executed, so it wouldn't exactly be async. If something bad happens during processing of this breakpoint all bets would be off. For example, the user could kill the child or detach the inferior. If they do that, things are going to get really ugly when we attempt to proceed processing the breakpoint handling...

[radare]

yep. that callback will be only usable when using the api, but my idea was to use it interanlly because of those hairy corners

On 16 May 2016, at 20:52, Joshua J. Drake notifications@github.com wrote:

After thinking some more about the issue, I think it's quite doable to have a callback function that gets called whenever a breakpoint is hit. That said, such a thing is only useful when using the r_debug API directly via C or bindings, right? In all likelihood the code would be executed right before "cmd.bp" is executed, so it wouldn't exactly be async. If something bad happens during processing of this breakpoint all bets would be off. For example, the user could kill the child or detach the inferior. If they do that, things are going to get really ugly when we attempt to proceed processing the breakpoint handling...

—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub #946 (comment)

[jduck]

Luckily the handling of when breakpoints are hit is much simpler than continuing afterward. hehe.

[radare]

ping

[radare]

@jduck ping?

[jduck]

Sorry, no time lately :-/

[radare]

cc @leberus