diff options
| author | Alexis La Goutte <alexis.lagoutte@gmail.com> | 2016-06-04 18:06:42 +0200 |
|---|---|---|
| committer | Michael Mann <mmann78@netscape.net> | 2016-06-05 23:44:10 +0000 |
| commit | 372d4de58a164ea50109c02722c7844857edec76 (patch) | |
| tree | 5c98cb3831c5359bc4fad2fe01a4e71da3d3394c /epan/dissectors/packet-quic.c | |
| parent | 3a2dd2fe39d2c17a2b4dc6c160d989c79350a7c4 (diff) | |
QUIC: Diversification nonce is only present from server to client
For avoid problem with some middle boxes (don't yet supported >= Q033)
See https://groups.google.com/a/chromium.org/d/msg/proto-quic/kQVDZal_iwo/7dNgo4u6FgAJ for more information
Change-Id: I918e7f4400549fd6e78f3ce727e8cae0c10b464b
Reviewed-on: https://code.wireshark.org/review/15720
Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Michael Mann <mmann78@netscape.net>
Diffstat (limited to 'epan/dissectors/packet-quic.c')
| -rw-r--r-- | epan/dissectors/packet-quic.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/epan/dissectors/packet-quic.c b/epan/dissectors/packet-quic.c index 1c86a670bf..b451f9b252 100644 --- a/epan/dissectors/packet-quic.c +++ b/epan/dissectors/packet-quic.c @@ -1841,11 +1841,12 @@ dissect_quic_common(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, /* Diversification Nonce */ if(puflags & PUFLAGS_DNONCE && quic_info->version >= 33){ - proto_tree_add_item(quic_tree, hf_quic_diversification_nonce, tvb, offset, 32, ENC_NA); - offset += 32; + if(pinfo->srcport == 443){ /* Diversification nonce is only present from server to client */ + proto_tree_add_item(quic_tree, hf_quic_diversification_nonce, tvb, offset, 32, ENC_NA); + offset += 32; + } } - /* Packet Number */ /* Get len of packet number (and packet number), may be a more easy function to get the length... */ |