Use the "rx_frm_len" field from the trace packet data record header,
[obnox/wireshark/wip.git] / packet-ldap.c
index afe4bf0c4f6390f7a53a376bcf3bd48c686f7897..78371f32809c4d84a1d78a5a8eb56b112a3ddf0e 100644 (file)
@@ -1,22 +1,24 @@
 /* packet-ldap.c
  * Routines for ldap packet dissection
  *
- * $Id: packet-ldap.c,v 1.20 2001/01/03 16:41:06 gram Exp $
+ * See RFC 1777 (LDAP v2), RFC 2251 (LDAP v3), and RFC 2222 (SASL).
+ *
+ * $Id: packet-ldap.c,v 1.56 2003/06/09 07:45:36 guy Exp $
  *
  * Ethereal - Network traffic analyzer
- * By Gerald Combs <gerald@zing.org>
+ * By Gerald Combs <gerald@ethereal.com>
  * Copyright 1998 Gerald Combs
  *
  * This program is free software; you can redistribute it and/or
  * modify it under the terms of the GNU General Public License
  * as published by the Free Software Foundation; either version 2
  * of the License, or (at your option) any later version.
- * 
+ *
  * This program is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  * GNU General Public License for more details.
- * 
+ *
  * You should have received a copy of the GNU General Public License
  * along with this program; if not, write to the Free Software
  * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA  02111-1307, USA.
  * This is not a complete implementation. It doesn't handle the full version 3, more specifically,
  * it handles only the commands of version 2, but any additional characteristics of the ver3 command are supported.
  * It's also missing extensible search filters.
- * 
+ *
  * There should probably be alot more error checking, I simply assume that if we have a full packet, it will be a complete
  * and correct packet.
- * 
+ *
  * AFAIK, it will handle all messages used by the OpenLDAP 1.2.9 server and libraries which was my goal. I do plan to add
  * the remaining commands as time permits but this is not a priority to me. Send me an email if you need it and I'll see what
  * I can do.
- * 
+ *
  * Doug Nazar
  * nazard@dragoninc.on.ca
  */
 
+/*
+ * 11/11/2002 - Fixed problem when decoding LDAP with desegmentation enabled and the
+ *              ASN.1 BER Universal Class Tag: "Sequence Of" header is encapsulated across 2
+ *              TCP segments.
+ *
+ * Ronald W. Henderson
+ * ronald.henderson@cognicaseusa.com
+ */
 #ifdef HAVE_CONFIG_H
 # include "config.h"
 #endif
 
 #include <stdio.h>
 
-#ifdef HAVE_SYS_TYPES_H
-# include <sys/types.h>
-#endif
-
-#ifdef HAVE_NETINET_IN_H
-# include <netinet/in.h>
-#endif
-
 #include <string.h>
 #include <glib.h>
 
 # include "snprintf.h"
 #endif
 
-#include "packet.h"
+#include <epan/packet.h>
 
 #include "packet-ldap.h"
 #include "asn1.h"
+#include "prefs.h"
+#include <epan/conversation.h>
+#include "packet-frame.h"
 
 static int proto_ldap = -1;
+static int hf_ldap_sasl_buffer_length = -1;
 static int hf_ldap_length = -1;
 static int hf_ldap_message_id = -1;
 static int hf_ldap_message_type = -1;
@@ -79,6 +85,9 @@ static int hf_ldap_message_bind_version = -1;
 static int hf_ldap_message_bind_dn = -1;
 static int hf_ldap_message_bind_auth = -1;
 static int hf_ldap_message_bind_auth_password = -1;
+static int hf_ldap_message_bind_auth_mechanism = -1;
+static int hf_ldap_message_bind_auth_credentials = -1;
+static int hf_ldap_message_bind_server_credentials = -1;
 
 static int hf_ldap_message_search_base = -1;
 static int hf_ldap_message_search_scope = -1;
@@ -105,11 +114,37 @@ static int hf_ldap_message_modify_delete = -1;
 static int hf_ldap_message_abandon_msgid = -1;
 
 static gint ett_ldap = -1;
-static gint ett_ldap_message = -1;
+static gint ett_ldap_gssapi_token = -1;
 static gint ett_ldap_referrals = -1;
 static gint ett_ldap_attribute = -1;
 
+/* desegmentation of LDAP */
+static gboolean ldap_desegment = TRUE;
+
 #define TCP_PORT_LDAP                  389
+#define UDP_PORT_CLDAP                 389
+
+static dissector_handle_t gssapi_handle;
+static dissector_handle_t gssapi_wrap_handle;
+
+/*
+ * Data structure attached to a conversation, giving authentication
+ * information from a bind request.
+ * We keep a linked list of them, so that we can free up all the
+ * authentication mechanism strings.
+ */
+typedef struct ldap_auth_info_t {
+  guint auth_type;             /* authentication type */
+  char *auth_mech;             /* authentication mechanism */
+  guint32 first_auth_frame;    /* first frame that would use a security layer */
+  struct ldap_auth_info_t *next;
+} ldap_auth_info_t;
+
+static GMemChunk *ldap_auth_info_chunk = NULL;
+
+static guint ldap_auth_info_chunk_count = 200;
+
+static ldap_auth_info_t *auth_info_items;
 
 static value_string msgTypes [] = {
   {LDAP_REQ_BIND, "Bind Request"},
@@ -122,7 +157,7 @@ static value_string msgTypes [] = {
   {LDAP_REQ_COMPARE, "Compare Request"},
   {LDAP_REQ_ABANDON, "Abandon Request"},
   {LDAP_REQ_EXTENDED, "Extended Request"},
-    
+
   {LDAP_RES_BIND, "Bind Result"},
   {LDAP_RES_SEARCH_ENTRY, "Search Entry"},
   {LDAP_RES_SEARCH_RESULT, "Search Result"},
@@ -132,7 +167,7 @@ static value_string msgTypes [] = {
   {LDAP_RES_DELETE, "Delete Result"},
   {LDAP_RES_MODRDN, "Modify RDN Result"},
   {LDAP_RES_COMPARE, "Compare Result"},
-  {LDAP_REQ_EXTENDED, "Extended Response"},
+  {LDAP_RES_EXTENDED, "Extended Response"},
   {0, NULL},
 };
 
@@ -140,17 +175,26 @@ static int read_length(ASN1_SCK *a, proto_tree *tree, int hf_id, guint *len)
 {
   guint length = 0;
   gboolean def = FALSE;
-  const guchar *start = a->pointer;
-  
-  asn1_length_decode(a, &def, &length);
+  int start = a->offset;
+  int ret;
+
+  ret = asn1_length_decode(a, &def, &length);
+  if (ret != ASN1_ERR_NOERROR) {
+    if (tree) {
+      proto_tree_add_text(tree, a->tvb, start, 0,
+        "%s: ERROR: Couldn't parse length: %s",
+        proto_registrar_get_name(hf_id), asn1_err_to_str(ret));
+    }
+    return ret;
+  }
 
   if (len)
     *len = length;
 
   if (tree)
-    proto_tree_add_uint(tree, hf_id, NullTVB, start-a->begin, a->pointer-start, length);
+    proto_tree_add_uint(tree, hf_id, a->tvb, start, a->offset-start, length);
 
-  return 0;
+  return ASN1_ERR_NOERROR;
 }
 
 static int read_sequence(ASN1_SCK *a, guint *len)
@@ -158,16 +202,18 @@ static int read_sequence(ASN1_SCK *a, guint *len)
   guint cls, con, tag;
   gboolean def;
   guint length;
-  
-  if (asn1_header_decode(a, &cls, &con, &tag, &def, &length) != ASN1_ERR_NOERROR)
-    return 1;
+  int ret;
+
+  ret = asn1_header_decode(a, &cls, &con, &tag, &def, &length);
+  if (ret != ASN1_ERR_NOERROR)
+    return ret;
   if (cls != ASN1_UNI || con != ASN1_CON || tag != ASN1_SEQ)
-    return 1;
-  
+    return ASN1_ERR_WRONG_TYPE;
+
   if (len)
     *len = length;
-  
-  return 0;
+
+  return ASN1_ERR_NOERROR;
 }
 
 static int read_set(ASN1_SCK *a, guint *len)
@@ -175,139 +221,256 @@ static int read_set(ASN1_SCK *a, guint *len)
   guint cls, con, tag;
   gboolean def;
   guint length;
-  
-  if (asn1_header_decode(a, &cls, &con, &tag, &def, &length) != ASN1_ERR_NOERROR)
-    return 1;
+  int ret;
+
+  ret = asn1_header_decode(a, &cls, &con, &tag, &def, &length);
+  if (ret != ASN1_ERR_NOERROR)
+    return ret;
   if (cls != ASN1_UNI || con != ASN1_CON || tag != ASN1_SET)
-    return 1;
-  
+    return ASN1_ERR_WRONG_TYPE;
+
   if (len)
     *len = length;
-  
-  return 0;
+
+  return ASN1_ERR_NOERROR;
 }
 
 static int read_integer_value(ASN1_SCK *a, proto_tree *tree, int hf_id,
-       proto_tree **new_tree, guint *i, const guchar *start, guint length)
+       proto_item **new_item, guint *i, int start, guint length)
 {
   guint integer = 0;
+  proto_item *temp_item = NULL;
+  int ret;
 
-  asn1_uint32_value_decode(a, length, &integer);
+  ret = asn1_uint32_value_decode(a, length, &integer);
+  if (ret != ASN1_ERR_NOERROR) {
+    if (tree) {
+      proto_tree_add_text(tree, a->tvb, start, 0,
+       "%s: ERROR: Couldn't parse value: %s",
+        proto_registrar_get_name(hf_id), asn1_err_to_str(ret));
+    }
+    return ret;
+  }
 
   if (i)
     *i = integer;
 
   if (tree)
-  {
-    proto_tree *temp_tree = 0;
-    temp_tree = proto_tree_add_uint(tree, hf_id, NullTVB, start-a->begin, a->pointer-start, integer);
-    if (new_tree)
-      *new_tree = temp_tree;
-  }
+    temp_item = proto_tree_add_uint(tree, hf_id, a->tvb, start, a->offset-start, integer);
+
+  if (new_item)
+    *new_item = temp_item;
 
-  return 0;
+  return ASN1_ERR_NOERROR;
 }
 
 static int read_integer(ASN1_SCK *a, proto_tree *tree, int hf_id,
-       proto_tree **new_tree, guint *i, guint expected_tag)
+       proto_item **new_item, guint *i, guint expected_tag)
 {
   guint cls, con, tag;
   gboolean def;
   guint length;
-  const guchar *start = a->pointer;
-  
-  if (asn1_header_decode(a, &cls, &con, &tag, &def, &length) != ASN1_ERR_NOERROR)
-    return 1;
-  if (cls != ASN1_UNI || con != ASN1_PRI || tag != expected_tag)
-    return 1;
-
-  return read_integer_value(a, tree, hf_id, new_tree, i, start, length);
+  int start = a->offset;
+  int ret;
+
+  ret = asn1_header_decode(a, &cls, &con, &tag, &def, &length);
+  if (ret == ASN1_ERR_NOERROR) {
+    if (cls != ASN1_UNI || con != ASN1_PRI || tag != expected_tag)
+      ret = ASN1_ERR_WRONG_TYPE;
+  }
+  if (ret != ASN1_ERR_NOERROR) {
+    if (tree) {
+      proto_tree_add_text(tree, a->tvb, start, 0,
+        "%s: ERROR: Couldn't parse header: %s",
+        (hf_id != -1) ? proto_registrar_get_name(hf_id) : "LDAP message",
+        asn1_err_to_str(ret));
+    }
+    return ret;
+  }
+
+  return read_integer_value(a, tree, hf_id, new_item, i, start, length);
 }
 
 static int read_boolean_value(ASN1_SCK *a, proto_tree *tree, int hf_id,
-       proto_tree **new_tree, guint *i, const guchar *start, guint length)
+       proto_item **new_item, guint *i, int start, guint length)
 {
   guint integer = 0;
+  proto_item *temp_item = NULL;
+  int ret;
 
-  asn1_uint32_value_decode(a, length, &integer);
+  ret = asn1_uint32_value_decode(a, length, &integer);
+  if (ret != ASN1_ERR_NOERROR) {
+    if (tree) {
+      proto_tree_add_text(tree, a->tvb, start, 0,
+        "%s: ERROR: Couldn't parse value: %s",
+        proto_registrar_get_name(hf_id), asn1_err_to_str(ret));
+    }
+    return ret;
+  }
 
   if (i)
     *i = integer;
 
   if (tree)
-  {
-    proto_tree *temp_tree = 0;
-    temp_tree = proto_tree_add_boolean(tree, hf_id, NullTVB, start-a->begin, a->pointer-start, integer);
-    if (new_tree)
-      *new_tree = temp_tree;
-  }
+    temp_item = proto_tree_add_boolean(tree, hf_id, a->tvb, start, a->offset-start, integer);
+  if (new_item)
+    *new_item = temp_item;
 
-  return 0;
+  return ASN1_ERR_NOERROR;
 }
 
 static int read_boolean(ASN1_SCK *a, proto_tree *tree, int hf_id,
-       proto_tree **new_tree, guint *i)
+       proto_item **new_item, guint *i)
 {
   guint cls, con, tag;
   gboolean def;
   guint length;
-  const guchar *start = a->pointer;
-  
-  if (asn1_header_decode(a, &cls, &con, &tag, &def, &length) != ASN1_ERR_NOERROR)
-    return 1;
-  if (cls != ASN1_UNI || con != ASN1_PRI || tag != ASN1_BOL)
-    return 1;
-
-  return read_boolean_value(a, tree, hf_id, new_tree, i, start, length);
+  int start = a->offset;
+  int ret;
+
+  ret = asn1_header_decode(a, &cls, &con, &tag, &def, &length);
+  if (ret == ASN1_ERR_NOERROR) {
+    if (cls != ASN1_UNI || con != ASN1_PRI || tag != ASN1_BOL)
+      ret = ASN1_ERR_WRONG_TYPE;
+  }
+  if (ret != ASN1_ERR_NOERROR) {
+    if (tree) {
+      proto_tree_add_text(tree, a->tvb, start, 0,
+        "%s: ERROR: Couldn't parse header: %s",
+        proto_registrar_get_name(hf_id), asn1_err_to_str(ret));
+    }
+    return ret;
+  }
+
+  return read_boolean_value(a, tree, hf_id, new_item, i, start, length);
 }
 
-static void read_string_value(ASN1_SCK *a, proto_tree *tree, int hf_id,
-       proto_tree **new_tree, char **s, const guchar *start, guint length)
+static int read_string_value(ASN1_SCK *a, proto_tree *tree, int hf_id,
+       proto_item **new_item, char **s, int start, guint length)
 {
   guchar *string;
-  
+  proto_item *temp_item = NULL;
+  int ret;
+
   if (length)
   {
-    asn1_string_value_decode(a, length, &string);
+    ret = asn1_string_value_decode(a, length, &string);
+    if (ret != ASN1_ERR_NOERROR) {
+      if (tree) {
+        proto_tree_add_text(tree, a->tvb, start, 0,
+          "%s: ERROR: Couldn't parse value: %s",
+          proto_registrar_get_name(hf_id), asn1_err_to_str(ret));
+      }
+      return ret;
+    }
     string = g_realloc(string, length + 1);
     string[length] = '\0';
   }
   else
     string = "(null)";
-    
+
   if (tree)
-  {
-    proto_tree *temp_tree;
-    temp_tree = proto_tree_add_string(tree, hf_id, NullTVB, start - a->begin, a->pointer - start, string);
-    if (new_tree)
-      *new_tree = temp_tree;
-  }
+    temp_item = proto_tree_add_string(tree, hf_id, a->tvb, start, a->offset - start, string);
+  if (new_item)
+    *new_item = temp_item;
 
   if (s && length)
     *s = string;
   else if (length)
     g_free(string);
+
+  return ASN1_ERR_NOERROR;
 }
 
 static int read_string(ASN1_SCK *a, proto_tree *tree, int hf_id,
-       proto_tree **new_tree, char **s, guint expected_cls, guint expected_tag)
+       proto_item **new_item, char **s, guint expected_cls, guint expected_tag)
 {
   guint cls, con, tag;
   gboolean def;
   guint length;
-  const guchar *start = a->pointer;
+  int start = a->offset;
   int ret;
-  
+
   ret = asn1_header_decode(a, &cls, &con, &tag, &def, &length);
-  if (ret != ASN1_ERR_NOERROR)
+  if (ret == ASN1_ERR_NOERROR) {
+    if (cls != expected_cls || con != ASN1_PRI || tag != expected_tag)
+      ret = ASN1_ERR_WRONG_TYPE;
+  }
+  if (ret != ASN1_ERR_NOERROR) {
+    if (tree) {
+      proto_tree_add_text(tree, a->tvb, start, 0,
+        "%s: ERROR: Couldn't parse header: %s",
+        proto_registrar_get_name(hf_id), asn1_err_to_str(ret));
+    }
     return ret;
-  if (cls != expected_cls || con != ASN1_PRI || tag != expected_tag)
-    return ASN1_ERR_WRONG_TYPE;
+  }
+
+  return read_string_value(a, tree, hf_id, new_item, s, start, length);
+}
+
+static int read_bytestring_value(ASN1_SCK *a, proto_tree *tree, int hf_id,
+       proto_item **new_item, char **s, int start, guint length)
+{
+  guchar *string;
+  proto_item *temp_item = NULL;
+  int ret;
+
+  if (length)
+  {
+    ret = asn1_string_value_decode(a, length, &string);
+    if (ret != ASN1_ERR_NOERROR) {
+      if (tree) {
+        proto_tree_add_text(tree, a->tvb, start, 0,
+          "%s: ERROR: Couldn't parse value: %s",
+          proto_registrar_get_name(hf_id), asn1_err_to_str(ret));
+      }
+      return ret;
+    }
+    string = g_realloc(string, length + 1);
+    string[length] = '\0';
+  }
+  else
+    string = "(null)";
+
+  if (tree)
+    temp_item = proto_tree_add_bytes(tree, hf_id, a->tvb, start, a->offset - start, string);
+  if (new_item)
+    *new_item = temp_item;
+
+  if (s && length)
+    *s = string;
+  else if (length)
+    g_free(string);
 
-  read_string_value(a, tree, hf_id, new_tree, s, start, length);
   return ASN1_ERR_NOERROR;
 }
 
