Cutter: Redirect STDOUT to Console Widget for Python Plugins

[ITAYC0HEN]

Is your feature request related to a problem? Please describe.

Currently, when Plugin author uses print("hello, world"), the output would be executed on the terminal which was executing Cutter (on windows usually there won't be a terminal for this).

This is unexpected behavior since we have a Console Widget

Describe the solution you'd like
I want STDOUT to be redirected to the Console widget. In general, but specifically here for Python Plugins.

Additional context

related issue: #966

[thestr4ng3r]

Related: https://github.com/radare/radare2/pull/13622 (to not clutter the output as much)

[pelijah]

We have CutterCore::message()
https://github.com/radareorg/cutter/blob/master/src/core/Cutter.cpp#L722

[pelijah]

So you can just use cutter.message() instead of print().

[ITAYC0HEN]

But people use print, which is okay, should be redirected to Cutter.Message

[ITAYC0HEN]

And there are more ways to generate output to STDOUT and STDERR in python (and any other thingies), not only prints. Even when python throws exception I'd expect it to be on Console widget and not on the executing Terminal (which not always exists)

[pelijah]

Python and plugins are initialized before MainWindow. So we have to change the order.

[thestr4ng3r]

We can have some kind of buffer for output received before the MainWindow has been started. There might be something interesting in it, even without Python Plugins.

[gaasedelen]

I agree that this should be the default. Existing interactive disassembler platforms already behave this way, so now all of your users will expect Cutter to behave the same way whether you like it or not 😅

Might be best not to overthink this. IDA has a pretty straightforward implementation that you can draw from for Cutter.

They hook the interpreter's stdout/stderr stream with a simple proxy object:

https://github.com/idapython/src/blob/15b9ab2535222e492cd21b8528c27f763fb799d6/python/init.py#L98-L101

The proxy simply passes along any file descriptor writes to idaapi.msg(), eg cutter.message():

https://github.com/idapython/src/blob/15b9ab2535222e492cd21b8528c27f763fb799d6/python/init.py#L59-L62

[ITAYC0HEN]

@gaasedelen sounds great! Want to have a look and try solve it?
Actually, maybe this can be fixed globally somehow for the whole application https://github.com/radareorg/cutter/issues/966

[thestr4ng3r]

Yes, this should not be done in python. There are ways to replace/redirect the actual stderr/stdout fds.

[yossizap]

Working on this issue in PR #1821. If you have any additional suggestions or recommendations based on your experience please add them there.

[ITAYC0HEN]

The pull request https://github.com/radareorg/cutter/pull/1821 is not merged but we still did not solve the issue for python printings