These are chat archives for BinaryAnalysisPlatform/qira

27th
Apr 2016
turnersr
@turnersr
Apr 27 2016 11:22
Why you no merge?!?!?! :(
serena159
@serena159
Apr 27 2016 12:39
i don't know .. i suffer typing address
Yahya Lmallas
@Maroc-OS
Apr 27 2016 21:44
@serena159 take a look at PR #172
try the commands for the plugins too
Yahya Lmallas
@Maroc-OS
Apr 27 2016 22:09
@nedwill
Humm! i think data address also needs checking against BADADDR and None.
there is another PR to fix this if you are with me at that point
Ned Williamson
@nedwill
Apr 27 2016 22:10
sorry, the maintainers are pretty busy lately. probably more geohot than me but i'm graduating soon so things should get moving again
@serena159 we agree PIE needs to be fixed
@Maroc-OS I agree, do you mind just putting that commit with the current PR?
Yahya Lmallas
@Maroc-OS
Apr 27 2016 22:12
yep
Ned Williamson
@nedwill
Apr 27 2016 22:13
was that leading to a crash for you? we had crashes and assumed it was due to thread-unsafe API usage from the python version of the IDA plugin but I'd absolutely love to kill off the c++ one if that's not the case, or at least move the more complex functionality to the python one
there are bugs in the C++ side of IDA plugins that I suspect aren't there in the Python side as more people use IDAPython
Yahya Lmallas
@Maroc-OS
Apr 27 2016 22:15
For me i did not got any crashes on the cpp plugin
but i agree with you about the python one
Ned Williamson
@nedwill
Apr 27 2016 22:16
Ok that's good, I think geohot reported he saw a crash but he may have been using a previous buggy one.
Yahya Lmallas
@Maroc-OS
Apr 27 2016 22:16
btw am using IDA SDK 6.8 So you should be sure of the version
Check the PR now
Ned Williamson
@nedwill
Apr 27 2016 22:17
6.8 should be fine for the IDA plugin as long as you don't call anything too recent. and i'm checking now
I'm getting IDA crashes when I scroll
Yahya Lmallas
@Maroc-OS
Apr 27 2016 22:18
yep and i think i have already modified the cpp to be compatible with 6.8 if i still remember
that's why i don't get crashes maybe?
Ned Williamson
@nedwill
Apr 27 2016 22:18
I'm referring to the python plugin
have you tested it?
Yahya Lmallas
@Maroc-OS
Apr 27 2016 22:19
i tested linux and windows binaries and they seems good
yeah for the python one he works like a charm
Ned Williamson
@nedwill
Apr 27 2016 22:19
see if scrolling up and down in the qira view triggers a bug (likely a race condition)
I'm on Mac btw
Yahya Lmallas
@Maroc-OS
Apr 27 2016 22:19
let me do that again to be sure
test again with the new version of the python plugin
yeah :) it seems okay now
let me check the data section
Yahya Lmallas
@Maroc-OS
Apr 27 2016 22:25
yeah a little crash ll debug that and see what i can do
Ned Williamson
@nedwill
Apr 27 2016 22:26
Yeah I think it has to do with thread-safety of the bpt add/remove APIs, and perhaps others
Yahya Lmallas
@Maroc-OS
Apr 27 2016 22:27
yeah maybe
Ned Williamson
@nedwill
Apr 27 2016 22:27
I'm 99% certain that's the problem
it looks like people have worked around it since I last looked into it - see http://www.williballenthin.com/blog/2015/09/04/idapython-synchronization-decorator/
perhaps you can incorporate that into the PR. if you can't figure it out I'll have a look
Yahya Lmallas
@Maroc-OS
Apr 27 2016 22:29
oh good link
let me investigate on it