ret-sync icon indicating copy to clipboard operation
ret-sync copied to clipboard

[BN] Update Dock widget to Sidebar widget

Open clubby789 opened this issue 8 months ago • 4 comments

Closes #130

I still get

[ScriptingProvider] [sync] set debugger dialect to gdb, enabling hotkeys
[ScriptingProvider] [sync] bp failed, remote base of None program unknown
[ScriptingProvider] [sync] client quit

when trying to place a breakpoint - not sure if this is to do with my changes, other errors or if I'm just using the plugin wrong.

clubby789 avatar Jun 03 '25 16:06 clubby789

Hi @clubby789, have you been been able to successfully use ret-sync with BinaryNinja and your changes at the end?

saidelike avatar Sep 29 '25 09:09 saidelike

Can't get it working with this PR in BN 5.1.8104-stable

[ScriptingProvider] Exception in thread Thread-4:
[ScriptingProvider] Traceback (most recent call last):
[ScriptingProvider]   File "C:\Users\CG\AppData\Roaming\Binary Ninja\plugins\retsync\sync.py", line 360, in run
[ScriptingProvider]     self.plugin.reset_client()
[ScriptingProvider]   File "C:\Users\CG\AppData\Roaming\Binary Ninja\plugins\retsync\sync.py", line 535, in reset_client
[ScriptingProvider]     self.widget.reset_client()
[ScriptingProvider] AttributeError: 'NoneType' object has no attribute 'reset_client'
[ScriptingProvider] During handling of the above exception, another exception occurred:
[ScriptingProvider] Traceback (most recent call last):
[ScriptingProvider]   File "threading.py", line 1016, in _bootstrap_inner
[ScriptingProvider]   File "C:\Users\CG\AppData\Roaming\Binary Ninja\plugins\retsync\sync.py", line 366, in run
[ScriptingProvider]     self.plugin.cmd_syncoff()
[ScriptingProvider]   File "C:\Users\CG\AppData\Roaming\Binary Ninja\plugins\retsync\sync.py", line 786, in cmd_syncoff
[ScriptingProvider]     self.widget.reset_status()
[ScriptingProvider] AttributeError: 'NoneType' object has no attribute 'reset_status'

tGecko avatar Oct 03 '25 10:10 tGecko

fix_widget_null_reference_errors.txt

This changes are needed to make it work again in BN 5.1.8104-stable

tGecko avatar Oct 03 '25 12:10 tGecko

Thanks @tGecko. Could you test the updated PR please? If it works as expected, I will merge it in the master branch

saidelike avatar Nov 15 '25 21:11 saidelike