From: Mike Stepanek (mstepane) Date: Wed, 12 Jun 2019 12:48:03 +0000 (-0400) Subject: Merge pull request #1643 in SNORT/snort3 from ~MIREDDEN/snort3:smtp_fix to master X-Git-Tag: 3.0.0-257~10 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=6ba5f79a3f73b83d5e76b581b25a961917fa9bc9;p=thirdparty%2Fsnort3.git Merge pull request #1643 in SNORT/snort3 from ~MIREDDEN/snort3:smtp_fix to master Squashed commit of the following: commit 6cbb3b865482a90da05f150f584a457e37209f39 Author: Mike Redden Date: Wed Jun 12 07:53:18 2019 -0400 smtp: Fix handle_header_line and normalize_data unit tests --- diff --git a/src/service_inspectors/smtp/smtp.cc b/src/service_inspectors/smtp/smtp.cc index 47fea6ee5..8af8455fc 100644 --- a/src/service_inspectors/smtp/smtp.cc +++ b/src/service_inspectors/smtp/smtp.cc @@ -1599,7 +1599,7 @@ TEST_CASE("handle_header_line", "[smtp]") uint8_t* eol = ptr + 38; Packet p; p.context = new IpsContext(1); - + SMTP_ResetAltBuffer(&p); int res = mime_ssn.handle_header_line(ptr, eol, 0, &p); REQUIRE((res == 0)); unsigned len = 0; @@ -1618,11 +1618,13 @@ TEST_CASE("normalize_data", "[smtp]") snort::DecodeConfig decode_conf; SmtpMime mime_ssn(&decode_conf, &log_config); smtp_normalizing = true; + SMTP_PROTO_CONF config; + mime_ssn.config = &config; uint8_t ptr[23] = "\r\n--wac7ysb48OaltWcw\r\n"; uint8_t* data_end = ptr + 22; Packet p; p.context = new IpsContext(1); - + SMTP_ResetAltBuffer(&p); int res = mime_ssn.normalize_data(ptr, data_end, &p); REQUIRE((res == 0)); unsigned len = 0;