radareorg / radare2-r2pipe

Access radare2 via pipe from any programming language!
388 stars 99 forks source link

[python3] r2.quit() hangs execution #135

Closed scmanjarrez closed 3 years ago

scmanjarrez commented 3 years ago

Environment

Python 3.8.11

lun ago 16 13:14:15 CEST 2021 radare2 5.4.0-git 26650 @ linux-x86-64 git.5.4.0-git commit: b0d8f6d3fc1262033af8d7366d99566958c6f9cb build: 2021-08-16__12:53:05 Linux x86_64 1.5.3

Description

Exiting r2 with quit() hangs the program.

Test

pip3 install r2pipe
python3
>>> import r2pipe
>>> r2 = r2pipe.open('/bin/ls')
>>> print(r2.cmd('?e Hello World'))
Hello World
>>> r2.quit()

Peek 2021-08-16 13-24

radare commented 3 years ago

can you pip install -UI r2pipe and try again?

On 16 Aug 2021, at 13:27, s.cmanjarrez @.***> wrote:

Environment

Python 3.8.11

lun ago 16 13:14:15 CEST 2021 radare2 5.4.0-git 26650 @ linux-x86-64 git.5.4.0-git commit: b0d8f6d3fc1262033af8d7366d99566958c6f9cb build: 2021-08-16__12:53:05 Linux x86_64 1.5.3

Description

Exiting radare2 with quit() hangs the program.

Test

pip3 install r2pipe python3

import r2pipe r2 = r2pipe.open('/bin/ls') print(r2.cmd('?e Hello World')) Hello World r2.quit() https://user-images.githubusercontent.com/9284124/129556420-2c4df296-8c27-469d-ac22-51329fa525cc.gif — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/radareorg/radare2-r2pipe/issues/135, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAG75FXQ4Y3DIZCDX3XYLXTT5DYYVANCNFSM5CHRK2TQ. Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email.

scmanjarrez commented 3 years ago

Hi, thank you very much. Version 1.5.5 (40de58ea722e91050c2ee33de3eaaf40fee438c9) now exits correctly. Closing issue.

trufae commented 3 years ago

Awesome! i found another issue and i'm fixing it, so expect 1.5.6 to be published soon. Thanks for reporting! We surely need to add more r2pipe tests in the main repo.