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

packet_analysis: Protect from nullptr sessions

Browse source 
Passing nullptr sessions to AnalyzerConfirmation and AnalyzerViolation
of protocol analyzers previously blew up - protect from that.

Related to zeek/spicy-plugin#133.
This commit is contained in:
Arne Welzel 2022-09-27 14:02:52 +02:00
parent fdedc6b200
commit fbf379b757
1 changed files with 6 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

6
src/packet_analysis/Analyzer.cc
View file

@ -186,6 +186,9 @@ void Analyzer::AnalyzerConfirmation(session::Session* session, zeek::Tag arg_tag
{
const auto& effective_tag = arg_tag ? arg_tag : GetAnalyzerTag();
if ( ! session )
return;
if ( session->AnalyzerState(effective_tag) == session::AnalyzerConfirmationState::CONFIRMED )
return;
@ -242,6 +245,9 @@ void Analyzer::AnalyzerViolation(const char* reason, session::Session* session,
{
const auto& effective_tag = arg_tag ? arg_tag : GetAnalyzerTag();
if ( ! session )
return;
if ( session->AnalyzerState(effective_tag) == session::AnalyzerConfirmationState::VIOLATED )
return;