+static int read_bytestring(ASN1_SCK *a, proto_tree *tree, int hf_id,
+       proto_item **new_item, char **s, guint expected_cls, guint expected_tag)
+{
+  guint cls, con, tag;
+  gboolean def;
+  guint length;
+  int start = a->offset;
+  int ret;
+
+  ret = asn1_header_decode(a, &cls, &con, &tag, &def, &length);
+  if (ret == ASN1_ERR_NOERROR) {
+    if (cls != expected_cls || con != ASN1_PRI || tag != expected_tag)
+      ret = ASN1_ERR_WRONG_TYPE;
+  }
+  if (ret != ASN1_ERR_NOERROR) {
+    if (tree) {
+      proto_tree_add_text(tree, a->tvb, start, 0,
+        "%s: ERROR: Couldn't parse header: %s",
+        proto_registrar_get_name(hf_id), asn1_err_to_str(ret));
+    }
+    return ret;
+  }
+
+  return read_bytestring_value(a, tree, hf_id, new_item, s, start, length);
+}
+
 static int parse_filter_strings(ASN1_SCK *a, char **filter, guint *filter_length, const guchar *operation)
 {
   guchar *string;
@@ -348,13 +511,13 @@ static int parse_filter_strings(ASN1_SCK *a, char **filter, guint *filter_length
 /* Richard Dawe: To parse substring filters, I added this function. */
 static int parse_filter_substrings(ASN1_SCK *a, char **filter, guint *filter_length)
 {
-  guchar *end;
+  int end;
   guchar *string;
   char *filterp;
   guint string_length;
   guint string_bytes;
   guint seq_len;
-  guint header_bytes;  
+  guint header_bytes;
   int ret, any_valued;
 
   /* For ASN.1 parsing of octet strings */
@@ -373,7 +536,7 @@ static int parse_filter_substrings(ASN1_SCK *a, char **filter, guint *filter_len
 
   *filter_length += 2 + 1 + string_length;
   *filter = g_realloc(*filter, *filter_length);
-  
+
   filterp = *filter + strlen(*filter);
   *filterp++ = '(';
   if (string_length != 0) {
@@ -386,9 +549,9 @@ static int parse_filter_substrings(ASN1_SCK *a, char **filter, guint *filter_len
 
   /* Now decode seq_len's worth of octet strings. */
   any_valued = 0;
-  end = (guchar *) (a->pointer + seq_len);
+  end = a->offset + seq_len;
 
-  while (a->pointer < end) {
+  while (a->offset < end) {
     /* Octet strings here are context-specific, which
      * asn1_octet_string_decode() barfs on. Emulate it, but don't barf. */
     ret = asn1_header_decode (a, &cls, &con, &tag, &def, &string_length);
@@ -437,7 +600,7 @@ static int parse_filter_substrings(ASN1_SCK *a, char **filter, guint *filter_len
     filterp = *filter + strlen(*filter);
     *filterp++ = '*';
   }
-  
+
   /* NB: Allocated byte for this earlier */
   *filterp++ = ')';
   *filterp = '\0';
@@ -446,7 +609,8 @@ static int parse_filter_substrings(ASN1_SCK *a, char **filter, guint *filter_len
 }
 
 /* Returns -1 if we're at the end, returns an ASN1_ERR value otherwise. */
-static int parse_filter(ASN1_SCK *a, char **filter, guint *filter_length, const guchar **end)
+static int parse_filter(ASN1_SCK *a, char **filter, guint *filter_length,
+                       int *end)
 {
   guint cls, con, tag;
   guint length;
@@ -456,10 +620,10 @@ static int parse_filter(ASN1_SCK *a, char **filter, guint *filter_length, const
   ret = asn1_header_decode(a, &cls, &con, &tag, &def, &length);
   if (ret != ASN1_ERR_NOERROR)
     return ret;
-  
+
   if (*end == 0)
   {
-    *end = a->pointer + length;
+    *end = a->offset + length;
     *filter_length = 1;
     *filter = g_malloc0(*filter_length);
   }
@@ -470,11 +634,11 @@ static int parse_filter(ASN1_SCK *a, char **filter, guint *filter_length, const
     {
      case LDAP_FILTER_AND:
       {
-        const guchar *add_end;
+        int add_end;
 
         if (con != ASN1_CON)
           return ASN1_ERR_WRONG_TYPE;
-        add_end = a->pointer + length;
+        add_end = a->offset + length;
         *filter_length += 3;
         *filter = g_realloc(*filter, *filter_length);
         strcat(*filter, "(&");
@@ -488,11 +652,11 @@ static int parse_filter(ASN1_SCK *a, char **filter, guint *filter_length, const
       break;
      case LDAP_FILTER_OR:
       {
-        const guchar *or_end;
+        int or_end;
 
         if (con != ASN1_CON)
           return ASN1_ERR_WRONG_TYPE;
-        or_end = a->pointer + length;
+        or_end = a->offset + length;
         *filter_length += 3;
         *filter = g_realloc(*filter, *filter_length);
         strcat(*filter, "(|");
@@ -506,11 +670,11 @@ static int parse_filter(ASN1_SCK *a, char **filter, guint *filter_length, const
       break;
      case LDAP_FILTER_NOT:
       {
-        const guchar *not_end;
+        int not_end;
 
         if (con != ASN1_CON)
           return ASN1_ERR_WRONG_TYPE;
-        not_end = a->pointer + length;
+        not_end = a->offset + length;
         *filter_length += 3;
         *filter = g_realloc(*filter, *filter_length);
         strcat(*filter, "(!");
@@ -524,14 +688,14 @@ static int parse_filter(ASN1_SCK *a, char **filter, guint *filter_length, const
       if (con != ASN1_CON)
         return ASN1_ERR_WRONG_TYPE;
       ret = parse_filter_strings(a, filter, filter_length, "=");
-      if (ret != -1 && ret != ASN1_ERR_NOERROR)
+      if (ret != ASN1_ERR_NOERROR)
         return ret;
       break;
      case LDAP_FILTER_GE:
       if (con != ASN1_CON)
         return ASN1_ERR_WRONG_TYPE;
       ret = parse_filter_strings(a, filter, filter_length, ">=");
-      if (ret != -1 && ret != ASN1_ERR_NOERROR)
+      if (ret != ASN1_ERR_NOERROR)
         return ret;
       break;
      case LDAP_FILTER_LE:
@@ -545,14 +709,14 @@ static int parse_filter(ASN1_SCK *a, char **filter, guint *filter_length, const
       if (con != ASN1_CON)
         return ASN1_ERR_WRONG_TYPE;
       ret = parse_filter_strings(a, filter, filter_length, "~=");
-      if (ret != -1 && ret != ASN1_ERR_NOERROR)
+      if (ret != ASN1_ERR_NOERROR)
         return ret;
       break;
      case LDAP_FILTER_PRESENT:
       {
         guchar *string;
         char *filterp;
-    
+
         if (con != ASN1_PRI)
           return ASN1_ERR_WRONG_TYPE;
         ret = asn1_string_value_decode(a, length, &string);
@@ -578,421 +742,1219 @@ static int parse_filter(ASN1_SCK *a, char **filter, guint *filter_length, const
         return ASN1_ERR_WRONG_TYPE;
       /* Richard Dawe: Handle substrings */
       ret = parse_filter_substrings(a, filter, filter_length);
-      if (ret != -1 && ret != ASN1_ERR_NOERROR)
+      if (ret != ASN1_ERR_NOERROR)
         return ret;
       break;
      default:
       return ASN1_ERR_WRONG_TYPE;
     }
   }
-  
-  if (a->pointer == *end)
+
+  if (a->offset == *end)
     return -1;
   else
-    return ret;
+    return ASN1_ERR_NOERROR;
 }
 
-static int read_filter(ASN1_SCK *a, proto_tree *tree, int hf_id)
+static gboolean read_filter(ASN1_SCK *a, proto_tree *tree, int hf_id)
 {
-  const guchar *start = a->pointer;
+  int start = a->offset;
   char *filter = 0;
   guint filter_length = 0;
-  const guchar *end = 0;
+  int end = 0;
   int ret;
-     
+
   while ((ret = parse_filter(a, &filter, &filter_length, &end))
        == ASN1_ERR_NOERROR)
     continue;
 
   if (tree) {
     if (ret != -1) {
-      proto_tree_add_text(tree, NullTVB, start-a->begin, 0,
-        "Error parsing filter (%d)", ret);
+      proto_tree_add_text(tree, a->tvb, start, 0,
+        "%s: ERROR: Can't parse filter: %s",
+        proto_registrar_get_name(hf_id), asn1_err_to_str(ret));
     } else
-      proto_tree_add_string(tree, hf_id, NullTVB, start-a->begin, a->pointer-start, filter);
+      proto_tree_add_string(tree, hf_id, a->tvb, start, a->offset-start, filter);
   }
 
   g_free(filter);
 
-  return 0;
+  return (ret == -1) ? TRUE : FALSE;
 }
 
 /********************************************************************************************/
 
-static int dissect_ldap_result(ASN1_SCK *a, proto_tree *tree)
+static void dissect_ldap_result(ASN1_SCK *a, proto_tree *tree)
 {
   guint resultCode = 0;
-  
-  read_integer(a, tree, hf_ldap_message_result, 0, &resultCode, ASN1_ENUM);
-  read_string(a, tree, hf_ldap_message_result_matcheddn, 0, 0, ASN1_UNI, ASN1_OTS);
-  read_string(a, tree, hf_ldap_message_result_errormsg, 0, 0, ASN1_UNI, ASN1_OTS);
+  int ret;
+
+  if (read_integer(a, tree, hf_ldap_message_result, 0, &resultCode, ASN1_ENUM) != ASN1_ERR_NOERROR)
+    return;
+  if (read_string(a, tree, hf_ldap_message_result_matcheddn, 0, 0, ASN1_UNI, ASN1_OTS) != ASN1_ERR_NOERROR)
+    return;
+  if (read_string(a, tree, hf_ldap_message_result_errormsg, 0, 0, ASN1_UNI, ASN1_OTS) != ASN1_ERR_NOERROR)
+    return;
 
   if (resultCode == 10)                /* Referral */
   {
-    const guchar *start = a->pointer;
-    const guchar *end;
+    int start = a->offset;
+    int end;
     guint length;
-    proto_tree *t, *referralTree;
-    
-    read_sequence(a, &length);
-    t = proto_tree_add_text(tree, NullTVB, start-a->begin, length, "Referral URLs");
-    referralTree = proto_item_add_subtree(t, ett_ldap_referrals);
-
-    end = a->pointer + length;;
-    while (a->pointer < end)
-      read_string(a, referralTree, hf_ldap_message_result_referral, 0, 0, ASN1_UNI, ASN1_OTS);
+    proto_item *ti;
+    proto_tree *referralTree;
+
+    ret = read_sequence(a, &length);
+    if (ret != ASN1_ERR_NOERROR) {
+      if (tree) {
+        proto_tree_add_text(tree, a->tvb, start, 0,
+            "ERROR: Couldn't parse referral URL sequence header: %s",
+            asn1_err_to_str(ret));
+      }
+      return;
+    }
+    ti = proto_tree_add_text(tree, a->tvb, start, length, "Referral URLs");
+    referralTree = proto_item_add_subtree(ti, ett_ldap_referrals);
+
+    end = a->offset + length;
+    while (a->offset < end) {
+      if (read_string(a, referralTree, hf_ldap_message_result_referral, 0, 0, ASN1_UNI, ASN1_OTS) != ASN1_ERR_NOERROR)
+        return;
+    }
   }
-    
-  return 0;
 }
 
-static int dissect_ldap_request_bind(ASN1_SCK *a, proto_tree *tree)
+static void dissect_ldap_request_bind(ASN1_SCK *a, proto_tree *tree,
+    tvbuff_t *tvb, packet_info *pinfo)
 {
   guint cls, con, tag;
-  guint def, length;
-  const guchar *start;
-
-  read_integer(a, tree, hf_ldap_message_bind_version, 0, 0, ASN1_INT);
-  read_string(a, tree, hf_ldap_message_bind_dn, 0, 0, ASN1_UNI, ASN1_OTS);
-
-  start = a->pointer;
-  if (asn1_header_decode(a, &cls, &con, &tag, &def, &length) != ASN1_ERR_NOERROR)
-    return 1;  /* XXX - right return value for an error? */
-  if (cls != ASN1_CTX)
-    return 1;  /* RFCs 1777 and 2251 say these are context-specific types */
-  proto_tree_add_uint(tree, hf_ldap_message_bind_auth, NullTVB, start - a->begin,
-                       a->pointer - start, tag);
+  gboolean def;
+  guint length;
+  int start;
+  int end;
+  int ret;
+  conversation_t *conversation;
+  ldap_auth_info_t *auth_info;
+  char *mechanism;
+  int token_offset;
+  gint available_length, reported_length;
+  tvbuff_t *new_tvb;
+  proto_item *gitem;
+  proto_tree *gtree = NULL;
+
+  if (read_integer(a, tree, hf_ldap_message_bind_version, 0, 0, ASN1_INT) != ASN1_ERR_NOERROR)
+    return;
+  if (read_string(a, tree, hf_ldap_message_bind_dn, 0, 0, ASN1_UNI, ASN1_OTS) != ASN1_ERR_NOERROR)
+    return;
+
+  start = a->offset;
+  ret = asn1_header_decode(a, &cls, &con, &tag, &def, &length);
+  if (ret == ASN1_ERR_NOERROR) {
+    if (cls != ASN1_CTX) {
+      /* RFCs 1777 and 2251 say these are context-specific types */
+      ret = ASN1_ERR_WRONG_TYPE;
+    }
+  }
+  if (ret != ASN1_ERR_NOERROR) {
+    proto_tree_add_text(tree, a->tvb, start, 0,
+      "%s: ERROR: Couldn't parse header: %s",
+      proto_registrar_get_name(hf_ldap_message_bind_auth),
+      asn1_err_to_str(ret));
+    return;
+  }
+  proto_tree_add_uint(tree, hf_ldap_message_bind_auth, a->tvb, start,
+                       a->offset - start, tag);
+  end = a->offset + length;
   switch (tag)
   {
    case LDAP_AUTH_SIMPLE:
-    read_string_value(a, tree, hf_ldap_message_bind_auth_password, NULL, NULL,
-                       start, length);
+    if (read_string_value(a, tree, hf_ldap_message_bind_auth_password, NULL,
+                          NULL, start, length) != ASN1_ERR_NOERROR)
+      return;
     break;
 
     /* For Kerberos V4, dissect it as a ticket. */
-    /* For SASL, dissect it as SaslCredentials. */
+
+   case LDAP_AUTH_SASL:
+    mechanism = NULL;
+    if (read_string(a, tree, hf_ldap_message_bind_auth_mechanism, NULL,
+                    &mechanism, ASN1_UNI, ASN1_OTS) != ASN1_ERR_NOERROR)
+      return;
+
+    /*
+     * We need to remember the authentication type and mechanism for this
+     * conversation.
+     *
+     * XXX - actually, we might need to remember more than one
+     * type and mechanism, if you can unbind and rebind with a
+     * different type and/or mechanism.
+     */
+    conversation = find_conversation(&pinfo->src, &pinfo->dst,
+                                     pinfo->ptype, pinfo->srcport,
+                                     pinfo->destport, 0);
+    if (conversation == NULL) {
+      /* We don't yet have a conversation, so create one. */
+      conversation = conversation_new(&pinfo->src, &pinfo->dst,
+                                      pinfo->ptype, pinfo->srcport,
+                                      pinfo->destport, 0);
+    }
+
+    /*
+     * Do we already have a type and mechanism?
+     */
+    auth_info = conversation_get_proto_data(conversation, proto_ldap);
+    if (auth_info == NULL) {
+      /* No.  Attach that information to the conversation, and add
+         it to the list of information structures. */
+      auth_info = g_mem_chunk_alloc(ldap_auth_info_chunk);
+      auth_info->auth_type = tag;
+      auth_info->auth_mech = mechanism;
+      auth_info->first_auth_frame = 0; /* not known until we see the bind reply */
+      conversation_add_proto_data(conversation, proto_ldap, auth_info);
+      auth_info->next = auth_info_items;
+      auth_info_items = auth_info;
+    } else {
+      /*
+       * Yes.
+       *
+       * If the mechanism in this request is an empty string (which is
+       * returned as a null pointer), use the saved mechanism instead.
+       * Otherwise, if the saved mechanism is an empty string (null),
+       * save this mechanism.
+       */
+      if (mechanism == NULL)
+       mechanism = auth_info->auth_mech;
+      else {
+        if (auth_info->auth_mech == NULL)
+          auth_info->auth_mech = mechanism;
+      }
+    }
+
+    if (a->offset < end) {
+      if (mechanism != NULL && strcmp(mechanism, "GSS-SPNEGO") == 0) {
+        /*
+         * This is a GSS-API token.
+         * Find out how big it is by parsing the ASN.1 header for the
+         * OCTET STREAM that contains it.
+         */
+        token_offset = a->offset;
+        ret = asn1_header_decode(a, &cls, &con, &tag, &def, &length);
+        if (ret != ASN1_ERR_NOERROR) {
+          proto_tree_add_text(tree, a->tvb, token_offset, 0,
+            "%s: ERROR: Couldn't parse header: %s",
+            proto_registrar_get_name(hf_ldap_message_bind_auth_credentials),
+            asn1_err_to_str(ret));
+          return;
+        }
+        if (tree) {
+          gitem = proto_tree_add_text(tree, tvb, token_offset,
+            (a->offset + length) - token_offset, "GSS-API Token");
+          gtree = proto_item_add_subtree(gitem, ett_ldap_gssapi_token);
+        }
+        available_length = tvb_length_remaining(tvb, token_offset);
+        reported_length = tvb_reported_length_remaining(tvb, token_offset);
+        g_assert(available_length >= 0);
+        g_assert(reported_length >= 0);
+        if (available_length > reported_length)
+          available_length = reported_length;
+        if ((guint)available_length > length)
+          available_length = length;
+        if ((guint)reported_length > length)
+          reported_length = length;
+        new_tvb = tvb_new_subset(tvb, a->offset, available_length, reported_length);
+        call_dissector(gssapi_handle, new_tvb, pinfo, gtree);
+        a->offset += length;
+      } else {
+        if (read_bytestring(a, tree, hf_ldap_message_bind_auth_credentials,
+                            NULL, NULL, ASN1_UNI, ASN1_OTS) != ASN1_ERR_NOERROR)
+          return;
+      }
+    }
+    break;
   }
-  
-  return 0;
 }
 
-static int dissect_ldap_response_bind(ASN1_SCK *a, proto_tree *tree)
+static void dissect_ldap_response_bind(ASN1_SCK *a, proto_tree *tree,
+               int start, guint length, tvbuff_t *tvb, packet_info *pinfo)
 {
+  guint cls, con, tag;
+  gboolean def;
+  guint cred_length;
+  int end;
+  int ret;
+  conversation_t *conversation;
+  ldap_auth_info_t *auth_info;
+  int token_offset;
+  gint available_length, reported_length;
+  tvbuff_t *new_tvb;
+  proto_item *gitem;
+  proto_tree *gtree = NULL;
+
+  end = start + length;
   dissect_ldap_result(a, tree);
-  /* FIXME: handle SASL data */
-  return 0;
+  if (a->offset < end) {
+    conversation = find_conversation(&pinfo->src, &pinfo->dst,
+                                     pinfo->ptype, pinfo->srcport,
+                                     pinfo->destport, 0);
+    if (conversation != NULL) {
+      auth_info = conversation_get_proto_data(conversation, proto_ldap);
+      if (auth_info != NULL) {
+        switch (auth_info->auth_type) {
+
+          /* For Kerberos V4, dissect it as a ticket. */
+          /* XXX - what about LDAP_AUTH_SIMPLE? */
+
+        case LDAP_AUTH_SASL:
+          /*
+           * All frames after this are assumed to use a security layer.
+           *
+           * XXX - won't work if there's another reply, with the security
+           * layer, starting in the same TCP segment that ends this
+           * reply, but as LDAP is a request/response protocol, and
+           * as the client probably can't start using authentication until
+           * it gets the bind reply and the server won't send a reply until
+           * it gets a request, that probably won't happen.
+           *
+           * XXX - that assumption is invalid; it's not clear where the
+           * hell you find out whether there's any security layer.  In
+           * one capture, we have two GSS-SPNEGO negotiations, both of
+           * which select MS KRB5, and the only differences in the tokens
+           * is in the RC4-HMAC ciphertext.  The various
+           * draft-ietf--cat-sasl-gssapi-NN.txt drafts seem to imply
+           * that the RFC 2222 spoo with the bitmask and maximum
+           * output message size stuff is done - but where does that
+           * stuff show up?  Is it in the ciphertext, which means it's
+           * presumably encrypted?
+           *
+           * Grrr.  We have to do a gross heuristic, checking whether the
+          * putative LDAP message begins with 0x00 or not, making the
+          * assumption that we won't have more than 2^24 bytes of
+          * encapsulated stuff.
+           */
+          auth_info->first_auth_frame = pinfo->fd->num + 1;
+          if (auth_info->auth_mech != NULL &&
+              strcmp(auth_info->auth_mech, "GSS-SPNEGO") == 0) {
+            /*
+             * This is a GSS-API token.
+             * Find out how big it is by parsing the ASN.1 header for the
+             * OCTET STREAM that contains it.
+             */
+            token_offset = a->offset;
+            ret = asn1_header_decode(a, &cls, &con, &tag, &def, &cred_length);
+            if (ret != ASN1_ERR_NOERROR) {
+              proto_tree_add_text(tree, a->tvb, token_offset, 0,
+                "%s: ERROR: Couldn't parse header: %s",
+                proto_registrar_get_name(hf_ldap_message_bind_auth_credentials),
+                asn1_err_to_str(ret));
+              return;
+            }
+            if (tree) {
+              gitem = proto_tree_add_text(tree, tvb, token_offset,
+                (a->offset + cred_length) - token_offset, "GSS-API Token");
+              gtree = proto_item_add_subtree(gitem, ett_ldap_gssapi_token);
+            }
+            available_length = tvb_length_remaining(tvb, token_offset);
+            reported_length = tvb_reported_length_remaining(tvb, token_offset);
+            g_assert(available_length >= 0);
+            g_assert(reported_length >= 0);
+            if (available_length > reported_length)
+              available_length = reported_length;
+            if ((guint)available_length > cred_length)
+              available_length = cred_length;
+            if ((guint)reported_length > cred_length)
+              reported_length = cred_length;
+            new_tvb = tvb_new_subset(tvb, a->offset, available_length, reported_length);
+            call_dissector(gssapi_handle, new_tvb, pinfo, gtree);
+            a->offset += cred_length;
+          } else {
+            if (read_bytestring(a, tree, hf_ldap_message_bind_server_credentials,
+                                NULL, NULL, ASN1_UNI, ASN1_OTS) != ASN1_ERR_NOERROR)
+              return;
+          }
+          break;
+
+        default:
+          if (read_bytestring(a, tree, hf_ldap_message_bind_server_credentials,
+                              NULL, NULL, ASN1_CTX, 7) != ASN1_ERR_NOERROR)
+            return;
+          break;
+        }
+      } else {
+        if (read_bytestring(a, tree, hf_ldap_message_bind_server_credentials,
+                            NULL, NULL, ASN1_CTX, 7) != ASN1_ERR_NOERROR)
+          return;
+      }
+    } else {
+      if (read_bytestring(a, tree, hf_ldap_message_bind_server_credentials,
+                          NULL, NULL, ASN1_CTX, 7) != ASN1_ERR_NOERROR)
+        return;
+    }
+  }
 }
 
-static int dissect_ldap_request_search(ASN1_SCK *a, proto_tree *tree)
+static void dissect_ldap_request_search(ASN1_SCK *a, proto_tree *tree)
 {
   guint seq_length;
-  const guchar *end;
+  int end;
   int ret;
-  
-  read_string(a, tree, hf_ldap_message_search_base, 0, 0, ASN1_UNI, ASN1_OTS);
-  read_integer(a, tree, hf_ldap_message_search_scope, 0, 0, ASN1_ENUM);
-  read_integer(a, tree, hf_ldap_message_search_deref, 0, 0, ASN1_ENUM);
-  read_integer(a, tree, hf_ldap_message_search_sizeLimit, 0, 0, ASN1_INT);
-  read_integer(a, tree, hf_ldap_message_search_timeLimit, 0, 0, ASN1_INT);
-  read_boolean(a, tree, hf_ldap_message_search_typesOnly, 0, 0);
-  ret = read_filter(a, tree, hf_ldap_message_search_filter);
-  if (ret != ASN1_ERR_NOERROR)
-    return ret;
-  read_sequence(a, &seq_length);
-  end = a->pointer + seq_length;
-  while (a->pointer < end) {
-    ret = read_string(a, tree, hf_ldap_message_attribute, 0, 0, ASN1_UNI, ASN1_OTS);
-    if (ret != ASN1_ERR_NOERROR)
-      return ret;
+
+  if (read_string(a, tree, hf_ldap_message_search_base, 0, 0, ASN1_UNI, ASN1_OTS) != ASN1_ERR_NOERROR)
+    return;
+  if (read_integer(a, tree, hf_ldap_message_search_scope, 0, 0, ASN1_ENUM) != ASN1_ERR_NOERROR)
+    return;
+  if (read_integer(a, tree, hf_ldap_message_search_deref, 0, 0, ASN1_ENUM) != ASN1_ERR_NOERROR)
+    return;
+  if (read_integer(a, tree, hf_ldap_message_search_sizeLimit, 0, 0, ASN1_INT) != ASN1_ERR_NOERROR)
+    return;
+  if (read_integer(a, tree, hf_ldap_message_search_timeLimit, 0, 0, ASN1_INT) != ASN1_ERR_NOERROR)
+    return;
+  if (read_boolean(a, tree, hf_ldap_message_search_typesOnly, 0, 0) != ASN1_ERR_NOERROR)
+    return;
+  if (!read_filter(a, tree, hf_ldap_message_search_filter))
+    return;
+  ret = read_sequence(a, &seq_length);
+  if (ret != ASN1_ERR_NOERROR) {
+    if (tree) {
+      proto_tree_add_text(tree, a->tvb, a->offset, 0,
+          "ERROR: Couldn't parse LDAP attribute sequence header: %s",
+          asn1_err_to_str(ret));
+    }
+    return;
+  }
+  end = a->offset + seq_length;
+  while (a->offset < end) {
+    if (read_string(a, tree, hf_ldap_message_attribute, 0, 0, ASN1_UNI,
+                    ASN1_OTS) != ASN1_ERR_NOERROR)
+      return;
   }
-  return ASN1_ERR_NOERROR;
 }
 
-static int dissect_ldap_response_search_entry(ASN1_SCK *a, proto_tree *tree)
+static void dissect_ldap_response_search_entry(ASN1_SCK *a, proto_tree *tree)
 {
   guint seq_length;
-  const guchar *end_of_sequence;
-  read_string(a, tree, hf_ldap_message_dn, 0, 0, ASN1_UNI, ASN1_OTS);
-  read_sequence(a, &seq_length);
+  int end_of_sequence;
+  int ret;
+
+  if (read_string(a, tree, hf_ldap_message_dn, 0, 0, ASN1_UNI, ASN1_OTS) != ASN1_ERR_NOERROR)
+    return;
+  ret = read_sequence(a, &seq_length);
+  if (ret != ASN1_ERR_NOERROR) {
+    if (tree) {
+      proto_tree_add_text(tree, a->tvb, a->offset, 0,
+          "ERROR: Couldn't parse search entry response sequence header: %s",
+          asn1_err_to_str(ret));
+    }
+    return;
+  }
 
-  end_of_sequence = a->pointer + seq_length;
-  while (a->pointer < end_of_sequence)
+  end_of_sequence = a->offset + seq_length;
+  while (a->offset < end_of_sequence)
   {
-    proto_tree *t, *attr_tree;
+    proto_item *ti;
+    proto_tree *attr_tree;
     guint set_length;
-    const guchar *end_of_set;
-
-    read_sequence(a, 0);
-    read_string(a, tree, hf_ldap_message_attribute, &t, 0, ASN1_UNI, ASN1_OTS);
-    attr_tree = proto_item_add_subtree(t, ett_ldap_attribute);
-
-    read_set(a, &set_length);
-    end_of_set = a->pointer + set_length;
-    while (a->pointer < end_of_set)
-      read_string(a, attr_tree, hf_ldap_message_value, 0, 0, ASN1_UNI, ASN1_OTS);
+    int end_of_set;
+
+    ret = read_sequence(a, 0);
+    if (ret != ASN1_ERR_NOERROR) {
+      if (tree) {
+        proto_tree_add_text(tree, a->tvb, a->offset, 0,
+            "ERROR: Couldn't parse LDAP attribute sequence header: %s",
+            asn1_err_to_str(ret));
+      }
+      return;
+    }
+    if (read_string(a, tree, hf_ldap_message_attribute, &ti, 0, ASN1_UNI, ASN1_OTS) != ASN1_ERR_NOERROR)
+      return;
+    attr_tree = proto_item_add_subtree(ti, ett_ldap_attribute);
+
+    ret = read_set(a, &set_length);
+    if (ret != ASN1_ERR_NOERROR) {
+      if (tree) {
+        proto_tree_add_text(attr_tree, a->tvb, a->offset, 0,
+            "ERROR: Couldn't parse LDAP value set header: %s",
+            asn1_err_to_str(ret));
+      }
+      return;
+    }
+    end_of_set = a->offset + set_length;
+    while (a->offset < end_of_set) {
+      if (read_string(a, attr_tree, hf_ldap_message_value, 0, 0, ASN1_UNI,
+                      ASN1_OTS) != ASN1_ERR_NOERROR)
+        return;
+    }
   }
-
-  return 0;
 }
 
-static int dissect_ldap_request_add(ASN1_SCK *a, proto_tree *tree)
+static void dissect_ldap_request_add(ASN1_SCK *a, proto_tree *tree)
 {
   guint seq_length;
-  const guchar *end_of_sequence;
-  
-  read_string(a, tree, hf_ldap_message_dn, 0, 0, ASN1_UNI, ASN1_OTS);
-
-  read_sequence(a, &seq_length);
-  end_of_sequence = a->pointer + seq_length;
-  while (a->pointer < end_of_sequence)
-  {
-    proto_tree *t, *attr_tree;
-    guint set_length;
-    const guchar *end_of_set;
+  int end_of_sequence;
+  int ret;
 
-    read_sequence(a, 0);
-    read_string(a, tree, hf_ldap_message_attribute, &t, 0, ASN1_UNI, ASN1_OTS);
-    attr_tree = proto_item_add_subtree(t, ett_ldap_attribute);
+  if (read_string(a, tree, hf_ldap_message_dn, 0, 0, ASN1_UNI, ASN1_OTS) != ASN1_ERR_NOERROR)
+    return;
 
-    read_set(a, &set_length);
-    end_of_set = a->pointer + set_length;
-    while (a->pointer < end_of_set)
-      read_string(a, attr_tree, hf_ldap_message_value, 0, 0, ASN1_UNI, ASN1_OTS);
+  ret = read_sequence(a, &seq_length);
+  if (ret != ASN1_ERR_NOERROR) {
+    if (tree) {
+      proto_tree_add_text(tree, a->tvb, a->offset, 0,
+          "ERROR: Couldn't parse add request sequence header: %s",
+          asn1_err_to_str(ret));
+    }
+    return;
   }
 
-  return 0;
+  end_of_sequence = a->offset + seq_length;
+  while (a->offset < end_of_sequence)
+  {
+    proto_item *ti;
+    proto_tree *attr_tree;
+    guint set_length;
+    int end_of_set;
+
+    ret = read_sequence(a, 0);
+    if (ret != ASN1_ERR_NOERROR) {
+      if (tree) {
+        proto_tree_add_text(tree, a->tvb, a->offset, 0,
+            "ERROR: Couldn't parse LDAP attribute sequence header: %s",
+            asn1_err_to_str(ret));
+      }
+      return;
+    }
+    if (read_string(a, tree, hf_ldap_message_attribute, &ti, 0, ASN1_UNI,
+                    ASN1_OTS) != ASN1_ERR_NOERROR)
+      return;
+    attr_tree = proto_item_add_subtree(ti, ett_ldap_attribute);
+
+    ret = read_set(a, &set_length);
+    if (ret != ASN1_ERR_NOERROR) {
+      if (tree) {
+        proto_tree_add_text(attr_tree, a->tvb, a->offset, 0,
+            "ERROR: Couldn't parse LDAP value set header: %s",
+            asn1_err_to_str(ret));
+      }
+      return;
+    }
+    end_of_set = a->offset + set_length;
+    while (a->offset < end_of_set) {
+      if (read_string(a, attr_tree, hf_ldap_message_value, 0, 0, ASN1_UNI, ASN1_OTS) != ASN1_ERR_NOERROR)
+        return;
+    }
+  }
 }
 
-static int dissect_ldap_request_delete(ASN1_SCK *a, proto_tree *tree,
-               const guchar *start, guint length)
+static void dissect_ldap_request_delete(ASN1_SCK *a, proto_tree *tree,
+               int start, guint length)
 {
   read_string_value(a, tree, hf_ldap_message_dn, NULL, NULL, start, length);
-  return 0;
 }
 
-static int dissect_ldap_request_modifyrdn(ASN1_SCK *a, proto_tree *tree,
+static void dissect_ldap_request_modifyrdn(ASN1_SCK *a, proto_tree *tree,
                guint length)
 {
-  const guchar *start = a->pointer;
+  int start = a->offset;
+
+  if (read_string(a, tree, hf_ldap_message_dn, 0, 0, ASN1_UNI, ASN1_OTS) != ASN1_ERR_NOERROR)
+    return;
+  if (read_string(a, tree, hf_ldap_message_modrdn_name, 0, 0, ASN1_UNI, ASN1_OTS) != ASN1_ERR_NOERROR)
+    return;
+  if (read_boolean(a, tree, hf_ldap_message_modrdn_delete, 0, 0) != ASN1_ERR_NOERROR)
+    return;
 
-  read_string(a, tree, hf_ldap_message_dn, 0, 0, ASN1_UNI, ASN1_OTS);
-  read_string(a, tree, hf_ldap_message_modrdn_name, 0, 0, ASN1_UNI, ASN1_OTS);
-  read_boolean(a, tree, hf_ldap_message_modrdn_delete, 0, 0);
-  
-  if (a->pointer < (start + length)) {
+  if (a->offset < (int) (start + length)) {
     /* LDAP V3 Modify DN operation, with newSuperior */
-    read_string(a, tree, hf_ldap_message_modrdn_superior, 0, 0, ASN1_UNI, ASN1_OTS);
+    /*      "newSuperior     [0] LDAPDN OPTIONAL" (0x80) */
+    if (read_string(a, tree, hf_ldap_message_modrdn_superior, 0, 0, ASN1_CTX, 0) != ASN1_ERR_NOERROR)
+      return;
   }
-
-  return 0;
 }
 
-static int dissect_ldap_request_compare(ASN1_SCK *a, proto_tree *tree)
+static void dissect_ldap_request_compare(ASN1_SCK *a, proto_tree *tree)
 {
-  const guchar *start;
+  int start;
   int length;
-  char *string1 = 0;
-  char *string2 = 0;
+  char *string1 = NULL;
+  char *string2 = NULL;
+  char *s1, *s2;
   char *compare;
-  
-  read_string(a, tree, hf_ldap_message_dn, 0, 0, ASN1_UNI, ASN1_OTS);
-  read_sequence(a, 0);
+  int ret;
 
-  start = a->pointer;
-  read_string(a, 0, -1, 0, &string1, ASN1_UNI, ASN1_OTS);
-  read_string(a, 0, -1, 0, &string2, ASN1_UNI, ASN1_OTS);
+  if (read_string(a, tree, hf_ldap_message_dn, 0, 0, ASN1_UNI, ASN1_OTS) != ASN1_ERR_NOERROR)
+    return;
+  ret = read_sequence(a, 0);
+  if (ret != ASN1_ERR_NOERROR) {
+    if (tree) {
+      proto_tree_add_text(tree, a->tvb, a->offset, 0,
+          "ERROR: Couldn't parse compare request sequence header: %s",
+          asn1_err_to_str(ret));
+    }
+    return;
+  }
+
+  start = a->offset;
+  ret = read_string(a, 0, -1, 0, &string1, ASN1_UNI, ASN1_OTS);
+  if (ret != ASN1_ERR_NOERROR) {
+    if (tree) {
+      proto_tree_add_text(tree, a->tvb, start, 0,
+        "ERROR: Couldn't parse compare type: %s", asn1_err_to_str(ret));
+    }
+    return;
+  }
+  ret = read_string(a, 0, -1, 0, &string2, ASN1_UNI, ASN1_OTS);
+  if (ret != ASN1_ERR_NOERROR) {
+    if (tree) {
+      proto_tree_add_text(tree, a->tvb, start, 0,
+        "ERROR: Couldn't parse compare value: %s", asn1_err_to_str(ret));
+    }
+    return;
+  }
 
-  length = 2 + strlen(string1) + strlen(string2);
+  s1 = (string1 == NULL) ? "(null)" : string1;
+  s2 = (string2 == NULL) ? "(null)" : string2;
+  length = 2 + strlen(s1) + strlen(s2);
   compare = g_malloc0(length);
-  snprintf(compare, length, "%s=%s", string1, string2);
-  proto_tree_add_string(tree, hf_ldap_message_compare, NullTVB, start-a->begin, a->pointer-start, compare);
-  
+  snprintf(compare, length, "%s=%s", s1, s2);
+  proto_tree_add_string(tree, hf_ldap_message_compare, a->tvb, start,
+      a->offset-start, compare);
+
   g_free(string1);
   g_free(string2);
   g_free(compare);
-  
-  return 0;
+
+  return;
 }
 
-static int dissect_ldap_request_modify(ASN1_SCK *a, proto_tree *tree)
+static void dissect_ldap_request_modify(ASN1_SCK *a, proto_tree *tree)
 {
   guint seq_length;
-  const guchar *end_of_sequence;
-  
-  read_string(a, tree, hf_ldap_message_dn, 0, 0, ASN1_UNI, ASN1_OTS);
-  read_sequence(a, &seq_length);
-  end_of_sequence = a->pointer + seq_length;
-  while (a->pointer < end_of_sequence)
+  int end_of_sequence;
+  int ret;
+
+  if (read_string(a, tree, hf_ldap_message_dn, 0, 0, ASN1_UNI, ASN1_OTS) != ASN1_ERR_NOERROR)
+    return;
+  ret = read_sequence(a, &seq_length);
+  if (ret != ASN1_ERR_NOERROR) {
+    if (tree) {
+      proto_tree_add_text(tree, a->tvb, a->offset, 0,
+          "ERROR: Couldn't parse modify request sequence header: %s",
+          asn1_err_to_str(ret));
+    }
+    return;
+  }
+  end_of_sequence = a->offset + seq_length;
+  while (a->offset < end_of_sequence)
   {
-    proto_tree *t = 0, *attr_tree;
+    proto_item *ti;
+    proto_tree *attr_tree;
     guint set_length;
-    const guchar *end_of_set;
+    int end_of_set;
     guint operation;
 
-    read_sequence(a, 0);
-    read_integer(a, 0, -1, 0, &operation, ASN1_ENUM);
-    read_sequence(a, 0);
+    ret = read_sequence(a, 0);
+    if (ret != ASN1_ERR_NOERROR) {
+      if (tree) {
+        proto_tree_add_text(tree, a->tvb, a->offset, 0,
+            "ERROR: Couldn't parse modify request item sequence header: %s",
+            asn1_err_to_str(ret));
+      }
+      return;
+    }
+    ret = read_integer(a, 0, -1, 0, &operation, ASN1_ENUM);
+    if (ret != ASN1_ERR_NOERROR) {
+      if (tree) {
+        proto_tree_add_text(tree, a->tvb, a->offset, 0,
+          "ERROR: Couldn't parse modify operation: %s",
+          asn1_err_to_str(ret));
+        return;
+      }
+    }
+    ret = read_sequence(a, 0);
+    if (ret != ASN1_ERR_NOERROR) {
+      if (tree) {
+        proto_tree_add_text(tree, a->tvb, a->offset, 0,
+            "ERROR: Couldn't parse modify request operation sequence header: %s",
+            asn1_err_to_str(ret));
+      }
+      return;
+    }
 
     switch (operation)
     {
      case LDAP_MOD_ADD:
-      read_string(a, tree, hf_ldap_message_modify_add, &t, 0, ASN1_UNI, ASN1_OTS);
+      if (read_string(a, tree, hf_ldap_message_modify_add, &ti, 0, ASN1_UNI,
+                      ASN1_OTS) != ASN1_ERR_NOERROR)
+        return;
       break;
+
      case LDAP_MOD_REPLACE:
-      read_string(a, tree, hf_ldap_message_modify_replace, &t, 0, ASN1_UNI, ASN1_OTS);
+      if (read_string(a, tree, hf_ldap_message_modify_replace, &ti, 0,
+                      ASN1_UNI, ASN1_OTS) != ASN1_ERR_NOERROR)
+        return;
       break;
+
      case LDAP_MOD_DELETE:
-      read_string(a, tree, hf_ldap_message_modify_delete, &t, 0, ASN1_UNI, ASN1_OTS);
+      if (read_string(a, tree, hf_ldap_message_modify_delete, &ti, 0,
+                      ASN1_UNI, ASN1_OTS) != ASN1_ERR_NOERROR)
+        return;
       break;
-    }
-    attr_tree = proto_item_add_subtree(t, ett_ldap_attribute);
 
-    read_set(a, &set_length);
-    end_of_set = a->pointer + set_length;
-    while (a->pointer < end_of_set)
-      read_string(a, attr_tree, hf_ldap_message_value, 0, 0, ASN1_UNI, ASN1_OTS);
+     default:
+       proto_tree_add_text(tree, a->tvb, a->offset, 0,
+            "Unknown LDAP modify operation (%u)", operation);
+       return;
+    }
+    attr_tree = proto_item_add_subtree(ti, ett_ldap_attribute);
+
+    ret = read_set(a, &set_length);
+    if (ret != ASN1_ERR_NOERROR) {
+      if (tree) {
+        proto_tree_add_text(attr_tree, a->tvb, a->offset, 0,
+            "ERROR: Couldn't parse LDAP value set header: %s",
+            asn1_err_to_str(ret));
+      }
+      return;
+    }
+    end_of_set = a->offset + set_length;
+    while (a->offset < end_of_set) {
+      if (read_string(a, attr_tree, hf_ldap_message_value, 0, 0, ASN1_UNI,
+                      ASN1_OTS) != ASN1_ERR_NOERROR)
+        return;
+    }
   }
-
-  return 0;
 }
 
-static int dissect_ldap_request_abandon(ASN1_SCK *a, proto_tree *tree,
-               const guchar *start, guint length)
+static void dissect_ldap_request_abandon(ASN1_SCK *a, proto_tree *tree,
+               int start, guint length)
 {
   read_integer_value(a, tree, hf_ldap_message_abandon_msgid, NULL, NULL,
-                       start, length); 
-  return 0;
+                           start, length);
 }
 
 static void
-dissect_ldap(const u_char *pd, int offset, frame_data *fd, proto_tree *tree)
+dissect_ldap_message(tvbuff_t *tvb, int offset, packet_info *pinfo,
+                     proto_tree *ldap_tree, gboolean first_time)
 {
-  proto_tree *ldap_tree = 0, *ti, *msg_tree;
+  int message_id_start;
+  int message_id_length;
   guint messageLength;
   guint messageId;
+  int next_offset;
   guint protocolOpCls, protocolOpCon, protocolOpTag;
   gchar *typestr;
   guint opLen;
   ASN1_SCK a;
-  const guchar *start;
-  int first_time = 1;
+  int start;
   int ret;
 
-  OLD_CHECK_DISPLAY_AS_DATA(proto_ldap, pd, offset, fd, tree);
+  asn1_open(&a, tvb, offset);
+
+  ret = read_sequence(&a, &messageLength);
+  if (ret != ASN1_ERR_NOERROR)
+  {
+    if (first_time)
+    {
+      if (check_col(pinfo->cinfo, COL_INFO))
+      {
+        col_add_fstr(pinfo->cinfo, COL_INFO,
+                    "Invalid LDAP message (Can't parse sequence header: %s)",
+                    asn1_err_to_str(ret));
+      }
+    }
+    if (ldap_tree)
+    {
+      proto_tree_add_text(ldap_tree, tvb, offset, -1,
+                         "Invalid LDAP message (Can't parse sequence header: %s)",
+                         asn1_err_to_str(ret));
+    }
+    return;
+  }
 
-  if (tree) 
+  message_id_start = a.offset;
+  ret = read_integer(&a, 0, hf_ldap_message_id, 0, &messageId, ASN1_INT);
+  if (ret != ASN1_ERR_NOERROR)
   {
-    ti = proto_tree_add_item(tree, proto_ldap, NullTVB, offset, END_OF_FRAME, FALSE);
-    ldap_tree = proto_item_add_subtree(ti, ett_ldap);
+    if (first_time && check_col(pinfo->cinfo, COL_INFO))
+      col_add_fstr(pinfo->cinfo, COL_INFO, "Invalid LDAP packet (Can't parse Message ID: %s)",
+                   asn1_err_to_str(ret));
+    if (ldap_tree)
+      proto_tree_add_text(ldap_tree, tvb, message_id_start, 1,
+                          "Invalid LDAP packet (Can't parse Message ID: %s)",
+                          asn1_err_to_str(ret));
+      return;
   }
+  message_id_length = a.offset - message_id_start;
 
-  asn1_open(&a, pd, pi.captured_len);
-  a.pointer += offset;
+  start = a.offset;
+  asn1_id_decode(&a, &protocolOpCls, &protocolOpCon, &protocolOpTag);
+  if (protocolOpCls != ASN1_APL)
+    typestr = "Bad message type (not Application)";
+  else
+    typestr = val_to_str(protocolOpTag, msgTypes, "Unknown message type (%u)");
+
+  if (first_time)
+  {
+    if (check_col(pinfo->cinfo, COL_INFO))
+      col_add_fstr(pinfo->cinfo, COL_INFO, "MsgId=%u MsgType=%s",
+                  messageId, typestr);
+  }
 
-  while (a.pointer < a.end)
+  if (ldap_tree)
   {
-    int message_id_start;
-    int message_id_length;
-    int message_start;
-    
-    message_start = a.pointer - a.begin;
-    if (read_sequence(&a, &messageLength))
+    proto_tree_add_uint(ldap_tree, hf_ldap_message_id, tvb, message_id_start, message_id_length, messageId);
+    if (protocolOpCls == ASN1_APL)
     {
-      if (ldap_tree)
-        proto_tree_add_text(ldap_tree, NullTVB, offset, 1, "Invalid LDAP packet");
-      break;
+      proto_tree_add_uint(ldap_tree, hf_ldap_message_type, tvb,
+                         start, a.offset - start, protocolOpTag);
     }
+    else
+    {
+      proto_tree_add_text(ldap_tree, tvb, start, a.offset - start,
+                         "%s", typestr);
+    }
+  }
+  start = a.offset;
+  if (read_length(&a, ldap_tree, hf_ldap_message_length, &opLen) != ASN1_ERR_NOERROR)
+    return;
 
-    if (messageLength > (a.end - a.pointer))
+  if (protocolOpCls == ASN1_APL)
+  {
+    switch (protocolOpTag)
     {
+     case LDAP_REQ_BIND:
+      dissect_ldap_request_bind(&a, ldap_tree, tvb, pinfo);
+      break;
+     case LDAP_REQ_UNBIND:
+      /* Nothing to dissect */
+      break;
+     case LDAP_REQ_SEARCH:
+      if (ldap_tree)
+        dissect_ldap_request_search(&a, ldap_tree);
+      break;
+     case LDAP_REQ_MODIFY:
+      if (ldap_tree)
+        dissect_ldap_request_modify(&a, ldap_tree);
+      break;
+     case LDAP_REQ_ADD:
+      if (ldap_tree)
+        dissect_ldap_request_add(&a, ldap_tree);
+      break;
+     case LDAP_REQ_DELETE:
+      if (ldap_tree)
+        dissect_ldap_request_delete(&a, ldap_tree, start, opLen);
+      break;
+     case LDAP_REQ_MODRDN:
+      if (ldap_tree)
+        dissect_ldap_request_modifyrdn(&a, ldap_tree, opLen);
+      break;
+     case LDAP_REQ_COMPARE:
+      if (ldap_tree)
+        dissect_ldap_request_compare(&a, ldap_tree);
+      break;
+     case LDAP_REQ_ABANDON:
+      if (ldap_tree)
+        dissect_ldap_request_abandon(&a, ldap_tree, start, opLen);
+      break;
+     case LDAP_RES_BIND:
+      dissect_ldap_response_bind(&a, ldap_tree, start, opLen, tvb, pinfo);
+      break;
+     case LDAP_RES_SEARCH_ENTRY:
+      if (ldap_tree)
+        dissect_ldap_response_search_entry(&a, ldap_tree);
+      break;
+     case LDAP_RES_SEARCH_RESULT:
+     case LDAP_RES_MODIFY:
+     case LDAP_RES_ADD:
+     case LDAP_RES_DELETE:
+     case LDAP_RES_MODRDN:
+     case LDAP_RES_COMPARE:
+      if (ldap_tree)
+        dissect_ldap_result(&a, ldap_tree);
+      break;
+     default:
       if (ldap_tree)
-        proto_tree_add_text(ldap_tree, NullTVB, message_start, END_OF_FRAME, "Short message! (expected: %u, actual: %u)",
-                           messageLength, a.end - a.pointer);
+      {
+        proto_tree_add_text(ldap_tree, a.tvb, a.offset, opLen,
+                            "Unknown LDAP operation (%u)", protocolOpTag);
+      }
       break;
     }
-  
-    message_id_start = a.pointer - a.begin;
-    read_integer(&a, 0, -1, 0, &messageId, ASN1_INT);
-    message_id_length = (a.pointer - a.begin) - message_id_start;
-
-    start = a.pointer;
-    asn1_id_decode(&a, &protocolOpCls, &protocolOpCon, &protocolOpTag);
-    if (protocolOpCls != ASN1_APL)
-      typestr = "Bad message type (not Application)";
-    else
-      typestr = val_to_str(protocolOpTag, msgTypes, "Bad message type (%u)");
+  }
 
-    if (first_time)
-    {
-      if (check_col(fd, COL_PROTOCOL))
-        col_set_str(fd, COL_PROTOCOL, "LDAP");
-
-      if (check_col(fd, COL_INFO))
-        col_add_fstr(fd, COL_INFO, "MsgId=%u MsgType=%s",
-                    messageId, typestr);
-      first_time = 0;
-      if (!tree)
-       return;
+  /*
+   * XXX - what if "next_offset" is past the offset of the next top-level
+   * sequence?  Show that as an error?
+   */
+  asn1_close(&a, &next_offset);        /* XXX - use the new value of next_offset? */
+}
+
+static void
+dissect_ldap(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
+{
+  int offset = 0;
+  gboolean first_time = TRUE;
+  conversation_t *conversation;
+  ldap_auth_info_t *auth_info = NULL;
+  gboolean doing_sasl_security = FALSE;
+  guint length_remaining;
+  guint32 sasl_length;
+  guint32 message_data_len;
+  proto_item *ti;
+  proto_tree *ldap_tree = NULL;
+  ASN1_SCK a;
+  int ret;
+  guint messageLength;
+  int messageOffset;
+  guint headerLength;
+  guint length;
+  gint available_length, reported_length;
+  int len;
+  proto_item *gitem = NULL;
+  proto_tree *gtree = NULL;
+  tvbuff_t *next_tvb;
+
+  /*
+   * Do we have a conversation for this connection?
+   */
+  conversation = find_conversation(&pinfo->src, &pinfo->dst,
+                                   pinfo->ptype, pinfo->srcport,
+                                   pinfo->destport, 0);
+  if (conversation != NULL) {
+    /*
+     * Yes - do we have any authentication mechanism for it?
+     */
+    auth_info = conversation_get_proto_data(conversation, proto_ldap);
+    if (auth_info != NULL) {
+      /*
+       * Yes - what's the authentication type?
+       */
+      switch (auth_info->auth_type) {
+
+      case LDAP_AUTH_SASL:
+        /*
+         * It's SASL; are we using a security layer?
+         */
+        if (auth_info->first_auth_frame != 0 &&
+            pinfo->fd->num >= auth_info->first_auth_frame)
+          doing_sasl_security = TRUE;  /* yes */
+      }
     }
+  }
 
-    if (ldap_tree) 
-    {
-      proto_tree_add_uint_hidden(ldap_tree, hf_ldap_message_id, NullTVB, message_id_start, message_id_length, messageId);
-      proto_tree_add_uint_hidden(ldap_tree, hf_ldap_message_type, NullTVB,
-                                start - a.begin, a.pointer - start, protocolOpTag);
-      ti = proto_tree_add_text(ldap_tree, NullTVB, message_id_start, messageLength, "Message: Id=%u  %s", messageId, typestr);
-      msg_tree = proto_item_add_subtree(ti, ett_ldap_message);
-      start = a.pointer;
-      read_length(&a, msg_tree, hf_ldap_message_length, &opLen);
-
-      switch (protocolOpTag)
+  while (tvb_reported_length_remaining(tvb, offset) > 0) {
+    /*
+     * This will throw an exception if we don't have any data left.
+     * That's what we want.  (See "tcp_dissect_pdus()", which is
+     * similar, but doesn't have to deal with the SASL issues.
+     * XXX - can we make "tcp_dissect_pdus()" provide enough information
+     * to the "get_pdu_len" routine so that we could have one dealing
+     * with the SASL issues, have that routine deal with SASL and
+     * ASN.1, and just use "tcp_dissect_pdus()"?)
+     */
+    length_remaining = tvb_ensure_length_remaining(tvb, offset);
+
+    /*
+     * Might we be doing a SASL security layer and, if so, *are* we doing
+     * one?
+     *
+     * Just because we've seen a bind reply for SASL, that doesn't mean
+     * that we're using a SASL security layer; I've seen captures in
+     * which some SASL negotiations lead to a security layer being used
+     * and other negotiations don't, and it's not obvious what's different
+     * in the two negotiations.  Therefore, we assume that if the first
+     * byte is 0, it's a length for a SASL security layer (that way, we
+     * never reassemble more than 16 megabytes, protecting us from
+     * chewing up *too* much memory), and otherwise that it's an LDAP
+     * message (actually, if it's an LDAP message it should begin with 0x30,
+     * but we want to parse garbage as LDAP messages rather than really
+     * huge lengths).
+     */
+    if (doing_sasl_security && tvb_get_guint8(tvb, offset) == 0) {
+      /*
+       * Yes.  The frame begins with a 4-byte big-endian length.
+       * Can we do reassembly?
+       */
+      if (ldap_desegment && pinfo->can_desegment) {
+        /*
+         * Yes - is the SASL length split across segment boundaries?
+         */
+        if (length_remaining < 4) {
+          /*
+           * Yes.  Tell the TCP dissector where the data for this message
+           * starts in the data it handed us, and how many more bytes we
+           * need, and return.
+           */
+          pinfo->desegment_offset = offset;
+          pinfo->desegment_len = 4 - length_remaining;
+          return;
+        }
+      }
+
+      /*
+       * Get the SASL length, which is the length of data in the buffer
+       * following the length (i.e., it's 4 less than the total length).
+       *
+       * XXX - do we need to reassemble buffers?  For now, we
+       * assume that each LDAP message is entirely contained within
+       * a buffer.
+       */
+      sasl_length = tvb_get_ntohl(tvb, offset);
+      message_data_len = sasl_length + 4;
+      if (message_data_len < 4) {
+        /*
+         * The message length was probably so large that the total length
+        * overflowed.
+         *
+         * Report this as an error.
+         */
+        show_reported_bounds_error(tvb, pinfo, tree);
+        return;
+      }
+
+      /*
+       * Can we do reassembly?
+       */
+      if (ldap_desegment && pinfo->can_desegment) {
+        /*
+         * Yes - is the buffer split across segment boundaries?
+         */
+        if (length_remaining < message_data_len) {
+          /*
+           * Yes.  Tell the TCP dissector where the data for this message
+           * starts in the data it handed us, and how many more bytes we
+           * need, and return.
+           */
+          pinfo->desegment_offset = offset;
+          pinfo->desegment_len = message_data_len - length_remaining;
+          return;
+        }
+      }
+
+      /*
+       * Construct a tvbuff containing the amount of the payload we have
+       * available.  Make its reported length the amount of data in the PDU.
+       *
+       * XXX - if reassembly isn't enabled. the subdissector will throw a
+       * BoundsError exception, rather than a ReportedBoundsError exception.
+       * We really want a tvbuff where the length is "length", the reported
+       * length is "plen", and the "if the snapshot length were infinite"
+       * length is the minimum of the reported length of the tvbuff handed
+       * to us and "plen", with a new type of exception thrown if the offset
+       * is within the reported length but beyond that third length, with
+       * that exception getting the "Unreassembled Packet" error.
+       */
+      length = length_remaining;
+      if (length > message_data_len)
+        length = message_data_len;
+      next_tvb = tvb_new_subset(tvb, offset, length, message_data_len);
+
+      /*
+       * If this is the first PDU, set the Protocol column and clear the
+       * Info column.
+       */
+      if (first_time)
       {
-       case LDAP_REQ_BIND:
-        dissect_ldap_request_bind(&a, msg_tree);
-        break;
-       case LDAP_REQ_SEARCH:
-        ret = dissect_ldap_request_search(&a, msg_tree);
-        if (ret != ASN1_ERR_NOERROR)
-          break;
-        break;
-       case LDAP_REQ_ADD:
-        dissect_ldap_request_add(&a, msg_tree);
-        break;
-       case LDAP_REQ_DELETE:
-        dissect_ldap_request_delete(&a, msg_tree, start, opLen);
-        break;
-       case LDAP_REQ_MODRDN:
-        dissect_ldap_request_modifyrdn(&a, msg_tree, opLen);
-        break;
-       case LDAP_REQ_COMPARE:
-        dissect_ldap_request_compare(&a, msg_tree);
-        break;
-       case LDAP_REQ_MODIFY:
-        dissect_ldap_request_modify(&a, msg_tree);
-        break;
-       case LDAP_REQ_ABANDON:
-        dissect_ldap_request_abandon(&a, msg_tree, start, opLen);
-        break;
-       case LDAP_RES_BIND:
-        dissect_ldap_response_bind(&a, msg_tree);
-        break;
-       case LDAP_RES_SEARCH_ENTRY:
-        dissect_ldap_response_search_entry(&a, msg_tree);
-        break;
-       case LDAP_RES_SEARCH_RESULT:
-       case LDAP_RES_MODIFY:
-       case LDAP_RES_ADD:
-       case LDAP_RES_DELETE:
-       case LDAP_RES_MODRDN:
-       case LDAP_RES_COMPARE:
-        dissect_ldap_result(&a, msg_tree);
-        break;
+        if (check_col(pinfo->cinfo, COL_PROTOCOL))
+          col_set_str(pinfo->cinfo, COL_PROTOCOL, "LDAP");
+        if (check_col(pinfo->cinfo, COL_INFO))
+          col_clear(pinfo->cinfo, COL_INFO);
+      }
+
+      if (tree)
+      {
+        ti = proto_tree_add_item(tree, proto_ldap, next_tvb, 0, -1, FALSE);
+        ldap_tree = proto_item_add_subtree(ti, ett_ldap);
+
+        proto_tree_add_uint(ldap_tree, hf_ldap_sasl_buffer_length, tvb, 0, 4,
+                            sasl_length);
+      }
+
+      if (auth_info->auth_mech != NULL &&
+          strcmp(auth_info->auth_mech, "GSS-SPNEGO") == 0) {
+          /*
+           * This is GSS-API (using SPNEGO, but we should be done with
+           * the negotiation by now).
+           *
+           * Dissect the GSS_Wrap() token; it'll return the length of
+           * the token, from which we compute the offset in the tvbuff at
+           * which the plaintext data, i.e. the LDAP message, begins.
+           */
+          available_length = tvb_length_remaining(tvb, 4);
+          reported_length = tvb_reported_length_remaining(tvb, 4);
+          g_assert(available_length >= 0);
+          g_assert(reported_length >= 0);
+          if (available_length > reported_length)
+            available_length = reported_length;
+          if ((guint)available_length > sasl_length - 4)
+            available_length = sasl_length - 4;
+          if ((guint)reported_length > sasl_length - 4)
+            reported_length = sasl_length - 4;
+          next_tvb = tvb_new_subset(tvb, 4, available_length, reported_length);
+          if (tree)
+          {
+            gitem = proto_tree_add_text(ldap_tree, next_tvb, 0, -1, "GSS-API Token");
+            gtree = proto_item_add_subtree(gitem, ett_ldap_gssapi_token);
+          }
+          len = call_dissector(gssapi_wrap_handle, next_tvb, pinfo, gtree);
+          g_assert(len != 0);  /* GSS_Wrap() dissectors can't reject data */
+          if (gitem != NULL)
+              proto_item_set_len(gitem, len);
+
+          /*
+           * Now dissect the LDAP message.
+           */
+          dissect_ldap_message(tvb, 4 + len, pinfo, ldap_tree, first_time);
+      } else {
+        /*
+         * We don't know how to handle other authentication mechanisms
+         * yet, so just put in an entry for the SASL buffer.
+         */
+        proto_tree_add_text(ldap_tree, tvb, 4, -1, "SASL buffer");
       }
+      offset += message_data_len;
+    } else {
+      /*
+       * No, we're not doing a SASL security layer.  The frame begins
+       * with a "Sequence Of" header.
+       * Can we do reassembly?
+       */
+      if (ldap_desegment && pinfo->can_desegment) {
+        /*
+         * Yes - is the "Sequence Of" header split across segment
+         * boundaries?  We require at least 6 bytes for the header
+         * which allows for a 4 byte length (ASN.1 BER).
+         */
+        if (length_remaining < 6) {
+          pinfo->desegment_offset = offset;
+          pinfo->desegment_len = 6 - length_remaining;
+          return;
+        }
+      }
+
+      /*
+       * OK, try to read the "Sequence Of" header; this gets the total
+       * length of the LDAP message.
+       */
+      asn1_open(&a, tvb, offset);
+      ret = read_sequence(&a, &messageLength);
+      asn1_close(&a, &messageOffset);
+
+      if (ret == ASN1_ERR_NOERROR) {
+       /*
+        * Add the length of the "Sequence Of" header to the message
+        * length.
+        */
+       headerLength = messageOffset - offset;
+       messageLength += headerLength;
+        if (messageLength < headerLength) {
+          /*
+           * The message length was probably so large that the total length
+           * overflowed.
+           *
+           * Report this as an error.
+           */
+          show_reported_bounds_error(tvb, pinfo, tree);
+          return;
+        }
+      } else {
+       /*
+        * We couldn't parse the header; just make it the amount of data
+        * remaining in the tvbuff, so we'll give up on this segment
+        * after attempting to parse the message - there's nothing more
+        * we can do.  "dissect_ldap_message()" will display the error.
+        */
+       messageLength = length_remaining;
+      }
+
+      /*
+       * Can we do reassembly?
+       */
+      if (ldap_desegment && pinfo->can_desegment) {
+        /*
+         * Yes - is the message split across segment boundaries?
+         */
+        if (length_remaining < messageLength) {
+         /*
+          * Yes.  Tell the TCP dissector where the data for this message
+          * starts in the data it handed us, and how many more bytes
+          * we need, and return.
+          */
+         pinfo->desegment_offset = offset;
+         pinfo->desegment_len = messageLength - length_remaining;
+         return;
+        }
+      }
+
+      /*
+       * If this is the first PDU, set the Protocol column and clear the
+       * Info column.
+       */
+      if (first_time) {
+        if (check_col(pinfo->cinfo, COL_PROTOCOL))
+          col_set_str(pinfo->cinfo, COL_PROTOCOL, "LDAP");
+        if (check_col(pinfo->cinfo, COL_INFO))
+          col_clear(pinfo->cinfo, COL_INFO);
+      }
+
+      /*
+       * Construct a tvbuff containing the amount of the payload we have
+       * available.  Make its reported length the amount of data in the
+       * LDAP message.
+       *
+       * XXX - if reassembly isn't enabled. the subdissector will throw a
+       * BoundsError exception, rather than a ReportedBoundsError exception.
+       * We really want a tvbuff where the length is "length", the reported
+       * length is "plen", and the "if the snapshot length were infinite"
+       * length is the minimum of the reported length of the tvbuff handed
+       * to us and "plen", with a new type of exception thrown if the offset
+       * is within the reported length but beyond that third length, with
+       * that exception getting the "Unreassembled Packet" error.
+       */
+      length = length_remaining;
+      if (length > messageLength)
+        length = messageLength;
+      next_tvb = tvb_new_subset(tvb, offset, length, messageLength);
+
+      /*
+       * Now dissect the LDAP message.
+       */
+      if (tree) {
+        ti = proto_tree_add_item(tree, proto_ldap, next_tvb, 0, -1, FALSE);
+        ldap_tree = proto_item_add_subtree(ti, ett_ldap);
+      } else
+        ldap_tree = NULL;
+      dissect_ldap_message(next_tvb, 0, pinfo, ldap_tree, first_time);
+
+      offset += messageLength;
     }
+
+    first_time = FALSE;
   }
 }
 
+static void
+ldap_reinit(void)
+{
+  ldap_auth_info_t *auth_info;
+
+  /* Free up saved authentication mechanism strings */
+  for (auth_info = auth_info_items; auth_info != NULL;
+       auth_info = auth_info->next) {
+    if (auth_info->auth_mech != NULL)
+      g_free(auth_info->auth_mech);
+  }
+
+  if (ldap_auth_info_chunk != NULL)
+    g_mem_chunk_destroy(ldap_auth_info_chunk);
+
+  auth_info_items = NULL;
+
+  ldap_auth_info_chunk = g_mem_chunk_new("ldap_auth_info_chunk",
+               sizeof(ldap_auth_info_t),
+               ldap_auth_info_chunk_count * sizeof(ldap_auth_info_t),
+               G_ALLOC_ONLY);
+}
+
 void
 proto_register_ldap(void)
 {
@@ -1046,14 +2008,14 @@ proto_register_ldap(void)
     {LDAP_AUTH_SASL,      "SASL"},
     {0, NULL},
   };
-  
+
   static value_string search_scope[] = {
     {0x00, "Base"},
     {0x01, "Single"},
     {0x02, "Subtree"},
     {0x00, NULL},
   };
-    
+
   static value_string search_dereference[] = {
     {0x00, "Never"},
     {0x01, "Searching"},
@@ -1061,153 +2023,198 @@ proto_register_ldap(void)
     {0x03, "Always"},
     {0x00, NULL},
   };
-  
+
   static hf_register_info hf[] = {
+    { &hf_ldap_sasl_buffer_length,
+      { "SASL Buffer Length",  "ldap.sasl_buffer_length",
+       FT_UINT32, BASE_DEC, NULL, 0x0,
+       "SASL Buffer Length", HFILL }},
+
     { &hf_ldap_length,
       { "Length",              "ldap.length",
        FT_UINT32, BASE_DEC, NULL, 0x0,
-       "LDAP Length" }},
-         
+       "LDAP Length", HFILL }},
+
     { &hf_ldap_message_id,
       { "Message Id",          "ldap.message_id",
        FT_UINT32, BASE_DEC, NULL, 0x0,
-       "LDAP Message Id" }},
+       "LDAP Message Id", HFILL }},
     { &hf_ldap_message_type,
       { "Message Type",                "ldap.message_type",
        FT_UINT8, BASE_HEX, &msgTypes, 0x0,
-       "LDAP Message Type" }},
+       "LDAP Message Type", HFILL }},
     { &hf_ldap_message_length,
       { "Message Length",              "ldap.message_length",
        FT_UINT32, BASE_DEC, NULL, 0x0,
-       "LDAP Message Length" }},
+       "LDAP Message Length", HFILL }},
 
     { &hf_ldap_message_result,
       { "Result Code",         "ldap.result.code",
        FT_UINT8, BASE_HEX, result_codes, 0x0,
-       "LDAP Result Code" }},
+       "LDAP Result Code", HFILL }},
     { &hf_ldap_message_result_matcheddn,
       { "Matched DN",          "ldap.result.matcheddn",
        FT_STRING, BASE_NONE, NULL, 0x0,
-       "LDAP Result Matched DN" }},
+       "LDAP Result Matched DN", HFILL }},
     { &hf_ldap_message_result_errormsg,
       { "Error Message",               "ldap.result.errormsg",
        FT_STRING, BASE_NONE, NULL, 0x0,
-       "LDAP Result Error Message" }},
+       "LDAP Result Error Message", HFILL }},
     { &hf_ldap_message_result_referral,
       { "Referral",            "ldap.result.referral",
        FT_STRING, BASE_NONE, NULL, 0x0,
-       "LDAP Result Referral URL" }},
+       "LDAP Result Referral URL", HFILL }},
 
     { &hf_ldap_message_bind_version,
       { "Version",             "ldap.bind.version",
        FT_UINT32, BASE_DEC, NULL, 0x0,
-       "LDAP Bind Version" }},
+       "LDAP Bind Version", HFILL }},
     { &hf_ldap_message_bind_dn,
       { "DN",                  "ldap.bind.dn",
        FT_STRING, BASE_NONE, NULL, 0x0,
-       "LDAP Bind Distinguished Name" }},
+       "LDAP Bind Distinguished Name", HFILL }},
     { &hf_ldap_message_bind_auth,
       { "Auth Type",           "ldap.bind.auth_type",
        FT_UINT8, BASE_HEX, auth_types, 0x0,
-       "LDAP Bind Auth Type" }},
+       "LDAP Bind Auth Type", HFILL }},
     { &hf_ldap_message_bind_auth_password,
       { "Password",            "ldap.bind.password",
        FT_STRING, BASE_NONE, NULL, 0x0,
-       "LDAP Bind Password" }},
+       "LDAP Bind Password", HFILL }},
+    { &hf_ldap_message_bind_auth_mechanism,
+      { "Mechanism",           "ldap.bind.mechanism",
+       FT_STRING, BASE_NONE, NULL, 0x0,
+       "LDAP Bind Mechanism", HFILL }},
+    { &hf_ldap_message_bind_auth_credentials,
+      { "Credentials",         "ldap.bind.credentials",
+       FT_BYTES, BASE_NONE, NULL, 0x0,
+       "LDAP Bind Credentials", HFILL }},
+    { &hf_ldap_message_bind_server_credentials,
+      { "Server Credentials",  "ldap.bind.server_credentials",
+       FT_BYTES, BASE_NONE, NULL, 0x0,
+       "LDAP Bind Server Credentials", HFILL }},
 
     { &hf_ldap_message_search_base,
       { "Base DN",             "ldap.search.basedn",
        FT_STRING, BASE_NONE, NULL, 0x0,
-       "LDAP Search Base Distinguished Name" }},
+       "LDAP Search Base Distinguished Name", HFILL }},
     { &hf_ldap_message_search_scope,
       { "Scope",                       "ldap.search.scope",
        FT_UINT8, BASE_HEX, search_scope, 0x0,
-       "LDAP Search Scope" }},
+       "LDAP Search Scope", HFILL }},
     { &hf_ldap_message_search_deref,
       { "Dereference",         "ldap.search.dereference",
        FT_UINT8, BASE_HEX, search_dereference, 0x0,
-       "LDAP Search Dereference" }},
+       "LDAP Search Dereference", HFILL }},
     { &hf_ldap_message_search_sizeLimit,
       { "Size Limit",          "ldap.search.sizelimit",
        FT_UINT32, BASE_DEC, NULL, 0x0,
-       "LDAP Search Size Limit" }},
+       "LDAP Search Size Limit", HFILL }},
     { &hf_ldap_message_search_timeLimit,
       { "Time Limit",          "ldap.search.timelimit",
        FT_UINT32, BASE_DEC, NULL, 0x0,
-       "LDAP Search Time Limit" }},
+       "LDAP Search Time Limit", HFILL }},
     { &hf_ldap_message_search_typesOnly,
       { "Attributes Only",     "ldap.search.typesonly",
        FT_BOOLEAN, BASE_NONE, NULL, 0x0,
-       "LDAP Search Attributes Only" }},
+       "LDAP Search Attributes Only", HFILL }},
     { &hf_ldap_message_search_filter,
       { "Filter",              "ldap.search.filter",
        FT_STRING, BASE_NONE, NULL, 0x0,
-       "LDAP Search Filter" }},
+       "LDAP Search Filter", HFILL }},
     { &hf_ldap_message_dn,
       { "Distinguished Name",  "ldap.dn",
        FT_STRING, BASE_NONE, NULL, 0x0,
-       "LDAP Distinguished Name" }},
+       "LDAP Distinguished Name", HFILL }},
     { &hf_ldap_message_attribute,
       { "Attribute",           "ldap.attribute",
        FT_STRING, BASE_NONE, NULL, 0x0,
-       "LDAP Attribute" }},
+       "LDAP Attribute", HFILL }},
+    /*
+     * XXX - not all LDAP values are text strings; we'd need a file
+     * describing which values (by name) are text strings and which are
+     * binary.
+     *
+     * Some values that are, at least in Microsoft's schema, binary
+     * are:
+     *
+     * invocationId
+     * nTSecurityDescriptor
+     * objectGUID
+     */
     { &hf_ldap_message_value,
       { "Value",               "ldap.value",
        FT_STRING, BASE_NONE, NULL, 0x0,
-       "LDAP Value" }},
+       "LDAP Value", HFILL }},
 
     { &hf_ldap_message_modrdn_name,
       { "New Name",            "ldap.modrdn.name",
        FT_STRING, BASE_NONE, NULL, 0x0,
-       "LDAP New Name" }},
+       "LDAP New Name", HFILL }},
     { &hf_ldap_message_modrdn_delete,
       { "Delete Values",       "ldap.modrdn.delete",
        FT_BOOLEAN, BASE_NONE, NULL, 0x0,
-       "LDAP Modify RDN - Delete original values" }},
+       "LDAP Modify RDN - Delete original values", HFILL }},
     { &hf_ldap_message_modrdn_superior,
       { "New Location",                "ldap.modrdn.superior",
        FT_STRING, BASE_NONE, NULL, 0x0,
-       "LDAP Modify RDN - New Location" }},
+       "LDAP Modify RDN - New Location", HFILL }},
 
     { &hf_ldap_message_compare,
       { "Test",                "ldap.compare.test",
        FT_STRING, BASE_NONE, NULL, 0x0,
-       "LDAP Compare Test" }},
+       "LDAP Compare Test", HFILL }},
 
     { &hf_ldap_message_modify_add,
       { "Add",                 "ldap.modify.add",
        FT_STRING, BASE_NONE, NULL, 0x0,
-       "LDAP Add" }},
+       "LDAP Add", HFILL }},
     { &hf_ldap_message_modify_replace,
       { "Replace",             "ldap.modify.replace",
        FT_STRING, BASE_NONE, NULL, 0x0,
-       "LDAP Replace" }},
+       "LDAP Replace", HFILL }},
     { &hf_ldap_message_modify_delete,
       { "Delete",              "ldap.modify.delete",
        FT_STRING, BASE_NONE, NULL, 0x0,
-       "LDAP Delete" }},
+       "LDAP Delete", HFILL }},
 
     { &hf_ldap_message_abandon_msgid,
       { "Abandon Msg Id",      "ldap.abandon.msgid",
        FT_UINT32, BASE_DEC, NULL, 0x0,
-       "LDAP Abandon Msg Id" }},
+       "LDAP Abandon Msg Id", HFILL }},
   };
 
   static gint *ett[] = {
     &ett_ldap,
-    &ett_ldap_message,
+    &ett_ldap_gssapi_token,
     &ett_ldap_referrals,
     &ett_ldap_attribute
   };
