fkie-cad / cwe_checker

cwe_checker finds vulnerable patterns in binary executables
https://docs.cwe-checker.io
GNU Lesser General Public License v3.0
1.1k stars 115 forks source link

Feature Request: PE File Support #258

Open MrSynAckSter opened 2 years ago

MrSynAckSter commented 2 years ago

The documentation claims that the system only supports ELF binaries.

It would be very useful to me if the system supported PE files.

Enkelmann commented 2 years ago

I would also very much like better PE-support! :-)

For future reference some notes on the current state of PE support (at the time of this comment):

Unfortunately, ELF files have priority for us right now and we do not have enough people working on the project to tackle better PE support at the same time. At some point we are going to work on this, but I cannot yet say when this will be. In the meantime I would be happy to help and mentor any outside contributors willing to work on the subject!

MrSynAckSter commented 2 years ago

That's awesome to know. I am currently trying to improve my static analysis skills. If you have any suggestions for contributing to the project, I'd be glad to help. I'll trying and bang my head against https://github.com/fkie-cad/cwe_checker/issues/250 and see if I can't understand how your system works.

Enkelmann commented 2 years ago

Feel free to ask many questions about the things you don't understand. :-) For #250 a possible starting point would be to look at the use cases of the Project::get_standard_calling_convention method and to figure out how these cases should be handled for PE files. Another starting point is to just take a simple example binary and trying to figure out what happens internally by using lots of debug printing.