Gallopsled / pwntools

CTF framework and exploit development library
http://pwntools.com
Other
11.99k stars 1.7k forks source link

Redirect kitty kill command stderr to /dev/null #2472

Closed k4lizen closed 5 days ago

k4lizen commented 1 week ago

If one closes the window then quickly Ctrl+C's on the pwntools process they might get a message like Error: No matching windows for expression: id:38

This is also very apparent if the debugee SIGSEGVS, then after closing the debugging window an interactive shell will not close automatically and requires a Ctrl+C, always getting the error message.

We don't want to show this error message to the user since in reality everything is fine and it will just confuse them.

Note that this doesn't fix the weird bytes that get printed when closing the process.

peace-maker commented 1 week ago

I don't think this needs a change log entry. I'll let this cook a bit while you battle test it :D

k4lizen commented 1 week ago

kay, leave it for a few weeks then I'll try to look into the printed bytes bug and push it here as well, just dont have the time for that ATM

peace-maker commented 5 days ago

Ah, lets just merge this now and not postpone it artificially any further. 😆