Fix a couple of Coverity findings (1433618, 1433619)

This commit is contained in:
Tim Wojtulewicz 2020-10-19 16:29:47 -07:00
parent 66eaf9c7cc
commit ce2b00fe83
2 changed files with 14 additions and 9 deletions

View file

@ -681,19 +681,23 @@ bool NetSessions::WantConnection(uint16_t src_port, uint16_t dst_port,
void NetSessions::Weird(const char* name, const Packet* pkt, const char* addl)
{
const char* weird_name = name;
if ( pkt )
{
pkt->dump_packet = true;
const char* weird_name;
if ( pkt->encap && pkt->encap->LastType() != BifEnum::Tunnel::NONE )
weird_name = util::fmt("%s_in_tunnel", name);
else
weird_name = name;
if ( pkt->encap && pkt->encap->LastType() != BifEnum::Tunnel::NONE )
weird_name = util::fmt("%s_in_tunnel", name);
if ( pkt->ip_hdr )
reporter->Weird(pkt->ip_hdr->SrcAddr(), pkt->ip_hdr->DstAddr(), weird_name, addl);
else
reporter->Weird(weird_name, addl);
if ( pkt->ip_hdr )
{
reporter->Weird(pkt->ip_hdr->SrcAddr(), pkt->ip_hdr->DstAddr(), weird_name, addl);
return;
}
}
reporter->Weird(weird_name, addl);
}
void NetSessions::Weird(const char* name, const IP_Hdr* ip, const char* addl)

View file

@ -26,6 +26,7 @@ IPAnalyzer::IPAnalyzer()
IPAnalyzer::~IPAnalyzer()
{
delete discarder;
}
bool IPAnalyzer::AnalyzePacket(size_t len, const uint8_t* data, Packet* packet)