Skip to content
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

New analyser: File Strings #314

Closed
wants to merge 2 commits into from

Conversation

j-i-m-b-o
Copy link

A simple analyser to find printable strings within files.

I've personally found this analyser to be a useful first step when dealing with various types of unknown file so wanted to make it available to others.

@j-i-m-b-o j-i-m-b-o mentioned this pull request Jul 27, 2018
@jeromeleonard jeromeleonard changed the base branch from master to develop October 21, 2018 08:44
Copy link
Contributor

@3c7 3c7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added python3 compatibility

@3c7 3c7 added this to the 1.14.0 milestone Oct 23, 2018
@jeromeleonard jeromeleonard removed this from the 1.14.0 milestone Oct 24, 2018
@geekscrapy
Copy link

It may also be good to take a look at this at the same time:
https://github.com/fireeye/flare-floss

@jeromeleonard
Copy link
Contributor

I propose to add this feature as a submodule in FileInfo.

@jeromeleonard jeromeleonard added this to the 2.6.0 milestone Jan 20, 2020
@dadokkio dadokkio self-assigned this Mar 13, 2020
@dadokkio
Copy link
Contributor

I've added the feature as submodule. I just need to finish templates.
For flare-floss the support must wait because it doesn't support python3 yet

@geekscrapy
Copy link

So, if you use this version it's not floss per-say, but it's more capable than basic strings, and it can rank the strings in order of maliciousness with rankstrings

https://github.com/fireeye/stringsifter

@geekscrapy
Copy link

Also, could always use the binary version of floss?

@dadokkio
Copy link
Contributor

Ok, I'll try with the binary tomorrow.
In any case template for strings looks very strange and way too long.

This is a preview:
image

@3c7
Copy link
Contributor

3c7 commented Mar 14, 2020

I've already added FLOSS as a submodule for FileInfo: https://github.com/TheHive-Project/Cortex-Analyzers/tree/feature/fileinfo_floss

@3c7
Copy link
Contributor

3c7 commented Mar 14, 2020

(That's why I was assigned ;) )

@dadokkio
Copy link
Contributor

sorry, I didn't knew that and it seems way better than my first attempt with floss :)

@geekscrapy
Copy link

geekscrapy commented Mar 14, 2020

I'd say, if you want to at least have the most useful stuff at the top of the long list, use rankstrings.py on the generated output from strings. Might help with the 'long list' situation in the template...

@dadokkio
Copy link
Contributor

@3c7 will you propose a pull request with your changes?

@3c7
Copy link
Contributor

3c7 commented Mar 17, 2020

Need more time to fix the template. Currently no time to work on Analyzer stuff.

@dadokkio
Copy link
Contributor

no problem. The plan is to release milestone 2.6.0 this friday, if you need more time I'll move your pr to next release 👍

@jeromeleonard jeromeleonard modified the milestones: 2.6.0, 2.7.0 Mar 19, 2020
@jeromeleonard jeromeleonard modified the milestones: 2.7.0, 2.8.0 Apr 24, 2020
@jeromeleonard jeromeleonard modified the milestones: 2.8.0, 2.9.0 Jul 2, 2020
@To-om To-om force-pushed the develop branch 3 times, most recently from fb8f5aa to 23be632 Compare July 29, 2020 15:56
@garanews garanews modified the milestones: 2.9.0, 3.0.0 Aug 12, 2020
@dadokkio
Copy link
Contributor

I've added a fix for the template in your repo.
Probably this should resolve the issue (related to duplicated items in the ng-repeat loop)

image

@dadokkio dadokkio mentioned this pull request Dec 14, 2020
@garanews garanews closed this Mar 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope:analyzer Issue is analyzer related status:pr-submitted
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants