Skip to content

Commit

Permalink
ssl : SSLProbingParser overflow fix
Browse files Browse the repository at this point in the history
Found by fuzzing
Fixes ssl detection evasion by packet splitting
  • Loading branch information
catenacyber authored and victorjulien committed Apr 30, 2019
1 parent 666bb1b commit 316a411
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/app-layer-ssl.c
Original file line number Diff line number Diff line change
Expand Up @@ -2628,7 +2628,7 @@ static AppProto SSLProbingParser(Flow *f, uint8_t direction,
uint8_t *input, uint32_t ilen, uint8_t *rdir)
{
/* probably a rst/fin sending an eof */
if (ilen == 0)
if (ilen < 3)
return ALPROTO_UNKNOWN;

/* for now just the 3 byte header ones */
Expand Down

0 comments on commit 316a411

Please sign in to comment.