Merge pull request #1236 in SNORT/snort3 from appid_null_referer to master

Squashed commit of the following:

commit 6b32a70e6296eef35610e0789ba026e06d362f6c
Author: Shravan Rangaraju <shrarang@cisco.com>
Date:   Tue May 22 19:31:32 2018 -0400

    appid: create referer object only for non-null string

diff --git a/src/network_inspectors/appid/appid_http_session.cc b/src/network_inspectors/appid/appid_http_session.cc
index f2d889edaecf06b81b060eb0fd6d1609a5981c11..b93504b2fd9836edb37272d67da79873356ceee2 100644 (file)
--- a/src/network_inspectors/appid/appid_http_session.cc
+++ b/src/network_inspectors/appid/appid_http_session.cc
@@ -972,7 +972,8 @@ void AppIdHttpSession::update_referer(const uint8_t* new_referer, int32_t len)
 {
     if ( referer )
         delete referer;
-    referer = new std::string((const char*)new_referer, len);
+    if ( new_referer and len ) referer = new std::string((const char*)new_referer, len);
+    else referer = nullptr;
 }
 
 void AppIdHttpSession::update_x_working_with(const uint8_t* new_xww, int32_t len)