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

GH-1589: Avoid extracting IP-like strings from SMTP headers

Browse source
This commit is contained in:
Tim Wojtulewicz 2021-08-10 11:53:10 -07:00 • committed by Tim Wojtulewicz
parent f8e87d3814
commit 569552b320
5 changed files with 43 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

2
scripts/base/protocols/smtp/main.zeek
View file

@ -109,7 +109,7 @@ event zeek_init() &priority=5
function find_address_in_smtp_header(header: string): string
{
local ips = extract_ip_addresses(header);
local ips = extract_ip_addresses(header, T);
# If there are more than one IP address found, return the second.
if ( |ips| > 1 )
return ips[1];

23
scripts/base/utils/addrs.zeek
View file

@ -83,8 +83,12 @@ function has_valid_octets(octets: string_vec): bool
##
## input: a string that may contain an IP address anywhere within it.
##
## check_wrapping: if true, will only return IP addresses that are wrapped in matching
## pairs of spaces, square brackets, curly braces, or parens. This can be used to avoid
## extracting strings that look like IPs from innocuous strings, such as SMTP headers.
##
## Returns: an array containing all valid IP address strings found in *input*.
function extract_ip_addresses(input: string): string_vec
function extract_ip_addresses(input: string, check_wrapping: bool &default=F): string_vec
{
local parts = split_string_all(input, ip_addr_regex);
local output: string_vec;
@ -92,8 +96,25 @@ function extract_ip_addresses(input: string): string_vec
for ( i in parts )
{
if ( i % 2 == 1 && is_valid_ip(parts[i]) )
{
if ( ! check_wrapping )
{
output += parts[i];
}
else if ( i > 0 && i < |parts| - 1 )
{
local p1 = parts[i-1];
local p3 = parts[i+1];
if ( ( |p1| == 0 && |p3| == 0 ) ||
( p1[-1] == "\[" && p3[0] == "\]" ) ||
( p1[-1] == "\(" && p3[0] == "\)" ) ||
( p1[-1] == "\{" && p3[0] == "\}" ) ||
( p1[-1] == " " && p3[0] == " " ) )
output += parts[i];
}
}
}
return output;
}

9
testing/btest/Baseline/scripts.base.utils.addrs/output
View file

@ -51,4 +51,13 @@ F
============ test extract_ip_addresses()
[1.1.1.1, 2.2.2.2, 3.3.3.3]
[1.1.1.1, 0:0:0:0:0:0:0:0, 3.3.3.3]
[1.1.1.1]
[1.1.1.1]
[]
[1.1.1.1]
[1.1.1.1]
[1.1.1.1]
[1.1.1.1, 2.2.2.2]
[1.1.1.1]
[1.1.1.1]
[6:1:2::3:4:5:6]

9
testing/btest/scripts/base/utils/addrs.test
View file

@ -135,6 +135,15 @@ event zeek_init()
print "============ test extract_ip_addresses()";
print extract_ip_addresses("this is 1.1.1.1 a test 2.2.2.2 string with ip addresses 3.3.3.3");
print extract_ip_addresses("this is 1.1.1.1 a test 0:0:0:0:0:0:0:0 string with ip addresses 3.3.3.3");
print extract_ip_addresses("text 1.1.1.1 text", T);
print extract_ip_addresses("text 1.1.1.1 text", F);
print extract_ip_addresses("text1.1.1.1text", T);
print extract_ip_addresses("text1.1.1.1text", F);
print extract_ip_addresses("text[1.1.1.1]text", T);
print extract_ip_addresses("text[1.1.1.1]text", F);
print extract_ip_addresses("[1.1.1.1] [2.2.2.2]", T);
print extract_ip_addresses("1.1.1.1", T);
print extract_ip_addresses("1.1.1.1", F);
# This will use the leading 6 from "IPv6" (maybe that's not intended
# by a person trying to parse such a string, but that's just what's going

2
testing/external/commit-hash.zeek-testing-private vendored
View file

@ -1 +1 @@
b33b7c939df51317595039ed408838712b738bbb
4b88837c49ade5d9fd980d5e6cf02ec91d19a3bb