search

geohot / qira

QEMU Interactive Runtime Analyser
MIT License
3.95k stars 469 forks source link

Patched qira to work on modern and arch based systems #251

Closed pAplakidis closed 3 years ago

pAplakidis commented 3 years ago

I used a repo i found for ARCH and I patched it. So not only does ./install.sh run error-free (I hope so at least) it also runs on ARCH based systems. NOTE: I did not port qira to arch, I found the repository i mentioned and patched the files so that ./install.sh (specificly ./qemu_build.sh) work. Hope this helps since the community (including me) had a lot of problems running qira. It is an amazing tool so I needed it to be accessible to me and others who had trouble installing

geohot commented 3 years ago

Holy crap that's a huge diff

pAplakidis commented 3 years ago

Holy crap that's a huge diff

I know, I realised I made a mistake, I actually deleted everything from my fork then replaced it with the repo I mentioned and then "patched" qemu to not show compilation errors. Basically I believe the only changes are in install.sh which makes it install in multiple distros (like arch) and in qemu c files. Again, my bad there, I made it work while many people had errors and I just thought I could help since this is a very useful tool for binary exploitation and debugging.

At the end of the day just ignore this pull request, my fork works but it was not by changing your repo but the one I found in AUR. Big mistake using "git add ." on forks instead of specifying the files, was my first pull request so I didn't think of it then, might try again so that the didd is not that big, but i don't know how much the other repo I used has changed to your original.