From: Pranav Bhalerao (prbhaler) <prbhaler@cisco.com>
Date: Tue, 8 Mar 2022 06:05:30 +0000 (+0000)
Subject: Pull request #3301: ssh: NULL check for session pointer before access.
X-Git-Tag: 3.1.25.0~4
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=b5e95e3dc3d699b57c4c33a816501ddb942ac328;p=thirdparty%2Fsnort3.git

Pull request #3301: ssh: NULL check for session pointer before access.

Merge in SNORT/snort3 from ~PRBHALER/snort3:ssh_crash to master

Squashed commit of the following:

commit d1425cd466acbecc7e25dcd7bce141f5ca0c015d
Author: Pranav Bhalerao <prbhaler@cisco.com>
Date:   Mon Mar 7 17:42:43 2022 +0530

    ssh: NULL check for session pointer before access.
---

diff --git a/src/service_inspectors/ssh/ssh.cc b/src/service_inspectors/ssh/ssh.cc
index 0dfc8075c..69ab71671 100644
--- a/src/service_inspectors/ssh/ssh.cc
+++ b/src/service_inspectors/ssh/ssh.cc
@@ -90,6 +90,8 @@ static void snort_ssh(SSH_PROTO_CONF* config, Packet* p)
 
     // Attempt to get a previously allocated SSH block.
     SSHData* sessp = get_session_data(p->flow);
+    if (!sessp)
+        return;
 
     // Don't process if we've missed packets
     if (sessp->state_flags & SSH_FLG_MISSED_PACKETS)