diff options
author | Uli Heilmeier <uh@heilmeier.eu> | 2021-03-22 22:34:20 +0100 |
---|---|---|
committer | Uli Heilmeier <uh@heilmeier.eu> | 2021-03-24 08:28:20 +0000 |
commit | 2e6d3b571b50f6a0df31787cb95ba0f66c4fa85f (patch) | |
tree | 74e937d999d2f730c64400cfb684cce8ed1e26af /epan/dissectors/packet-ldap.c | |
parent | 337bdf8eb34be3fc5ccd1197868077e6276a2d6c (diff) |
LDAP: SASL Buffer doesn't include Length field
SASL Buffer starts after the SASL Buffer Length field. Therefore
we should only mark the bytes without the Length field.
Sample capture can be found in wireshark/wireshark#15128
Diffstat (limited to 'epan/dissectors/packet-ldap.c')
-rw-r--r-- | epan/dissectors/packet-ldap.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/epan/dissectors/packet-ldap.c b/epan/dissectors/packet-ldap.c index 1b59f77460..8bf1b513ff 100644 --- a/epan/dissectors/packet-ldap.c +++ b/epan/dissectors/packet-ldap.c @@ -4098,7 +4098,7 @@ static void proto_tree_add_uint(ldap_tree, hf_ldap_sasl_buffer_length, sasl_tvb, 0, 4, sasl_len); - sasl_tree = proto_tree_add_subtree(ldap_tree, sasl_tvb, 0, sasl_msg_len, ett_ldap_sasl_blob, NULL, "SASL Buffer"); + sasl_tree = proto_tree_add_subtree(ldap_tree, sasl_tvb, 4, sasl_msg_len - 4, ett_ldap_sasl_blob, NULL, "SASL Buffer"); if (ldap_info->auth_mech != NULL && ((strcmp(ldap_info->auth_mech, "GSS-SPNEGO") == 0) || |