Mirror/zeek
1
0
Fork 0
mirror of https://github.com/zeek/zeek.git synced 2025-10-10 02:28:21 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 3

Merge remote-tracking branch 'origin/topic/jsiwek/ssl-empty-files'

Browse source 
* origin/topic/jsiwek/ssl-empty-files:
  Skip file analysis for zero-length SSL/TLS data
This commit is contained in:
Johanna Amann 2020-01-29 11:09:45 -08:00
commit ad18014bed
4 changed files with 16 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

6
src/analyzer/protocol/ssl/proc-certificate.pac
View file

@ -16,6 +16,12 @@
{
const bytestring& cert = (*certificates)[i];
if ( cert.length() <= 0 )
{
reporter->Weird(bro_analyzer()->Conn(), "zero_length_certificate");
continue;
}
ODesc file_handle;
file_handle.Add(common.Description());
file_handle.Add(i);

6
src/analyzer/protocol/ssl/tls-handshake-analyzer.pac
View file

@ -303,7 +303,7 @@ refine connection Handshake_Conn += {
common.AddRaw("F");
bro_analyzer()->Conn()->IDString(&common);
if ( status_type == 1 ) // ocsp
if ( status_type == 1 && response.length() > 0 ) // ocsp
{
ODesc file_handle;
file_handle.Add(common.Description());
@ -323,6 +323,10 @@ refine connection Handshake_Conn += {
file_mgr->EndOfFile(file_id);
}
else if ( response.length() == 0 )
{
reporter->Weird(bro_analyzer()->Conn(), "SSL_zero_length_stapled_OCSP_message");
}
return true;
%}