CAPEv2 icon indicating copy to clipboard operation
CAPEv2 copied to clipboard

YungBinary Add Stealc Config Extractor

Open YungBinary opened this issue 1 year ago • 3 comments

Here's a preview of config extraction :) been trying to find a job doing this full time so if anyone has any leads lmk!

Screenshot from 2024-10-23 09-38-52

YungBinary avatar Oct 23 '24 09:10 YungBinary

Can you please add test for the extractor too?

doomedraven avatar Oct 23 '24 11:10 doomedraven

Can you please add test for the extractor too?

On it, thanks!

YungBinary avatar Oct 23 '24 18:10 YungBinary

@doomedraven Okay I've added the tests including one for lumma, here's the PR for the test files repo: https://github.com/CAPESandbox/CAPE-TestFiles/pull/10 planning to re-run the failed test here once that PR is merged

YungBinary avatar Oct 23 '24 19:10 YungBinary

Thanks for this contribution 🙏 Unfortunately I am not keen on the idea of having decrypted strings in the config - for dynamically extracted strings there is already the facility to have them appear as a text file in the payloads tab. I am working with doomed on a potential mechanism to have strings dumped from static parsers to appear in a more suitable place than the config. For this reason I will hold off for now on merging this, and will be on leave for a week or so. But this addition is very much appreciated, I would request your patience while we work out the details.

kevoreilly avatar Oct 24 '24 14:10 kevoreilly

Thanks for this contribution 🙏 Unfortunately I am not keen on the idea of having decrypted strings in the config - for dynamically extracted strings there is already the facility to have them appear as a text file in the payloads tab. I am working with doomed on a potential mechanism to have strings dumped from static parsers to appear in a more suitable place than the config. For this reason I will hold off for now on merging this, and will be on leave for a week or so. But this addition is very much appreciated, I would request your patience while we work out the details.

Thanks for the update. The goal was to have output useful for reverse engineers so they can pop the sample into IDA and be able to map labels like dword_x to specific strings, otherwise the strings aren't very meaningful on their own.

YungBinary avatar Oct 24 '24 20:10 YungBinary

Yep absolutely - useful output no doubt. It's that the place for such output is really somewhere like the payload tab, just not the config.

kevoreilly avatar Oct 24 '24 20:10 kevoreilly

Yep absolutely - useful output no doubt. It's that the place for such output is really somewhere like the payload tab, just not the config.

Okay gotcha that sounds good. I'll keep an eye out for DMs or messages here in the event y'all find something suitable and I need to make changes. Also, just wrote up a KoiLoader config extractor and test and added to this PR as I can't fork the repo twice but can remove if it should be in its own PR, thanks!

YungBinary avatar Oct 24 '24 21:10 YungBinary

Hello, I would suggest to split commits per parser, as having both parsers here will be on hold till we have time to work on strings instead of directly merge the second one

doomedraven avatar Oct 25 '24 05:10 doomedraven

you don't need to fork repo twice, you can work on different branches

doomedraven avatar Oct 25 '24 06:10 doomedraven

@kevoreilly as quick workout for this i would just kick strings from web gui(allowing users to enable that in config if they want). so we can merge this and work on the move of strings properly once we have more spare time

doomedraven avatar Oct 25 '24 06:10 doomedraven

i have disabled string storing for now, we will be able to restore that later, so i can merge this PR :), thanks again @YungBinary

doomedraven avatar Oct 25 '24 08:10 doomedraven