diff options
author | Richard Sharpe <realrichardsharpe@gmail.com> | 2017-11-08 02:26:19 -0800 |
---|---|---|
committer | Richard Sharpe <realrichardsharpe@gmail.com> | 2017-11-08 14:55:45 +0000 |
commit | 2125275c7cc261e101731162ac1eeba7626ce9e8 (patch) | |
tree | a3c064e3958b1486422401bb109f5adec36ff842 /epan | |
parent | 3f189bed2f343893a4b9df626875a930e03c91e1 (diff) |
smb2: Fix bug 6027 to correct SRT computations.
Fix a problem where SMB2 cancel and break messages
are included in SRT computations when it should not be.
Bug: 6027
Change-Id: I690fe6d9e14f26d85fe5c71830869780046d5d12
Reviewed-on: https://code.wireshark.org/review/24295
Reviewed-by: Richard Sharpe <realrichardsharpe@gmail.com>
Diffstat (limited to 'epan')
-rw-r--r-- | epan/dissectors/packet-smb2.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/epan/dissectors/packet-smb2.c b/epan/dissectors/packet-smb2.c index 6dd0eb2821..3e853214ec 100644 --- a/epan/dissectors/packet-smb2.c +++ b/epan/dissectors/packet-smb2.c @@ -855,6 +855,11 @@ smb2stat_packet(void *pss, packet_info *pinfo, epan_dissect_t *edt _U_, const vo if(!(si->flags&SMB2_FLAGS_RESPONSE)){ return 0; } + /* We should not include cancel and oplock break requests either */ + if (si->opcode == SMB2_COM_CANCEL || si->opcode == SMB2_COM_BREAK) { + return 0; + } + /* if we haven't seen the request, just ignore it */ if(!si->saved){ return 0; |