Mirror/zeek
1
0
Fork 0
mirror of https://github.com/zeek/zeek.git synced 2025-10-02 14:48:21 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

Tweak find-filtered-trace to not flag traces if they have non-TCP

Browse source 
traffic.

Closes #160.
This commit is contained in:
Robin Sommer 2020-09-25 11:27:54 +00:00
parent ee5212c4a9
commit 1f6fc4415f
5 changed files with 10 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

7
scripts/base/misc/find-filtered-trace.zeek
View file

@ -32,6 +32,7 @@ function should_detect(): bool
global saw_tcp_conn_with_data: bool = F; global saw_tcp_conn_with_data: bool = F;
global saw_a_tcp_conn: bool = F; global saw_a_tcp_conn: bool = F;
global saw_a_non_tcp_conn: bool = F;
event connection_state_remove(c: connection) event connection_state_remove(c: connection)
{ {
@ -42,7 +43,10 @@ event connection_state_remove(c: connection)
return; return;
if ( ! is_tcp_port(c$id$orig_p) ) if ( ! is_tcp_port(c$id$orig_p) )
{
saw_a_non_tcp_conn = T;
return; return;
}
saw_a_tcp_conn = T; saw_a_tcp_conn = T;
@ -58,6 +62,9 @@ event zeek_done()
if ( ! saw_a_tcp_conn ) if ( ! saw_a_tcp_conn )
return; return;
if ( saw_a_non_tcp_conn )
return;
if ( ! saw_tcp_conn_with_data ) if ( ! saw_tcp_conn_with_data )
Reporter::warning("The analyzed trace file was determined to contain only TCP control packets, which may indicate it's been pre-filtered. By default, Zeek reports the missing segments for this type of trace, but the 'detect_filtered_trace' option may be toggled if that's not desired."); Reporter::warning("The analyzed trace file was determined to contain only TCP control packets, which may indicate it's been pre-filtered. By default, Zeek reports the missing segments for this type of trace, but the 'detect_filtered_trace' option may be toggled if that's not desired.");
} }

2
testing/btest/Baseline/scripts.base.misc.find-filtered-trace/out1
View file

@ -1 +1 @@
1389719059.311687 warning in /home/jon/pro/zeek/zeek/scripts/base/misc/find-filtered-trace.zeek, line 62: The analyzed trace file was determined to contain only TCP control packets, which may indicate it's been pre-filtered. By default, Zeek reports the missing segments for this type of trace, but the 'detect_filtered_trace' option may be toggled if that's not desired. 1389719059.311687 warning in /Users/robin/bro/topic/scripts/base/misc/find-filtered-trace.zeek, line 69: The analyzed trace file was determined to contain only TCP control packets, which may indicate it's been pre-filtered. By default, Zeek reports the missing segments for this type of trace, but the 'detect_filtered_trace' option may be toggled if that's not desired.

0
testing/btest/Baseline/scripts.base.misc.find-filtered-trace/out3 Normal file
View file

BIN
testing/btest/Traces/wikipedia-filtered-plus-udp.trace Normal file
View file

Binary file not shown.

2
testing/btest/scripts/base/misc/find-filtered-trace.test
View file

@ -1,6 +1,8 @@
# @TEST-EXEC: zeek -b -r $TRACES/http/bro.org-filtered.pcap %INPUT >out1 2>&1 # @TEST-EXEC: zeek -b -r $TRACES/http/bro.org-filtered.pcap %INPUT >out1 2>&1
# @TEST-EXEC: zeek -b -r $TRACES/http/bro.org-filtered.pcap %INPUT "FilteredTraceDetection::enable=F" >out2 2>&1 # @TEST-EXEC: zeek -b -r $TRACES/http/bro.org-filtered.pcap %INPUT "FilteredTraceDetection::enable=F" >out2 2>&1
# @TEST-EXEC: zeek -b -r $TRACES/wikipedia-filtered-plus-udp.trace %INPUT >out3 2>&1
# @TEST-EXEC: TEST_DIFF_CANONIFIER=$SCRIPTS/diff-remove-abspath btest-diff out1 # @TEST-EXEC: TEST_DIFF_CANONIFIER=$SCRIPTS/diff-remove-abspath btest-diff out1
# @TEST-EXEC: btest-diff out2 # @TEST-EXEC: btest-diff out2
# @TEST-EXEC: btest-diff out3
@load base/misc/find-filtered-trace @load base/misc/find-filtered-trace