diff options
| author | guy <guy@f5534014-38df-0310-8fa8-9805f1628bb7> | 2002-11-25 19:20:44 +0000 |
|---|---|---|
| committer | guy <guy@f5534014-38df-0310-8fa8-9805f1628bb7> | 2002-11-25 19:20:44 +0000 |
| commit | f39e67d9379d32a63815135c268d97b063bc5a7c (patch) | |
| tree | 632d183b4c11a6b33afcdd4f4efb44f8c0285ee6 /packet-ldap.c | |
| parent | ddd77778dcdf77a7acd3851442e193513e65a51f (diff) | |
From Jason Greene:
This patch fixes decoding of the newSuperior attribute of an
LDAPv3 modrdn request. The current implementation attempts to
decode the attribute as an LDAPDN (Octext String, 0x4), when its
definition is actually Context 0 (0x80).
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@6672 f5534014-38df-0310-8fa8-9805f1628bb7
Diffstat (limited to 'packet-ldap.c')
| -rw-r--r-- | packet-ldap.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/packet-ldap.c b/packet-ldap.c index 7681ebfaad..7fa04fd934 100644 --- a/packet-ldap.c +++ b/packet-ldap.c @@ -1,7 +1,7 @@ /* packet-ldap.c * Routines for ldap packet dissection * - * $Id: packet-ldap.c,v 1.48 2002/11/12 21:37:22 guy Exp $ + * $Id: packet-ldap.c,v 1.49 2002/11/25 19:20:42 guy Exp $ * * Ethereal - Network traffic analyzer * By Gerald Combs <gerald@ethereal.com> @@ -1231,7 +1231,8 @@ static void dissect_ldap_request_modifyrdn(ASN1_SCK *a, proto_tree *tree, if (a->offset < (int) (start + length)) { /* LDAP V3 Modify DN operation, with newSuperior */ - if (read_string(a, tree, hf_ldap_message_modrdn_superior, 0, 0, ASN1_UNI, ASN1_OTS) != ASN1_ERR_NOERROR) + /* "newSuperior [0] LDAPDN OPTIONAL" (0x80) */ + if (read_string(a, tree, hf_ldap_message_modrdn_superior, 0, 0, ASN1_CTX, 0) != ASN1_ERR_NOERROR) return; } } |