diff options
author | Peter Wu <peter@lekensteyn.nl> | 2016-09-04 02:06:50 +0200 |
---|---|---|
committer | Peter Wu <peter@lekensteyn.nl> | 2016-09-06 11:53:31 +0000 |
commit | 10e84a612b629341acd9cd20876517e5bab63d37 (patch) | |
tree | 5a416f6273b2bd68ae79cab698f54b4857d6e9b1 /epan/dissectors/packet-ssl-utils.h | |
parent | c8de455f4bcab0c560ec74bc0c1d3c46dad07270 (diff) | |
download | wireshark-10e84a612b629341acd9cd20876517e5bab63d37.tar.gz |
ssl: really fix session resumption expert info
In a two-pass dissection with renegotiated sessions, the
is_session_resumed flag is not updated according to the current protocol
flow. Fix this by performing detection of abbreviated handshakes in
all cases, do not limit it to the decryption stage (where ssl != NULL).
Reset the resumption assumption after the first ChangeCipherSpec
(normally from the server side, but explicitly add this in case client
packets somehow arrive earlier in the capture). This should not have a
functional effect on normal TLS captures with Session Tickets.
Bug: 12793
Change-Id: I1eb2a8262b4e359b8c1d3d0a1e004a9e856bec8c
Reviewed-on: https://code.wireshark.org/review/17483
Petri-Dish: Peter Wu <peter@lekensteyn.nl>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Peter Wu <peter@lekensteyn.nl>
Diffstat (limited to 'epan/dissectors/packet-ssl-utils.h')
-rw-r--r-- | epan/dissectors/packet-ssl-utils.h | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/epan/dissectors/packet-ssl-utils.h b/epan/dissectors/packet-ssl-utils.h index 7dcc83203a..25961ec12d 100644 --- a/epan/dissectors/packet-ssl-utils.h +++ b/epan/dissectors/packet-ssl-utils.h @@ -229,7 +229,6 @@ typedef struct _StringInfo { #define SSL_PRE_MASTER_SECRET (1<<6) #define SSL_CLIENT_EXTENDED_MASTER_SECRET (1<<7) #define SSL_SERVER_EXTENDED_MASTER_SECRET (1<<8) -#define SSL_SERVER_HELLO_DONE (1<<9) #define SSL_NEW_SESSION_TICKET (1<<10) #define SSL_EXTENDED_MASTER_SECRET_MASK (SSL_CLIENT_EXTENDED_MASTER_SECRET|SSL_SERVER_EXTENDED_MASTER_SECRET) |