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

Add test data for hp-socket #220

Closed
wants to merge 2 commits into from

Conversation

Still34
Copy link
Contributor

@Still34 Still34 commented Nov 22, 2023

@williballenthin
Copy link
Collaborator

CI tests warn this file takes a while to analyze (5 minutes?). @Still34 do you think it's feasible to find a smaller sample or is this the best we'll get?

i can poke around on VT later today, too.

@Still34
Copy link
Contributor Author

Still34 commented Nov 22, 2023

Unfotunately this is the only green sample I know of. Technically you can replace the sample with something like the official compiled demo/test files, and it'll match too - it just won't be matching a necessarily malicious file.

@williballenthin
Copy link
Collaborator

thanks!

@Still34 Still34 force-pushed the patches/linking-hp-socket branch from 9500339 to 9084ed5 Compare November 22, 2023 17:02
@Still34
Copy link
Contributor Author

Still34 commented Nov 22, 2023

Huh, weird, vivisect is having a bit of a problem with the ELF. I'll look into it.

@Still34
Copy link
Contributor Author

Still34 commented Nov 22, 2023

Yeah I'm struggling to find a sample that'll play nice with the CI pipeline. Any help?

@williballenthin
Copy link
Collaborator

no problem!

i will take look around on VT and/or we can make an exception here. though, i'll be afk for a couple days - hope that's no problem, just know im not ignoring this thread!

@williballenthin
Copy link
Collaborator

i'm not going to be online much the next few weeks. i recommend we move forward with the non-malicious sample binary. @mr-tz would you help close out these PRs please?

@mr-tz
Copy link
Collaborator

mr-tz commented Dec 1, 2023

I can try to find a sample next week.

@mr-tz
Copy link
Collaborator

mr-tz commented Apr 9, 2024

sample too slow for analysis during testing

@mr-tz mr-tz closed this Apr 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants