From: Shanmugam S (shanms) <shanms@cisco.com>
Date: Fri, 14 Oct 2022 12:29:03 +0000 (+0000)
Subject: Pull request #3614: appid: return APP_ID_NONE only if hsession is not present for... 
X-Git-Tag: 3.1.45.0~8
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=5fb5c68bb8734293f234697438e5e422201be285;p=thirdparty%2Fsnort3.git

Pull request #3614: appid: return APP_ID_NONE only if hsession is not present for http3

Merge in SNORT/snort3 from ~SHIKV/snort3:appid_ss to master

Squashed commit of the following:

commit c366852482ce8e0580a64055896220e07c57fe99
Author: shibin k v <shikv@cisco.com>
Date:   Tue Oct 4 10:44:05 2022 +0000

    appid: return APP_ID_NONE only if hsession is not present for http3
---

diff --git a/src/network_inspectors/appid/appid_session.cc b/src/network_inspectors/appid/appid_session.cc
index 08f25e8c0..fa84fc27f 100644
--- a/src/network_inspectors/appid/appid_session.cc
+++ b/src/network_inspectors/appid/appid_session.cc
@@ -821,7 +821,8 @@ AppId AppIdSession::pick_service_app_id() const
 
 AppId AppIdSession::pick_ss_misc_app_id() const
 {
-    if (api.service.get_id() == APP_ID_HTTP2 or api.service.get_id() == APP_ID_HTTP3)
+    if (api.service.get_id() == APP_ID_HTTP2 or
+        (api.service.get_id() == APP_ID_HTTP3 and !api.hsessions.empty()))
         return APP_ID_NONE;
 
     if (misc_app_id > APP_ID_NONE)
@@ -838,7 +839,8 @@ AppId AppIdSession::pick_ss_misc_app_id() const
 
 AppId AppIdSession::pick_ss_client_app_id() const
 {
-    if (api.service.get_id() == APP_ID_HTTP2 or api.service.get_id() == APP_ID_HTTP3)
+    if (api.service.get_id() == APP_ID_HTTP2 or
+        (api.service.get_id() == APP_ID_HTTP3 and !api.hsessions.empty()))
         return APP_ID_NONE;
 
     AppId tmp_id = APP_ID_NONE;
@@ -868,7 +870,8 @@ AppId AppIdSession::pick_ss_client_app_id() const
 
 AppId AppIdSession::pick_ss_payload_app_id(AppId service_id) const
 {
-    if (service_id == APP_ID_HTTP2 or service_id == APP_ID_HTTP3)
+    if (service_id == APP_ID_HTTP2 or
+        (service_id == APP_ID_HTTP3 and !api.hsessions.empty()))
         return APP_ID_NONE;
 
     if (tp_payload_app_id_deferred)
diff --git a/src/network_inspectors/appid/ips_appid_option.cc b/src/network_inspectors/appid/ips_appid_option.cc
index 975f915ed..99c38612a 100644
--- a/src/network_inspectors/appid/ips_appid_option.cc
+++ b/src/network_inspectors/appid/ips_appid_option.cc
@@ -124,7 +124,8 @@ IpsOption::EvalStatus AppIdIpsOption::eval(Cursor&, Packet* p)
     AppId service_id = session->get_api().get_service_app_id();
     OdpContext& odp_ctxt = session->get_odp_ctxt();
 
-    if (service_id != APP_ID_HTTP2 and service_id != APP_ID_HTTP3)
+    if ((service_id != APP_ID_HTTP2 and service_id != APP_ID_HTTP3) or
+        (service_id == APP_ID_HTTP3 and session->get_api().get_hsessions_size() == 0))
     {
         AppId app_ids[APP_PROTOID_MAX];