diff options
author | Guy Harris <guy@alum.mit.edu> | 2002-11-25 19:20:44 +0000 |
---|---|---|
committer | Guy Harris <guy@alum.mit.edu> | 2002-11-25 19:20:44 +0000 |
commit | a1c648718a6c59f9aac62b37330d1d545291801d (patch) | |
tree | 632d183b4c11a6b33afcdd4f4efb44f8c0285ee6 /packet-ldap.c | |
parent | 751eaa43d9a32f09a30ac56cc69e5d2746b9f6c1 (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).
svn path=/trunk/; revision=6672
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; } } |