Skip to content
This repository has been archived by the owner on Apr 26, 2021. It is now read-only.

Also show local IP addresses #649

Open
jbremer opened this issue Sep 22, 2015 · 7 comments
Open

Also show local IP addresses #649

jbremer opened this issue Sep 22, 2015 · 7 comments
Assignees
Milestone

Comments

@jbremer
Copy link
Member

jbremer commented Sep 22, 2015

Currently local IP addresses are filtered and not shown in the Network Analysis tab. Do make them show as targeted samples may contain hardcoded IP addresses to other boxes in the same private network.

@botherder botherder added the Task label Oct 5, 2015
@botherder botherder self-assigned this Oct 5, 2015
@botherder
Copy link
Member

We should however filter out those that are involved for Cuckoo operations. I feel that otherwise it's mostly gonna cause lots of confusion. @jekil thoughts?

@KillerInstinct
Copy link
Contributor

Cuckoo operations shouldn't be captured at all due to bpf filtering of host <-> guest in the sniffer auxiliary module. I think what Jurriaan is getting as is the dropping of packets if it's a local network which is done in the network processing module:
https://github.com/cuckoobox/cuckoo/blob/master/modules/processing/network.py#L179

Why not make it configurable? :)

@botherder
Copy link
Member

There's still a lot that bpf filter isn't filtering out. Regular Windows UDP traffic and shit related to the execution of the VM and configuration of the internal network. That kind of stuff I also considered as "Cuckoo operations".

@botherder botherder added this to the 2.0 milestone Dec 23, 2015
@KillerInstinct
Copy link
Contributor

While that may be true, there is no UDP traffic parsers that we particularly care about. I deleted that line for a custom inetsim rig and that generated some 'bloat' stuff. So really the only thing you have to do is ignore some of the tcp/udp dissection in the tcp/udp panes. But honestly, that is probably the least useful information overall when quickly looking at an analysis -- IMO.

@jbremer
Copy link
Member Author

jbremer commented Dec 24, 2015

@KillerInstinct just to be clear, what do you refer to with the least useful information overall here?

@KillerInstinct
Copy link
Contributor

The TCP/UDP tabs. They are generally unuseful for quickly looking at an analysis and really only provide supporting information to other networking data.

@jbremer
Copy link
Member Author

jbremer commented Dec 24, 2015

Agreed, now would be a good time to port those to http/https/etc protocols and remove the streams from the tcp/udp tabs for all streams that are represented in another tab.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants
@botherder @jbremer @KillerInstinct and others