+  module_t *ldap_module;
 
   proto_ldap = proto_register_protocol("Lightweight Directory Access Protocol",
                                       "LDAP", "ldap");
   proto_register_field_array(proto_ldap, hf, array_length(hf));
   proto_register_subtree_array(ett, array_length(ett));
+
+  ldap_module = prefs_register_protocol(proto_ldap, NULL);
+  prefs_register_bool_preference(ldap_module, "desegment_ldap_messages",
+    "Desegment all LDAP messages spanning multiple TCP segments",
+    "Whether the LDAP dissector should desegment all messages spanning multiple TCP segments",
+    &ldap_desegment);
+
+  register_init_routine(ldap_reinit);
 }
 
 void
 proto_reg_handoff_ldap(void)
 {
-  old_dissector_add("tcp.port", TCP_PORT_LDAP, dissect_ldap);
+  dissector_handle_t ldap_handle;
+
+  ldap_handle = create_dissector_handle(dissect_ldap, proto_ldap);
+  dissector_add("tcp.port", TCP_PORT_LDAP, ldap_handle);
+  dissector_add("udp.port", UDP_PORT_CLDAP, ldap_handle);
+
+  gssapi_handle = find_dissector("gssapi");
+  gssapi_wrap_handle = find_dissector("gssapi_verf");
 }