-
Notifications
You must be signed in to change notification settings - Fork 425
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Updates RAT King Parser to commit b85abe5 #2302
Updates RAT King Parser to commit b85abe5 #2302
Conversation
hello, oh nice, thank you for update, i will try to review it in few days and add tests |
c85eaae
to
3cc89f2
Compare
@doomedraven : Thank you again so much for the time you spent reviewing, as well as the samples you sent over. I was planning on doing a refactor of RKP for a while now, and after seeing your comments in private chat, I decided that now was as good of a time as any, and spent the last few weeks refactoring RKP to what is now v3.0.0. I've now updated this PR with that new code, and whenever you have the time, I'd love to know if this passes with the samples that were giving you trouble before (and hopefully it hasn't broken anything new😄 ). Please let me know if you run into any issues or see anything out of sorts. Thanks as always to you and the team! |
Right I've begun reviewing this, and have ended up having to create a PR to make some changes. This seems over-complicated when my changes were so trivial but I couldn't work out a better way. I have also pushed some corresponding updates to the yara sigs in community repo (although I'm considering moving these to main repo as I think if the main repo has a parser for a family, the yara should be there too). |
Not sure what has gone wrong and why my mods haven't appeared here yet... |
I cannot be bothered to try to understand why it didn't work so I will just make the minimal changes in line! Thank you @jeFF0Falltrades for all this work! |
Thanks to you, @kevoreilly , for the diligent review! |
Hi CAPE Team!
This PR brings the CAPEv2 implementation of RAT King Parser (ported over by @doomedraven - thank you again 💖) to the latest commit of the RAT King Parser.
The updates included introduce a few key changes to the parser that produced better reliability of the parser across a large sample set of AsyncRAT/DcRAT/VenomRAT/QuasarRAT/XenoRAT/XWorm samples, particularly among obfuscated samples.
I also swapped out the existing AsyncRAT and XWorm parsers in CAPEv2 with the RAT King Parser, as I believe RKP will be more robust than the existing configuration parsers as it's designed to not rely on specific config field names (as these are often obfuscated or changed by malware authors); However, if you would prefer to keep those existing parsers, please feel free to revert my changes.
I tried to keep the changes minimal and align them with the CAPEv2 style guide, but would appreciate someone checking the format and testing the configuration parsers within CAPE as I do not have a local instance stood up yet to test on.
Lastly, I am using a specific set of YARA rules in the non-ported version of RAT King Parser to detect payloads that can be extracted with RKP; You can find those in the RKP repository, or in my YARA signature repo, if they are needed for testing, and the following samples (found in the RKP README) can also be downloaded and used for testing:
Please let me know if there are any questions or edits to be made, and thank you for your continued contributions to the community!