radareorg / radare2-r2pipe

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

Failed to retrieve cmd/cmdj results on python r2pipe 1.0.0 #83

Closed hongxuchen closed 5 years ago

hongxuchen commented 6 years ago

When using r2pipe 1.0.0, it has issues to retrieve cmdj/cmd results. Here are some of the tries. 2018-12-01-004214_1130x736_scrot The behavior is quite different from previous one such as 0.9.9 where we can use cmdj/cmd to get the output directly.

hongxuchen commented 6 years ago

/bin/zgrep is not a good example since it's a script. but binaries such as /bin/tempfile also have this issue.

radare commented 6 years ago

Can you write a test as a pullreq? Because iirc i tried all the tests in the r2pipe repo before publishing the new version. I am not maintaining this code but some ppl was asking me to push this release because they said it was fixing many things.. but seems like its the other way :D

On 30 Nov 2018, at 17:54, hongxu notifications@github.com wrote:

/bin/zgrep is not a good example since it's a script. but binaries such as /bin/tempfile also have this issue.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.

hongxuchen commented 6 years ago

@radare The problem seems specific to Python3; it happens on 3.5.2, 3.6.6, 3.6.7, 3.7.1 on my machine; but works fine on 2.7.15rc1.

a1ext commented 5 years ago

@HongxuChen could you test on 1.1.0 version?

hongxuchen commented 5 years ago

@a1ext It works now!