diff options
author | Ronnie Sahlberg <ronnie_sahlberg@ozemail.com.au> | 2004-12-19 04:10:05 +0000 |
---|---|---|
committer | Ronnie Sahlberg <ronnie_sahlberg@ozemail.com.au> | 2004-12-19 04:10:05 +0000 |
commit | 62f26aaeb4babd4d5e59792a9915e3f9803df630 (patch) | |
tree | 5326fcfe8af78325e167d54f8d62b74806f36b08 /epan/dissectors/packet-ldap.c | |
parent | 1e51ddbae309d93dc63b1c7b4d567b8d5add966c (diff) |
abort trying to dissect ldap controls if the header doesnt look right.
it will not solve the problem in the c06- testmenageri capture that
contains unknown types of ldap commands but it will at least
stop the ldap controls dissector from dumping core.
someone interested in ldap might want to look at those "unknown ldap packets"
in the trace.
svn path=/trunk/; revision=12773
Diffstat (limited to 'epan/dissectors/packet-ldap.c')
-rw-r--r-- | epan/dissectors/packet-ldap.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/epan/dissectors/packet-ldap.c b/epan/dissectors/packet-ldap.c index e6746b12ab..b50e1ae834 100644 --- a/epan/dissectors/packet-ldap.c +++ b/epan/dissectors/packet-ldap.c @@ -1952,6 +1952,7 @@ static void dissect_ldap_controls(ASN1_SCK *a, proto_tree *tree) proto_tree_add_text(tree, a->tvb, a->offset, 0, "ERROR: Couldn't parse LDAP Controls: %s", asn1_err_to_str(ASN1_ERR_WRONG_TYPE)); + return; } ctrls_item = proto_tree_add_text(tree, a->tvb, start, length, "LDAP Controls"); |