From Stefan Metzmacher:
[obnox/wireshark/wip.git] / epan / dissectors / packet-dcerpc.c
index 494d5944ae869de6c5584343bb0fa5c06bf28792..14b852147d7a38b381917bd458e96d449f84cb17 100644 (file)
@@ -5,8 +5,8 @@
  *
  * $Id$
  *
- * Ethereal - Network traffic analyzer
- * By Gerald Combs <gerald@ethereal.com>
+ * Wireshark - Network traffic analyzer
+ * By Gerald Combs <gerald@wireshark.org>
  * Copyright 1998 Gerald Combs
  *
  * This program is free software; you can redistribute it and/or
@@ -46,6 +46,7 @@
 #include <epan/dissectors/packet-frame.h>
 #include <epan/dissectors/packet-dcerpc-nt.h>
 #include <epan/expert.h>
+#include <epan/strutil.h>
 
 static int dcerpc_tap = -1;
 
@@ -109,7 +110,7 @@ static const value_string authn_protocol_vals[] = {
        { DCE_C_RPC_AUTHN_PROTOCOL_NTLMSSP, "NTLMSSP" },
        { DCE_C_RPC_AUTHN_PROTOCOL_GSS_SCHANNEL, "SCHANNEL SSP" },
        { DCE_C_RPC_AUTHN_PROTOCOL_GSS_KERBEROS, "Kerberos SSP" },
-       { DCE_C_RPC_AUTHN_PROTOCOL_DPA, 
+       { DCE_C_RPC_AUTHN_PROTOCOL_DPA,
                "Distributed Password Authentication SSP"},
        { DCE_C_RPC_AUTHN_PROTOCOL_MSN, "MSN SSP"},
        { DCE_C_RPC_AUTHN_PROTOCOL_DIGEST, "Digest SSP"},
@@ -222,6 +223,8 @@ static const value_string p_provider_reason_vals[] = {
 #define DEFAULT_CONTEXT_NOT_SUPPORTED  5 /* not used */
 #define USER_DATA_NOT_READABLE         6 /* not used */
 #define NO_PSAP_AVAILABLE              7 /* not used */
+#define AUTH_TYPE_NOT_RECOGNIZED       8
+#define INVALID_CHECKSUM                       9
 
 static const value_string reject_reason_vals[] = {
        { REASON_NOT_SPECIFIED,           "Reason not specified" },
@@ -232,6 +235,8 @@ static const value_string reject_reason_vals[] = {
        { DEFAULT_CONTEXT_NOT_SUPPORTED,  "Default context not supported" },
        { USER_DATA_NOT_READABLE,         "User data not readable" },
        { NO_PSAP_AVAILABLE,              "No PSAP available" },
+       { AUTH_TYPE_NOT_RECOGNIZED,       "Authentication type not recognized" },
+       { INVALID_CHECKSUM,               "Invalid checksum" },
        { 0,                              NULL }
 };
 
@@ -286,8 +291,8 @@ static const value_string reject_status_vals[] = {
        { 0x1c010014, "nca_server_too_busy" },
        { 0x1c010017, "nca_unsupported_type" },
        /* MS Windows specific values
-        * see: http://msdn.microsoft.com/library/default.asp?url=/library/en-us/debug/base/system_error_codes__1700-3999_.asp 
-        * and: http://msdn.microsoft.com/library/default.asp?url=/library/en-us/seccrypto/security/common_hresult_values.asp 
+        * see: http://msdn.microsoft.com/library/default.asp?url=/library/en-us/debug/base/system_error_codes__1700-3999_.asp
+        * and: http://msdn.microsoft.com/library/default.asp?url=/library/en-us/seccrypto/security/common_hresult_values.asp
         * and: http://www.megos.ch/support/doserrors.txt
         *
         * XXX - we might need a way to dynamically add entries here, as higher layer protocols use these values too,
@@ -295,10 +300,14 @@ static const value_string reject_status_vals[] = {
        { 0x80004001, "E_NOTIMPL" },
        { 0x80004003, "E_POINTER" },
        { 0x80004004, "E_ABORT" },
+       { 0x8000FFFF, "E_UNEXPECTED" },
        { 0x80010105, "RPC_E_SERVERFAULT" },
        { 0x80010108, "RPC_E_DISCONNECTED" },
        { 0x80010113, "RPC_E_INVALID_IPID" },
+       { 0x8001011F, "RPC_E_TIMEOUT" },
+       { 0x80020003, "DISP_E_MEMBERNOTFOUND" },
        { 0x80020006, "DISP_E_UNKNOWNNAME" },
+       { 0x8002000E, "DISP_E_BADPARAMCOUNT" },
        { 0x8004CB00, "CBA_E_MALFORMED" },
        { 0x8004CB01, "CBA_E_UNKNOWNOBJECT" },
        { 0x8004CB09, "CBA_E_INVALIDCOOKIE" },
@@ -306,8 +315,15 @@ static const value_string reject_status_vals[] = {
        { 0x8004CB0C, "CBA_E_QOSVALUEUNSUPPORTED" },
        { 0x8004CB0F, "CBA_E_NOTAPPLICABLE" },
        { 0x8004CB12, "CBA_E_LIMITVIOLATION" },
+       { 0x8004CB13, "CBA_E_QOSTYPENOTAPPLICABLE" },
+       { 0x8004CB18, "CBA_E_OUTOFPARTNERACCOS" },
+       { 0x8004CB1C, "CBA_E_FLAGUNSUPPORTED" },
+       { 0x8004CB23, "CBA_E_FRAMECOUNTUNSUPPORTED" },
+       { 0x8004CB25, "CBA_E_MODECHANGE" },
+       { 0x8007000E, "E_OUTOFMEMORY" },
        { 0x80070057, "E_INVALIDARG" },
        { 0x800706d1, "RPC_S_PROCNUM_OUT_OF_RANGE" },
+       { 0x80070776, "OR_INVALID_OXID" },
        { 0,          NULL }
 };
 
@@ -349,11 +365,14 @@ static int hf_dcerpc_cn_max_xmit = -1;
 static int hf_dcerpc_cn_max_recv = -1;
 static int hf_dcerpc_cn_assoc_group = -1;
 static int hf_dcerpc_cn_num_ctx_items = -1;
+static int hf_dcerpc_cn_ctx_item = -1;
 static int hf_dcerpc_cn_ctx_id = -1;
 static int hf_dcerpc_cn_num_trans_items = -1;
+static int hf_dcerpc_cn_bind_abstract_syntax = -1;
 static int hf_dcerpc_cn_bind_if_id = -1;
 static int hf_dcerpc_cn_bind_if_ver = -1;
 static int hf_dcerpc_cn_bind_if_ver_minor = -1;
+static int hf_dcerpc_cn_bind_trans_syntax = -1;
 static int hf_dcerpc_cn_bind_trans_id = -1;
 static int hf_dcerpc_cn_bind_trans_ver = -1;
 static int hf_dcerpc_cn_alloc_hint = -1;
@@ -442,6 +461,7 @@ static gint ett_dcerpc = -1;
 static gint ett_dcerpc_cn_flags = -1;
 static gint ett_dcerpc_cn_ctx = -1;
 static gint ett_dcerpc_cn_iface = -1;
+static gint ett_dcerpc_cn_trans_syntax = -1;
 static gint ett_dcerpc_drep = -1;
 static gint ett_dcerpc_dg_flags1 = -1;
 static gint ett_dcerpc_dg_flags2 = -1;
@@ -470,38 +490,6 @@ static const fragment_items dcerpc_frag_items = {
 /* list of hooks to be called when init_protocols is done */
 GHookList dcerpc_hooks_init_protos;
 
-#ifdef _WIN32
-int ResolveWin32UUID(e_uuid_t if_id, char *UUID_NAME, int UUID_NAME_MAX_LEN)
-{
-       char REG_UUID_NAME[MAX_PATH];
-       HKEY hKey = NULL;
-       DWORD UUID_MAX_SIZE = MAX_PATH;
-       char REG_UUID_STR[MAX_PATH];
-       
-       if(UUID_NAME_MAX_LEN < 2)
-               return 0;
-       REG_UUID_NAME[0] = '\0';
-       g_snprintf(REG_UUID_STR, MAX_PATH, "SOFTWARE\\Classes\\Interface\\{%08x-%04x-%04x-%02x%02x-%02x%02x%02x%02x%02x%02x}",
-                       if_id.Data1, if_id.Data2, if_id.Data3,
-                       if_id.Data4[0], if_id.Data4[1],
-                       if_id.Data4[2], if_id.Data4[3],
-                       if_id.Data4[4], if_id.Data4[5],
-                       if_id.Data4[6], if_id.Data4[7]);
-       if (RegOpenKeyEx(HKEY_LOCAL_MACHINE, (LPCSTR)REG_UUID_STR, 0, KEY_QUERY_VALUE, &hKey) == ERROR_SUCCESS)
-       {
-               if (RegQueryValueEx(hKey, NULL, NULL, NULL, (LPBYTE)REG_UUID_NAME, &UUID_MAX_SIZE) == ERROR_SUCCESS && UUID_MAX_SIZE <= MAX_PATH)
-                       {
-                       g_snprintf(UUID_NAME, UUID_NAME_MAX_LEN, "%s", REG_UUID_NAME);
-                       RegCloseKey(hKey);
-                       return strlen(REG_UUID_NAME);
-               }
-               RegCloseKey(hKey);
-       }
-       return 0; /* we didn't find anything anyhow. Please don't use the string! */
-       
-}
-#endif
-
 static dcerpc_info *
 get_next_di(void)
 {
@@ -519,19 +507,24 @@ get_next_di(void)
 static gboolean dcerpc_cn_desegment = TRUE;
 
 /* reassemble DCE/RPC fragments */
-/* reassembly of dcerpc fragments will not work for the case where ONE frame
+/* reassembly of cl dcerpc fragments will not work for the case where ONE frame
    might contain multiple dcerpc fragments for different PDUs.
    this case would be so unusual/weird so if you got captures like that:
        too bad
+
+   reassembly of co dcerpc fragments will not work for the case where TCP/SMB frames
+   are coming in out of sequence, but that will hurt in a lot of other places as well.
 */
-static gboolean dcerpc_reassemble = FALSE;
+static gboolean dcerpc_reassemble = TRUE;
+static GHashTable *dcerpc_co_fragment_table = NULL;
 static GHashTable *dcerpc_co_reassemble_table = NULL;
 static GHashTable *dcerpc_cl_reassemble_table = NULL;
 
 static void
 dcerpc_reassemble_init(void)
 {
-  fragment_table_init(&dcerpc_co_reassemble_table);
+  fragment_table_init(&dcerpc_co_fragment_table);
+  reassembled_table_init(&dcerpc_co_reassemble_table);
   dcerpc_fragment_table_init(&dcerpc_cl_reassemble_table);
 }
 
@@ -557,7 +550,7 @@ static dcerpc_auth_subdissector_fns *get_auth_subdissector_fns(
        for (i = 0; (data = g_slist_nth_data(dcerpc_auth_subdissector_list, i)); i++) {
                dcerpc_auth_subdissector *asd = (dcerpc_auth_subdissector *)data;
 
-               if (asd->auth_level == auth_level && 
+               if (asd->auth_level == auth_level &&
                    asd->auth_type == auth_type)
                        return &asd->auth_fns;
        }
@@ -585,9 +578,9 @@ void register_dcerpc_auth_subdissector(guint8 auth_level, guint8 auth_type,
 /* Hand off verifier data to a registered dissector */
 
 static void dissect_auth_verf(tvbuff_t *auth_tvb, packet_info *pinfo,
-                             proto_tree *tree, 
+                             proto_tree *tree,
                              dcerpc_auth_subdissector_fns *auth_fns,
-                             e_dce_cn_common_hdr_t *hdr, 
+                             e_dce_cn_common_hdr_t *hdr,
                              dcerpc_auth_info *auth_info)
 {
        dcerpc_dissect_fnct_t *volatile fn = NULL;
@@ -611,7 +604,7 @@ static void dissect_auth_verf(tvbuff_t *auth_tvb, packet_info *pinfo,
                fn = auth_fns->resp_verf_fn;
                break;
 
-               /* Don't know how to handle authentication data in this 
+               /* Don't know how to handle authentication data in this
                   pdu type. */
 
        default:
@@ -625,8 +618,8 @@ static void dissect_auth_verf(tvbuff_t *auth_tvb, packet_info *pinfo,
        else {
                tvb_ensure_bytes_exist(auth_tvb, 0, hdr->auth_len);
                proto_tree_add_text(tree, auth_tvb, 0, hdr->auth_len,
-                                   "%s Verifier", 
-                                   val_to_str(auth_info->auth_type, 
+                                   "%s Verifier",
+                                   val_to_str(auth_info->auth_type,
                                               authn_protocol_vals,
                                               "Unknown (%u)"));
        }
@@ -634,11 +627,11 @@ static void dissect_auth_verf(tvbuff_t *auth_tvb, packet_info *pinfo,
 
 /* Hand off payload data to a registered dissector */
 
-static tvbuff_t *decode_encrypted_data(tvbuff_t *data_tvb, 
+static tvbuff_t *decode_encrypted_data(tvbuff_t *data_tvb,
                                       tvbuff_t *auth_tvb,
                                       packet_info *pinfo,
                                       dcerpc_auth_subdissector_fns *auth_fns,
-                                      gboolean is_request, 
+                                      gboolean is_request,
                                       dcerpc_auth_info *auth_info)
 {
        dcerpc_decode_data_fnct_t *fn;
@@ -701,32 +694,13 @@ dcerpc_init_uuid (int proto, int ett, e_uuid_t *uuid, guint16 ver,
 
     hf_info = proto_registrar_get_nth(opnum_hf);
     hf_info->strings = value_string_from_subdissectors(procs);
-}
-
-
-/* try to get registered name for this uuid */
-const gchar *dcerpc_get_uuid_name(e_uuid_t *uuid, guint16 ver)
-{
-    dcerpc_uuid_key key;
-    dcerpc_uuid_value *sub_proto;
 
-
-       /* try to get registered uuid "name" of if_id */
-       key.uuid = *uuid;
-       key.ver = ver;
-
-       if ((sub_proto = g_hash_table_lookup (dcerpc_uuids, &key)) != NULL
-                && proto_is_protocol_enabled(sub_proto->proto)) {
-
-               return sub_proto->name;
-       }
-
-       return NULL;
+    /* add this GUID to the global name resolving */
+    guids_add_uuid(uuid, proto_get_protocol_short_name (value->proto));
 }
 
-
 /* Function to find the name of a registered protocol
- * or NULL if the protocol/version is not known to ethereal.
+ * or NULL if the protocol/version is not known to wireshark.
  */
 const char *
 dcerpc_get_proto_name(e_uuid_t *uuid, guint16 ver)
@@ -743,7 +717,7 @@ dcerpc_get_proto_name(e_uuid_t *uuid, guint16 ver)
 }
 
 /* Function to find the opnum hf-field of a registered protocol
- * or -1 if the protocol/version is not known to ethereal.
+ * or -1 if the protocol/version is not known to wireshark.
  */
 int
 dcerpc_get_proto_hf_opnum(e_uuid_t *uuid, guint16 ver)
@@ -788,7 +762,7 @@ value_string *value_string_from_subdissectors(dcerpc_sub_dissector *sd)
 }
 
 /* Function to find the subdissector table of a registered protocol
- * or NULL if the protocol/version is not known to ethereal.
+ * or NULL if the protocol/version is not known to wireshark.
  */
 dcerpc_sub_dissector *
 dcerpc_get_proto_sub_dissector(e_uuid_t *uuid, guint16 ver)
@@ -1013,7 +987,7 @@ dissect_dcerpc_time_t (tvbuff_t *tvb, gint offset, packet_info *pinfo _U_,
     if (tree) {
         if(data==0xffffffff){
             /* special case,   no time specified */
-            proto_tree_add_time_format(tree, hfindex, tvb, offset, 4, &tv, "%s: No time specified", proto_registrar_get_nth(hfindex)->name);
+            proto_tree_add_time_format_value(tree, hfindex, tvb, offset, 4, &tv, "No time specified");
         } else {
             proto_tree_add_time (tree, hfindex, tvb, offset, 4, &tv);
         }
@@ -1046,7 +1020,7 @@ dissect_dcerpc_uint64 (tvbuff_t *tvb, gint offset, packet_info *pinfo _U_,
 
 int
 dissect_dcerpc_float(tvbuff_t *tvb, gint offset, packet_info *pinfo _U_,
-                    proto_tree *tree, guint8 *drep, 
+                    proto_tree *tree, guint8 *drep,
                     int hfindex, gfloat *pdata)
 {
        gfloat data;
@@ -1080,7 +1054,7 @@ dissect_dcerpc_float(tvbuff_t *tvb, gint offset, packet_info *pinfo _U_,
 
 int
 dissect_dcerpc_double(tvbuff_t *tvb, gint offset, packet_info *pinfo _U_,
-                    proto_tree *tree, guint8 *drep, 
+                    proto_tree *tree, guint8 *drep,
                     int hfindex, gdouble *pdata)
 {
     gdouble data;
@@ -1114,57 +1088,19 @@ dissect_dcerpc_double(tvbuff_t *tvb, gint offset, packet_info *pinfo _U_,
 
 int
 dissect_dcerpc_uuid_t (tvbuff_t *tvb, gint offset, packet_info *pinfo _U_,
-                    proto_tree *tree, char *drep,
+                    proto_tree *tree, guint8 *drep,
                     int hfindex, e_uuid_t *pdata)
 {
     e_uuid_t uuid;
-       header_field_info* hfi;
-#if 0
-       gchar *uuid_name;
-#endif
 
 
-    dcerpc_tvb_get_uuid (tvb, offset, drep, &uuid);
+    if (drep[0] & 0x10) {
+        tvb_get_letohguid (tvb, offset, (e_guid_t *) &uuid);
+    } else {
+        tvb_get_ntohguid (tvb, offset, (e_guid_t *) &uuid);
+    }
     if (tree) {
-               /* get name of protocol field to prepend it later */
-               hfi = proto_registrar_get_nth(hfindex);
-
-#if 0
-        /* XXX - get the name won't work correct, as we don't know the version of this uuid (if it has one) */
-               /* look for a registered uuid name */
-               uuid_name = dcerpc_get_uuid_name(&uuid, 0);
-
-               if (uuid_name) {
-                       /* we know the name of this uuid */
-                       proto_tree_add_string_format (tree, hfindex, tvb, offset, 16, "",
-                                      "%s: %s (%08x-%04x-%04x-%02x%02x-%02x%02x%02x%02x%02x%02x)",
-                                                                         hfi->name, uuid_name,
-                                      uuid.Data1, uuid.Data2, uuid.Data3,
-                                      uuid.Data4[0], uuid.Data4[1],
-                                      uuid.Data4[2], uuid.Data4[3],
-                                      uuid.Data4[4], uuid.Data4[5],
-                                      uuid.Data4[6], uuid.Data4[7]);
-               } else {
-#endif
-                       /* GUID have changed from FT_STRING to FT_GUID
-                          but we havent changed all dissectors yet.
-                        */
-                       if(hfi->type==FT_GUID){
-                               proto_tree_add_item(tree, hfindex, tvb, offset, 16, (drep[0] & 0x10));
-                       } else {
-                               /* we don't know the name of this uuid */
-                               proto_tree_add_string_format (tree, hfindex, tvb, offset, 16, "",
-                                     "%s: %08x-%04x-%04x-%02x%02x-%02x%02x%02x%02x%02x%02x",
-                                     hfi->name,
-                                      uuid.Data1, uuid.Data2, uuid.Data3,
-                                      uuid.Data4[0], uuid.Data4[1],
-                                      uuid.Data4[2], uuid.Data4[3],
-                                      uuid.Data4[4], uuid.Data4[5],
-                                      uuid.Data4[6], uuid.Data4[7]);
-                       }
-#if 0
-               }
-#endif
+               proto_tree_add_guid(tree, hfindex, tvb, offset, 16, (e_guid_t *) &uuid);
     }
     if (pdata) {
         *pdata = uuid;
@@ -1199,18 +1135,14 @@ dcerpc_tvb_get_ntohl (tvbuff_t *tvb, gint offset, guint8 *drep)
 void
 dcerpc_tvb_get_uuid (tvbuff_t *tvb, gint offset, guint8 *drep, e_uuid_t *uuid)
 {
-    unsigned int i;
-    uuid->Data1 = dcerpc_tvb_get_ntohl (tvb, offset, drep);
-    uuid->Data2 = dcerpc_tvb_get_ntohs (tvb, offset+4, drep);
-    uuid->Data3 = dcerpc_tvb_get_ntohs (tvb, offset+6, drep);
-
-    for (i=0; i<sizeof (uuid->Data4); i++) {
-        uuid->Data4[i] = tvb_get_guint8 (tvb, offset+8+i);
+    if (drep[0] & 0x10) {
+        tvb_get_letohguid (tvb, offset, (e_guid_t *) uuid);
+    } else {
+        tvb_get_ntohguid (tvb, offset, (e_guid_t *) uuid);
     }
 }
 
 
-
 /* NDR arrays */
 /* function to dissect a unidimensional conformant array */
 int
@@ -1278,7 +1210,10 @@ dissect_ndr_ucvarray(tvbuff_t *tvb, gint offset, packet_info *pinfo,
 
                /* real run, dissect the elements */
                for(i=0;i<di->array_actual_count;i++){
+                       old_offset = offset;
                        offset = (*fnct)(tvb, offset, pinfo, tree, drep);
+                        if (offset <= old_offset)
+                                THROW(ReportedBoundsError);
                }
        }
 
@@ -1337,7 +1272,7 @@ dissect_ndr_uvarray(tvbuff_t *tvb, gint offset, packet_info *pinfo,
    that are conformant and varying - they're stored like conformant
    varying arrays of bytes.  */
 int
-dissect_ndr_byte_array(tvbuff_t *tvb, int offset, packet_info *pinfo, 
+dissect_ndr_byte_array(tvbuff_t *tvb, int offset, packet_info *pinfo,
                             proto_tree *tree, guint8 *drep)
 {
     dcerpc_info *di;
@@ -1380,7 +1315,7 @@ dissect_ndr_byte_array(tvbuff_t *tvb, int offset, packet_info *pinfo,
    XXX - does this need to do all the conformant array stuff that
    "dissect_ndr_ucvarray()" does?  */
 int
-dissect_ndr_cvstring(tvbuff_t *tvb, int offset, packet_info *pinfo, 
+dissect_ndr_cvstring(tvbuff_t *tvb, int offset, packet_info *pinfo,
                     proto_tree *tree, guint8 *drep, int size_is,
                     int hfindex, gboolean add_subtree, char **data)
 {
@@ -1465,7 +1400,7 @@ dissect_ndr_cvstring(tvbuff_t *tvb, int offset, packet_info *pinfo,
            *data = s;
     else
            g_free(s);
-    
+
     offset += buffer_len;
 
     proto_item_set_end(string_item, tvb, offset);
@@ -1483,7 +1418,7 @@ dissect_ndr_cvstring(tvbuff_t *tvb, int offset, packet_info *pinfo,
    such as C.  Is this ever used for purely counted strings?
    (Not that it matters if it is.) */
 int
-dissect_ndr_char_cvstring(tvbuff_t *tvb, int offset, packet_info *pinfo, 
+dissect_ndr_char_cvstring(tvbuff_t *tvb, int offset, packet_info *pinfo,
                        proto_tree *tree, guint8 *drep)
 {
     dcerpc_info *di;
@@ -1504,7 +1439,7 @@ dissect_ndr_char_cvstring(tvbuff_t *tvb, int offset, packet_info *pinfo,
    such as C.  Is this ever used for purely counted strings?
    (Not that it matters if it is.) */
 int
-dissect_ndr_wchar_cvstring(tvbuff_t *tvb, int offset, packet_info *pinfo, 
+dissect_ndr_wchar_cvstring(tvbuff_t *tvb, int offset, packet_info *pinfo,
                        proto_tree *tree, guint8 *drep)
 {
     dcerpc_info *di;
@@ -1520,7 +1455,7 @@ dissect_ndr_wchar_cvstring(tvbuff_t *tvb, int offset, packet_info *pinfo,
    the elements are assumed to be characters or wide characters.
 */
 int
-dissect_ndr_vstring(tvbuff_t *tvb, int offset, packet_info *pinfo, 
+dissect_ndr_vstring(tvbuff_t *tvb, int offset, packet_info *pinfo,
                     proto_tree *tree, guint8 *drep, int size_is,
                     int hfindex, gboolean add_subtree, char **data)
 {
@@ -1601,7 +1536,7 @@ dissect_ndr_vstring(tvbuff_t *tvb, int offset, packet_info *pinfo,
            *data = s;
     else
            g_free(s);
-    
+
     offset += buffer_len;
 
     proto_item_set_end(string_item, tvb, offset);
@@ -1618,7 +1553,7 @@ dissect_ndr_vstring(tvbuff_t *tvb, int offset, packet_info *pinfo,
    such as C.  Is this ever used for purely counted strings?
    (Not that it matters if it is.) */
 int
-dissect_ndr_char_vstring(tvbuff_t *tvb, int offset, packet_info *pinfo, 
+dissect_ndr_char_vstring(tvbuff_t *tvb, int offset, packet_info *pinfo,
                        proto_tree *tree, guint8 *drep)
 {
     dcerpc_info *di;
@@ -1639,7 +1574,7 @@ dissect_ndr_char_vstring(tvbuff_t *tvb, int offset, packet_info *pinfo,
    such as C.  Is this ever used for purely counted strings?
    (Not that it matters if it is.) */
 int
-dissect_ndr_wchar_vstring(tvbuff_t *tvb, int offset, packet_info *pinfo, 
+dissect_ndr_wchar_vstring(tvbuff_t *tvb, int offset, packet_info *pinfo,
                        proto_tree *tree, guint8 *drep)
 {
     dcerpc_info *di;
@@ -1794,7 +1729,7 @@ dissect_deferred_pointers(packet_info *pinfo, tvbuff_t *tvb, int offset, guint8
 
 static void
 add_pointer_to_list(packet_info *pinfo, proto_tree *tree, proto_item *item,
-                   dcerpc_dissect_fnct_t *fnct, guint32 id, int hf_index, 
+                   dcerpc_dissect_fnct_t *fnct, guint32 id, int hf_index,
                    dcerpc_callback_fnct_t *callback, void *callback_args)
 {
        ndr_pointer_data_t *npd;
@@ -1884,7 +1819,7 @@ find_pointer_index(guint32 id)
 int
 dissect_ndr_pointer_cb(tvbuff_t *tvb, gint offset, packet_info *pinfo,
                    proto_tree *tree, guint8 *drep, dcerpc_dissect_fnct_t *fnct,
-                   int type, const char *text, int hf_index, 
+                   int type, const char *text, int hf_index,
                    dcerpc_callback_fnct_t *callback, void *callback_args)
 {
        dcerpc_info *di;
@@ -1910,7 +1845,7 @@ dissect_ndr_pointer_cb(tvbuff_t *tvb, gint offset, packet_info *pinfo,
                        "%s", text);
                tr=proto_item_add_subtree(item,ett_dcerpc_pointer_data);
 
-               add_pointer_to_list(pinfo, tr, item, fnct, 0xffffffff, 
+               add_pointer_to_list(pinfo, tr, item, fnct, 0xffffffff,
                                    hf_index, callback, callback_args);
                goto after_ref_id;
        }
@@ -1948,7 +1883,7 @@ dissect_ndr_pointer_cb(tvbuff_t *tvb, gint offset, packet_info *pinfo,
                        "%s", text);
                tr=proto_item_add_subtree(item,ett_dcerpc_pointer_data);
                proto_tree_add_uint(tr, hf_dcerpc_referent_id, tvb, offset-4, 4, id);
-               add_pointer_to_list(pinfo, tr, item, fnct, id, hf_index, 
+               add_pointer_to_list(pinfo, tr, item, fnct, id, hf_index,
                                    callback, callback_args);
                goto after_ref_id;
        }
@@ -1974,7 +1909,7 @@ dissect_ndr_pointer_cb(tvbuff_t *tvb, gint offset, packet_info *pinfo,
                        "%s", text);
                tr=proto_item_add_subtree(item,ett_dcerpc_pointer_data);
                proto_tree_add_uint(tr, hf_dcerpc_referent_id, tvb, offset-4, 4, id);
-               add_pointer_to_list(pinfo, tr, item, fnct, 0xffffffff, 
+               add_pointer_to_list(pinfo, tr, item, fnct, 0xffffffff,
                                    hf_index, callback, callback_args);
                goto after_ref_id;
        }
@@ -1994,7 +1929,7 @@ dissect_ndr_pointer_cb(tvbuff_t *tvb, gint offset, packet_info *pinfo,
                        "%s",text);
                tr=proto_item_add_subtree(item,ett_dcerpc_pointer_data);
                proto_tree_add_uint(tr, hf_dcerpc_referent_id, tvb, offset-4, 4, id);
-               add_pointer_to_list(pinfo, tr, item, fnct, 0xffffffff, 
+               add_pointer_to_list(pinfo, tr, item, fnct, 0xffffffff,
                                    hf_index, callback, callback_args);
                goto after_ref_id;
        }
@@ -2021,7 +1956,7 @@ dissect_ndr_pointer_cb(tvbuff_t *tvb, gint offset, packet_info *pinfo,
                        "%s",text);
                tr=proto_item_add_subtree(item,ett_dcerpc_pointer_data);
                proto_tree_add_uint(tr, hf_dcerpc_referent_id, tvb, offset-4, 4, id);
-               add_pointer_to_list(pinfo, tr, item, fnct, 0xffffffff, 
+               add_pointer_to_list(pinfo, tr, item, fnct, 0xffffffff,
                                    hf_index, callback, callback_args);
                goto after_ref_id;
        }
@@ -2059,7 +1994,7 @@ dissect_ndr_pointer_cb(tvbuff_t *tvb, gint offset, packet_info *pinfo,
                        "%s", text);
                tr=proto_item_add_subtree(item,ett_dcerpc_pointer_data);
                proto_tree_add_uint(tr, hf_dcerpc_referent_id, tvb, offset-4, 4, id);
-               add_pointer_to_list(pinfo, tr, item, fnct, id, hf_index, 
+               add_pointer_to_list(pinfo, tr, item, fnct, id, hf_index,
                                    callback, callback_args);
                goto after_ref_id;
        }
@@ -2192,14 +2127,12 @@ dcerpc_try_handoff (packet_info *pinfo, proto_tree *tree,
     dcerpc_dissect_fnct_t *volatile sub_dissect;
     const char *volatile saved_proto;
     void *volatile saved_private_data;
-    guint length, reported_length;
+    guint length = 0, reported_length = 0;
     tvbuff_t *volatile stub_tvb;
     volatile guint auth_pad_len;
     volatile int auth_pad_offset;
-#ifdef _WIN32
-    char UUID_NAME[MAX_PATH];
-#endif
-    proto_item *sub_item;
+    proto_item *sub_item=NULL;
+    proto_item *pi;
 
     key.uuid = info->call_data->uuid;
     key.ver = info->call_data->ver;
@@ -2215,20 +2148,8 @@ dcerpc_try_handoff (packet_info *pinfo, proto_tree *tree,
        proto_tree_add_boolean_hidden(dcerpc_tree, hf_dcerpc_unknown_if_id,
                                          tvb, offset, 0, TRUE);
        if (check_col (pinfo->cinfo, COL_INFO)) {
-#ifdef _WIN32
-               if(ResolveWin32UUID(info->call_data->uuid, UUID_NAME, MAX_PATH))
-                       col_append_fstr (pinfo->cinfo, COL_INFO, " [%s] UUID: %08x-%04x-%04x-%02x%02x-%02x%02x%02x%02x%02x%02x rpcver: %u",
-                               UUID_NAME, info->call_data->uuid.Data1, info->call_data->uuid.Data2, info->call_data->uuid.Data3, info->call_data->uuid.Data4[0],
-                               info->call_data->uuid.Data4[1], info->call_data->uuid.Data4[2], info->call_data->uuid.Data4[3],
-                               info->call_data->uuid.Data4[4], info->call_data->uuid.Data4[5], info->call_data->uuid.Data4[6],
-                               info->call_data->uuid.Data4[7], info->call_data->ver);
-else
-#endif
-               col_append_fstr (pinfo->cinfo, COL_INFO, " UNKUUID: %08x-%04x-%04x-%02x%02x-%02x%02x%02x%02x%02x%02x rpcver: %u",
-                       info->call_data->uuid.Data1, info->call_data->uuid.Data2, info->call_data->uuid.Data3, info->call_data->uuid.Data4[0],
-                       info->call_data->uuid.Data4[1], info->call_data->uuid.Data4[2], info->call_data->uuid.Data4[3],
-                       info->call_data->uuid.Data4[4], info->call_data->uuid.Data4[5], info->call_data->uuid.Data4[6],
-                       info->call_data->uuid.Data4[7], info->call_data->ver);
+               col_append_fstr (pinfo->cinfo, COL_INFO, " %s V%u",
+                       guids_resolve_uuid_to_str(&info->call_data->uuid), info->call_data->ver);
        }
 
         if (decrypted_tvb != NULL) {
@@ -2258,9 +2179,13 @@ else
                       name, (info->ptype == PDU_REQ) ? "request" : "response");
     }
 
+    sub_dissect = (info->ptype == PDU_REQ) ?
+           proc->dissect_rqst : proc->dissect_resp;
+
     if (tree) {
-        sub_item = proto_tree_add_item (tree, sub_proto->proto_id, tvb, 0,
-                                        -1, FALSE);
+        sub_item = proto_tree_add_item (tree, sub_proto->proto_id,
+                                       (decrypted_tvb != NULL)?decrypted_tvb:tvb,
+                                       0, -1, FALSE);
 
         if (sub_item) {
             sub_tree = proto_item_add_subtree (sub_item, sub_proto->ett);
@@ -2271,40 +2196,48 @@ else
          * Put the operation number into the tree along with
          * the operation's name.
          */
-
        if (sub_proto->opnum_hf != -1)
-            proto_tree_add_uint_format(sub_tree, sub_proto->opnum_hf,
-                                       tvb, 0, 0, info->call_data->opnum,
-                                       "Operation: %s (%u)",
-                                       name, info->call_data->opnum);
+                proto_tree_add_uint_format(sub_tree, sub_proto->opnum_hf,
+                                           tvb, 0, 0, info->call_data->opnum,
+                                           "Operation: %s (%u)",
+                                           name, info->call_data->opnum);
        else
-            proto_tree_add_uint_format(sub_tree, hf_dcerpc_op, tvb,
-                                       0, 0, info->call_data->opnum,
-                                       "Operation: %s (%u)",
-                                       name, info->call_data->opnum);
-    }
-
-    sub_dissect = (info->ptype == PDU_REQ) ? 
-           proc->dissect_rqst : proc->dissect_resp;
+                proto_tree_add_uint_format(sub_tree, hf_dcerpc_op, tvb,
+                                           0, 0, info->call_data->opnum,
+                                           "Operation: %s (%u)",
+                                           name, info->call_data->opnum);
+
+        if(info->ptype == PDU_REQ && info->call_data->rep_frame!=0) {
+            pi = proto_tree_add_uint(sub_tree, hf_dcerpc_response_in,
+                                     tvb, 0, 0, info->call_data->rep_frame);
+            PROTO_ITEM_SET_GENERATED(pi);
+       }
+        if(info->ptype == PDU_RESP && info->call_data->req_frame!=0) {
+            pi = proto_tree_add_uint(sub_tree, hf_dcerpc_request_in,
+                                     tvb, 0, 0, info->call_data->req_frame);
+            PROTO_ITEM_SET_GENERATED(pi);
+       }
+    } /* tree */
 
     if (decrypted_tvb != NULL) {
         /* Either there was no encryption or we successfully decrypted
-           the entrypted payload. */
+           the encrypted payload. */
         if (sub_dissect) {
             /* We have a subdissector - call it. */
             saved_proto = pinfo->current_proto;
             saved_private_data = pinfo->private_data;
             pinfo->current_proto = sub_proto->name;
             pinfo->private_data = (void *)info;
-           
+
             init_ndr_pointer_list(pinfo);
 
+            length = tvb_length(decrypted_tvb);
+            reported_length = tvb_reported_length(decrypted_tvb);
+
             /*
              * Remove the authentication padding from the stub data.
              */
             if (auth_info != NULL && auth_info->auth_pad_len != 0) {
-                length = tvb_length(decrypted_tvb);
-                reported_length = tvb_reported_length(decrypted_tvb);
                 if (reported_length >= auth_info->auth_pad_len) {
                     /*
                      * OK, the padding length isn't so big that it
@@ -2322,7 +2255,7 @@ else
                     if (length > reported_length)
                         length = reported_length;
 
-                    stub_tvb = tvb_new_subset(tvb, 0, length, reported_length);
+                    stub_tvb = tvb_new_subset(decrypted_tvb, 0, length, reported_length);
                     auth_pad_len = auth_info->auth_pad_len;
                     auth_pad_offset = reported_length;
                 } else {
@@ -2335,6 +2268,8 @@ else
                     stub_tvb = NULL;
                     auth_pad_len = reported_length;
                     auth_pad_offset = 0;
+                    length = 0;
+                    reported_length = 0;
                 }
             } else {
                 /*
@@ -2345,6 +2280,10 @@ else
                 auth_pad_offset = 0;
             }
 
+            if (sub_item) {
+               proto_item_set_len(sub_item, length);
+            }
+
             if (stub_tvb != NULL) {
                 /*
                  * Catch all exceptions other than BoundsError, so that even
@@ -2356,25 +2295,31 @@ else
                  * dissect; just re-throw that exception.
                  */
                 TRY {
-                    offset = sub_dissect (decrypted_tvb, 0, pinfo, sub_tree,
+                    int remaining;
+
+                    offset = sub_dissect (stub_tvb, 0, pinfo, sub_tree,
                                           drep);
-                    if(tree) {
-                        proto_item_set_len(sub_item, offset);
-                    }
 
                     /* If we have a subdissector and it didn't dissect all
                        data in the tvb, make a note of it. */
-                    /* XXX - don't do this, as this could be just another RPC Req./Resp. in this PDU */
-                    /*if (tvb_reported_length_remaining(stub_tvb, offset) > 0) {
+                    remaining = tvb_reported_length_remaining(stub_tvb, offset);
+                    if (remaining > 0) {
+                        proto_tree_add_text(sub_tree, stub_tvb, offset,
+                                            remaining,
+                                            "[Long frame (%d byte%s)]",
+                                            remaining,
+                                            plurality(remaining, "", "s"));
                         if (check_col(pinfo->cinfo, COL_INFO))
                             col_append_fstr(pinfo->cinfo, COL_INFO,
-                                            "[Long frame (%d bytes)]",
-                                            tvb_reported_length_remaining(stub_tvb, offset));
-                    }*/
+                                            "[Long frame (%d byte%s)]",
+                                            remaining,
+                                            plurality(remaining, "", "s"));
+
+                    }
                 } CATCH(BoundsError) {
                     RETHROW;
                 } CATCH_ALL {
-                    show_exception(decrypted_tvb, pinfo, tree, EXCEPT_CODE, GET_MESSAGE);
+                    show_exception(stub_tvb, pinfo, tree, EXCEPT_CODE, GET_MESSAGE);
                 } ENDTRY;
             }
 
@@ -2406,7 +2351,7 @@ else
 }
 
 static int
-dissect_dcerpc_verifier (tvbuff_t *tvb, packet_info *pinfo, 
+dissect_dcerpc_verifier (tvbuff_t *tvb, packet_info *pinfo,
                         proto_tree *dcerpc_tree, e_dce_cn_common_hdr_t *hdr,
                         dcerpc_auth_info *auth_info)
 {
@@ -2455,14 +2400,14 @@ dissect_dcerpc_cn_auth (tvbuff_t *tvb, int stub_offset, packet_info *pinfo,
     volatile int offset;
 
     /*
-     * Initially set auth_level and auth_type to zero to indicate that we 
+     * Initially set auth_level and auth_type to zero to indicate that we
      * haven't yet seen any authentication level information.
      */
     auth_info->auth_level = 0;
     auth_info->auth_type = 0;
     auth_info->auth_size = 0;
     auth_info->auth_pad_len = 0;
-    
+
     /*
      * The authentication information is at the *end* of the PDU; in
      * request and response PDUs, the request and response stub data
@@ -2499,14 +2444,14 @@ dissect_dcerpc_cn_auth (tvbuff_t *tvb, int stub_offset, packet_info *pinfo,
              */
             TRY {
                 offset = dissect_dcerpc_uint8 (tvb, offset, pinfo, dcerpc_tree, hdr->drep,
-                                               hf_dcerpc_auth_type, 
+                                               hf_dcerpc_auth_type,
                                                &auth_info->auth_type);
                 offset = dissect_dcerpc_uint8 (tvb, offset, pinfo, dcerpc_tree, hdr->drep,
-                                               hf_dcerpc_auth_level, 
+                                               hf_dcerpc_auth_level,
                                                &auth_info->auth_level);
 
                 offset = dissect_dcerpc_uint8 (tvb, offset, pinfo, dcerpc_tree, hdr->drep,
-                                               hf_dcerpc_auth_pad_len, 
+                                               hf_dcerpc_auth_pad_len,
                                                &auth_info->auth_pad_len);
                 offset = dissect_dcerpc_uint8 (tvb, offset, pinfo, dcerpc_tree, hdr->drep,
                                                hf_dcerpc_auth_rsrvd, NULL);
@@ -2520,19 +2465,20 @@ dissect_dcerpc_cn_auth (tvbuff_t *tvb, int stub_offset, packet_info *pinfo,
                     tvbuff_t *auth_tvb;
                     dcerpc_auth_subdissector_fns *auth_fns;
 
-                    auth_tvb = tvb_new_subset(tvb, offset, hdr->auth_len,
-                                              hdr->auth_len);
+                    auth_tvb = tvb_new_subset(tvb, offset,
+                                   MIN(hdr->auth_len,tvb_length_remaining(tvb, offset)),
+                                   hdr->auth_len);
 
                     if ((auth_fns = get_auth_subdissector_fns(auth_info->auth_level,
                                                               auth_info->auth_type)))
-                        dissect_auth_verf(auth_tvb, pinfo, dcerpc_tree, auth_fns, 
+                        dissect_auth_verf(auth_tvb, pinfo, dcerpc_tree, auth_fns,
                                           hdr, auth_info);
                     else
                         proto_tree_add_text (dcerpc_tree, tvb, offset, hdr->auth_len,
                                              "Auth Credentials");
                 }
-       
-                /* Compute the size of the auth block.  Note that this should not 
+
+                /* Compute the size of the auth block.  Note that this should not
                    include auth padding, since when NTLMSSP encryption is used, the
                    padding is actually inside the encrypted stub */
                    auth_info->auth_size = hdr->auth_len + 8;
@@ -2546,7 +2492,7 @@ dissect_dcerpc_cn_auth (tvbuff_t *tvb, int stub_offset, packet_info *pinfo,
 
 /* We need to hash in the SMB fid number to generate a unique hash table
  * key as DCERPC over SMB allows several pipes over the same TCP/IP
- * socket. 
+ * socket.
  * We pass this function the transport type here to make sure we only look
  * at this function if it came across an SMB pipe.
  * Other transports might need to mix in their own extra multiplexing data
@@ -2570,7 +2516,7 @@ guint16 dcerpc_get_transport_salt (packet_info *pinfo)
  */
 
 static void
-dissect_dcerpc_cn_bind (tvbuff_t *tvb, gint offset, packet_info *pinfo, 
+dissect_dcerpc_cn_bind (tvbuff_t *tvb, gint offset, packet_info *pinfo,
                        proto_tree *dcerpc_tree, e_dce_cn_common_hdr_t *hdr)
 {
     conversation_t *conv = NULL;
@@ -2584,12 +2530,10 @@ dissect_dcerpc_cn_bind (tvbuff_t *tvb, gint offset, packet_info *pinfo,
     e_uuid_t trans_id;
     guint32 trans_ver;
     guint16 if_ver, if_ver_minor;
-    char uuid_str[DCERPC_UUID_STR_LEN]; 
-    int uuid_str_len;
     dcerpc_auth_info auth_info;
-#ifdef _WIN32
-    char UUID_NAME[MAX_PATH];
-#endif
+    char *uuid_str;
+    const char *uuid_name = NULL;
+       proto_item *iface_item;
 
     offset = dissect_dcerpc_uint16 (tvb, offset, pinfo, dcerpc_tree, hdr->drep,
                                     hf_dcerpc_cn_max_xmit, NULL);
@@ -2607,56 +2551,64 @@ dissect_dcerpc_cn_bind (tvbuff_t *tvb, gint offset, packet_info *pinfo,
     offset += 3;
 
     for (i = 0; i < num_ctx_items; i++) {
+           proto_item *ctx_item;
            proto_tree *ctx_tree = NULL, *iface_tree = NULL;
+        gint ctx_offset = offset;
 
-      offset = dissect_dcerpc_uint16 (tvb, offset, pinfo, NULL, hdr->drep,
+      dissect_dcerpc_uint16 (tvb, offset, pinfo, NULL, hdr->drep,
                                       hf_dcerpc_cn_ctx_id, &ctx_id);
 
+      if (check_col (pinfo->cinfo, COL_DCE_CTX)) {
+               if(pinfo->dcectxid == 0) {
+                       col_append_fstr (pinfo->cinfo, COL_DCE_CTX, "%u", ctx_id);
+               } else {
+                       /* this is not the first DCE-RPC request/response in this (TCP?-)PDU,
+                        * prepend a delimiter */
+                       col_append_fstr (pinfo->cinfo, COL_DCE_CTX, "#%u", ctx_id);
+               }
+      }
+
       /* save context ID for use with dcerpc_add_conv_to_bind_table() */
       /* (if we have multiple contexts, this might cause "decode as"
        *  to behave unpredictably) */
       pinfo->dcectxid = ctx_id;
 
       if (dcerpc_tree) {
-             proto_item *ctx_item;
-
-             ctx_item = proto_tree_add_item(dcerpc_tree, hf_dcerpc_cn_ctx_id,
-                                            tvb, offset - 2, 2, 
+             ctx_item = proto_tree_add_item(dcerpc_tree, hf_dcerpc_cn_ctx_item,
+                                            tvb, offset, 0,
                                             hdr->drep[0] & 0x10);
-
              ctx_tree = proto_item_add_subtree(ctx_item, ett_dcerpc_cn_ctx);
       }
 
+      offset = dissect_dcerpc_uint16 (tvb, offset, pinfo, ctx_tree, hdr->drep,
+                                      hf_dcerpc_cn_ctx_id, &ctx_id);
       offset = dissect_dcerpc_uint8 (tvb, offset, pinfo, ctx_tree, hdr->drep,
                                       hf_dcerpc_cn_num_trans_items, &num_trans_items);
 
+      if(dcerpc_tree) {
+        proto_item_append_text(ctx_item, "[%u]: ID:%u", i+1, ctx_id);
+      }
+
       /* padding */
       offset += 1;
 
-      /* XXX - use "dissect_ndr_uuid_t()"? */
       dcerpc_tvb_get_uuid (tvb, offset, hdr->drep, &if_id);
       if (ctx_tree) {
-         proto_item *iface_item;
-
-         uuid_str_len = g_snprintf(uuid_str, DCERPC_UUID_STR_LEN, 
-                                 "%08x-%04x-%04x-%02x%02x-%02x%02x%02x%02x%02x%02x",
-                                  if_id.Data1, if_id.Data2, if_id.Data3,
-                                  if_id.Data4[0], if_id.Data4[1],
-                                  if_id.Data4[2], if_id.Data4[3],
-                                  if_id.Data4[4], if_id.Data4[5],
-                                  if_id.Data4[6], if_id.Data4[7]);
-
-         if (uuid_str_len == -1 || uuid_str_len >= DCERPC_UUID_STR_LEN)
-                 memset(uuid_str, 0, DCERPC_UUID_STR_LEN);
-#ifdef _WIN32
-         if(ResolveWin32UUID(if_id, UUID_NAME, MAX_PATH))
-                 iface_item = proto_tree_add_string_format (ctx_tree, hf_dcerpc_cn_bind_if_id, tvb,
-                                        offset, 16, uuid_str, "Interface: %s\tUUID: %s", UUID_NAME, uuid_str);
-         else
-#endif
-          iface_item = proto_tree_add_string_format (ctx_tree, hf_dcerpc_cn_bind_if_id, tvb,
-                                        offset, 16, uuid_str, "Interface UUID: %s", uuid_str);
+
+      iface_item = proto_tree_add_item(ctx_tree, hf_dcerpc_cn_bind_abstract_syntax, tvb, offset, 0, FALSE);
          iface_tree = proto_item_add_subtree(iface_item, ett_dcerpc_cn_iface);
+
+      uuid_str = guid_to_str((e_guid_t*)&if_id);
+      uuid_name = guids_get_uuid_name(&if_id);
+      if(uuid_name) {
+                 proto_tree_add_guid_format (iface_tree, hf_dcerpc_cn_bind_if_id, tvb,
+                                        offset, 16, (e_guid_t *) &if_id, "Interface: %s UUID: %s", uuid_name, uuid_str);
+          proto_item_append_text(iface_item, ": %s", uuid_name);
+      } else {
+          proto_tree_add_guid_format (iface_tree, hf_dcerpc_cn_bind_if_id, tvb,
+                                        offset, 16, (e_guid_t *) &if_id, "Interface UUID: %s", uuid_str);
+          proto_item_append_text(iface_item, ": %s", uuid_str);
+      }
       }
       offset += 16;
 
@@ -2672,6 +2624,11 @@ dissect_dcerpc_cn_bind (tvbuff_t *tvb, gint offset, packet_info *pinfo,
                                           hf_dcerpc_cn_bind_if_ver, &if_ver);
       }
 
+      if (ctx_tree) {
+          proto_item_append_text(iface_item, " V%u.%u", if_ver, if_ver_minor);
+          proto_item_set_len(iface_item, 20);
+      }
+
       if (!saw_ctx_item) {
         conv = find_conversation (pinfo->fd->num, &pinfo->src, &pinfo->dst, pinfo->ptype,
                                   pinfo->srcport, pinfo->destport, 0);
@@ -2709,60 +2666,42 @@ dissect_dcerpc_cn_bind (tvbuff_t *tvb, gint offset, packet_info *pinfo,
        }
 
         if (check_col (pinfo->cinfo, COL_INFO)) {
-         dcerpc_uuid_key key;
-         dcerpc_uuid_value *value;
-
-         key.uuid = if_id;
-         key.ver = if_ver;
-
          if (num_ctx_items > 1)
                  col_append_fstr(pinfo->cinfo, COL_INFO, ", %u context items, 1st", num_ctx_items);
-               
-         if ((value = g_hash_table_lookup(dcerpc_uuids, &key)))
-                 col_append_fstr(pinfo->cinfo, COL_INFO, " UUID: %s", value->name);
-         else
-#ifdef _WIN32
-               if(ResolveWin32UUID(if_id, UUID_NAME, MAX_PATH))
-                       col_append_fstr(pinfo->cinfo, COL_INFO, " [%s] UUID: %08x-%04x-%04x-%02x%02x-%02x%02x%02x%02x%02x%02x ver %u.%u",
-                           UUID_NAME, if_id.Data1, if_id.Data2, if_id.Data3,
-                           if_id.Data4[0], if_id.Data4[1],
-                           if_id.Data4[2], if_id.Data4[3],
-                           if_id.Data4[4], if_id.Data4[5],
-                           if_id.Data4[6], if_id.Data4[7],
-                           if_ver, if_ver_minor);
-         else
-#endif
-                       col_append_fstr(pinfo->cinfo, COL_INFO, " UUID: %08x-%04x-%04x-%02x%02x-%02x%02x%02x%02x%02x%02x ver %u.%u",
-                           if_id.Data1, if_id.Data2, if_id.Data3,
-                           if_id.Data4[0], if_id.Data4[1],
-                           if_id.Data4[2], if_id.Data4[3],
-                           if_id.Data4[4], if_id.Data4[5],
-                           if_id.Data4[6], if_id.Data4[7],
-                           if_ver, if_ver_minor);
+
+               col_append_fstr(pinfo->cinfo, COL_INFO, " %s V%u.%u",
+                       guids_resolve_uuid_to_str(&if_id), if_ver, if_ver_minor);
         }
         saw_ctx_item = TRUE;
       }
 
       for (j = 0; j < num_trans_items; j++) {
-        /* XXX - use "dissect_ndr_uuid_t()"? */
+           proto_tree *trans_tree = NULL;
+           proto_item *trans_item = NULL;
+
         dcerpc_tvb_get_uuid (tvb, offset, hdr->drep, &trans_id);
-        if (iface_tree) {
-           uuid_str_len = g_snprintf(uuid_str, DCERPC_UUID_STR_LEN, 
-                                  "%08x-%04x-%04x-%02x%02x-%02x%02x%02x%02x%02x%02x",
-                                  trans_id.Data1, trans_id.Data2, trans_id.Data3,
-                                  trans_id.Data4[0], trans_id.Data4[1],
-                                  trans_id.Data4[2], trans_id.Data4[3],
-                                  trans_id.Data4[4], trans_id.Data4[5],
-                                  trans_id.Data4[6], trans_id.Data4[7]);
-            if (uuid_str_len == -1 || uuid_str_len >= DCERPC_UUID_STR_LEN)
-                memset(uuid_str, 0, DCERPC_UUID_STR_LEN);
-            proto_tree_add_string_format (iface_tree, hf_dcerpc_cn_bind_trans_id, tvb,
-                                          offset, 16, uuid_str, "Transfer Syntax: %s", uuid_str);
+        if (ctx_tree) {
+
+        trans_item = proto_tree_add_item(ctx_tree, hf_dcerpc_cn_bind_trans_syntax, tvb, offset, 0, FALSE);
+        trans_tree = proto_item_add_subtree(trans_item, ett_dcerpc_cn_trans_syntax);
+
+        uuid_str = guid_to_str((e_guid_t *) &trans_id);
+            proto_tree_add_guid_format (trans_tree, hf_dcerpc_cn_bind_trans_id, tvb,
+                                          offset, 16, (e_guid_t *) &trans_id, "Transfer Syntax: %s", uuid_str);
+            proto_item_append_text(trans_item, "[%u]: %s", j+1, uuid_str);
         }
         offset += 16;
 
-        offset = dissect_dcerpc_uint32 (tvb, offset, pinfo, iface_tree, hdr->drep,
+        offset = dissect_dcerpc_uint32 (tvb, offset, pinfo, trans_tree, hdr->drep,
                                         hf_dcerpc_cn_bind_trans_ver, &trans_ver);
+        if (ctx_tree) {
+          proto_item_set_len(trans_item, 20);
+          proto_item_append_text(trans_item, " V%u", trans_ver);
+        }
+      }
+
+      if(ctx_tree) {
+        proto_item_set_len(ctx_item, offset - ctx_offset);
       }
     }
 
@@ -2775,7 +2714,7 @@ dissect_dcerpc_cn_bind (tvbuff_t *tvb, gint offset, packet_info *pinfo,
 }
 
 static void
-dissect_dcerpc_cn_bind_ack (tvbuff_t *tvb, gint offset, packet_info *pinfo, 
+dissect_dcerpc_cn_bind_ack (tvbuff_t *tvb, gint offset, packet_info *pinfo,
                            proto_tree *dcerpc_tree, e_dce_cn_common_hdr_t *hdr)
 {
     guint16 max_xmit, max_recv;
@@ -2786,8 +2725,6 @@ dissect_dcerpc_cn_bind_ack (tvbuff_t *tvb, gint offset, packet_info *pinfo,
     guint16 reason;
     e_uuid_t trans_id;
     guint32 trans_ver;
-    char uuid_str[DCERPC_UUID_STR_LEN]; 
-    int uuid_str_len;
     dcerpc_auth_info auth_info;
 
     offset = dissect_dcerpc_uint16 (tvb, offset, pinfo, dcerpc_tree, hdr->drep,
@@ -2823,7 +2760,7 @@ dissect_dcerpc_cn_bind_ack (tvbuff_t *tvb, gint offset, packet_info *pinfo,
 
        if(dcerpc_tree){
                proto_item *ctx_item;
-               ctx_item = proto_tree_add_text(dcerpc_tree, tvb, offset, 24, "Context ID: %d", i);
+               ctx_item = proto_tree_add_text(dcerpc_tree, tvb, offset, 24, "Context ID[%u]", i+1);
                ctx_tree = proto_item_add_subtree(ctx_item, ett_dcerpc_cn_ctx);
        }
 
@@ -2842,20 +2779,11 @@ dissect_dcerpc_cn_bind_ack (tvbuff_t *tvb, gint offset, packet_info *pinfo,
             offset += 2;
         }
 
-        /* XXX - use "dissect_ndr_uuid_t()"? */
         dcerpc_tvb_get_uuid (tvb, offset, hdr->drep, &trans_id);
         if (ctx_tree) {
-           uuid_str_len = g_snprintf(uuid_str, DCERPC_UUID_STR_LEN, 
-                                  "%08x-%04x-%04x-%02x%02x-%02x%02x%02x%02x%02x%02x",
-                                  trans_id.Data1, trans_id.Data2, trans_id.Data3,
-                                  trans_id.Data4[0], trans_id.Data4[1],
-                                  trans_id.Data4[2], trans_id.Data4[3],
-                                  trans_id.Data4[4], trans_id.Data4[5],
-                                  trans_id.Data4[6], trans_id.Data4[7]);
-           if (uuid_str_len == -1 || uuid_str_len >= DCERPC_UUID_STR_LEN)
-                 memset(uuid_str, 0, DCERPC_UUID_STR_LEN);
-            proto_tree_add_string_format (ctx_tree, hf_dcerpc_cn_ack_trans_id, tvb,
-                                          offset, 16, uuid_str, "Transfer Syntax: %s", uuid_str);
+            proto_tree_add_guid_format (ctx_tree, hf_dcerpc_cn_ack_trans_id, tvb,
+                                          offset, 16, (e_guid_t *) &trans_id, "Transfer Syntax: %s",
+                                          guid_to_str((e_guid_t *) &trans_id));
         }
         offset += 16;
 
@@ -2887,7 +2815,7 @@ dissect_dcerpc_cn_bind_ack (tvbuff_t *tvb, gint offset, packet_info *pinfo,
 }
 
 static void
-dissect_dcerpc_cn_bind_nak (tvbuff_t *tvb, gint offset, packet_info *pinfo, 
+dissect_dcerpc_cn_bind_nak (tvbuff_t *tvb, gint offset, packet_info *pinfo,
                            proto_tree *dcerpc_tree, e_dce_cn_common_hdr_t *hdr)
 {
     guint16 reason;
@@ -2950,13 +2878,13 @@ static void
 dissect_dcerpc_cn_stub (tvbuff_t *tvb, int offset, packet_info *pinfo,
                         proto_tree *dcerpc_tree, proto_tree *tree,
                         e_dce_cn_common_hdr_t *hdr, dcerpc_info *di,
-                        dcerpc_auth_info *auth_info, guint32 alloc_hint,
+                        dcerpc_auth_info *auth_info, guint32 alloc_hint _U_,
                         guint32 frame)
 {
     gint length, reported_length;
     gboolean save_fragmented;
     fragment_data *fd_head=NULL;
-    guint32 tot_len;
+
     tvbuff_t *auth_tvb, *payload_tvb, *decrypted_tvb;
     proto_item *pi;
     proto_item *parent_pi;
@@ -2998,7 +2926,7 @@ dissect_dcerpc_cn_stub (tvbuff_t *tvb, int offset, packet_info *pinfo,
             * encrypted; attempt to decrypt it.
             */
            dcerpc_auth_subdissector_fns *auth_fns;
-           
+
            /* Start out assuming we won't succeed in decrypting. */
            decrypted_tvb = NULL;
 
@@ -3008,8 +2936,8 @@ dissect_dcerpc_cn_stub (tvbuff_t *tvb, int offset, packet_info *pinfo,
 
                    result = decode_encrypted_data(
                            payload_tvb, auth_tvb, pinfo, auth_fns,
-                           hdr->ptype == PDU_REQ, auth_info);      
-                   
+                           hdr->ptype == PDU_REQ, auth_info);
+
                    if (result) {
                            if (dcerpc_tree)
                                proto_tree_add_text(
@@ -3021,7 +2949,7 @@ dissect_dcerpc_cn_stub (tvbuff_t *tvb, int offset, packet_info *pinfo,
 
                            add_new_data_source(
                                    pinfo, result, "Decrypted stub data");
-                           
+
                            /* We succeeded. */
                            decrypted_tvb = result;
                    }
@@ -3036,7 +2964,7 @@ dissect_dcerpc_cn_stub (tvbuff_t *tvb, int offset, packet_info *pinfo,
        dcerpc_try_handoff(
                pinfo, tree, dcerpc_tree, payload_tvb, decrypted_tvb,
                hdr->drep, di, auth_info);
-       
+
        pinfo->fragmented = save_fragmented;
        return;
     }
@@ -3046,7 +2974,7 @@ dissect_dcerpc_cn_stub (tvbuff_t *tvb, int offset, packet_info *pinfo,
 
        /* debug output of essential fragment data. */
        /* leave it here for future debugging sessions */
-       /*printf("DCE num:%u offset:%u frag_len:%u tvb_len:%u\n", 
+       /*printf("DCE num:%u offset:%u frag_len:%u tvb_len:%u\n",
                   pinfo->fd->num, offset, hdr->frag_len, tvb_length(decrypted_tvb));*/
 
     /* if we are not doing reassembly and this is the first fragment
@@ -3059,12 +2987,12 @@ dissect_dcerpc_cn_stub (tvbuff_t *tvb, int offset, packet_info *pinfo,
        dcerpc_try_handoff(
                pinfo, tree, dcerpc_tree, payload_tvb, decrypted_tvb,
                hdr->drep, di, auth_info);
-       
+
         if (check_col(pinfo->cinfo, COL_INFO)) {
             col_append_fstr(pinfo->cinfo, COL_INFO,
                             " [DCE/RPC %s fragment]", fragment_type(hdr->flags));
         }
-               expert_add_info_format(pinfo, NULL, PI_REASSEMBLE, PI_CHAT, 
+               expert_add_info_format(pinfo, NULL, PI_REASSEMBLE, PI_CHAT,
                        "%s fragment", fragment_type(hdr->flags));
         pinfo->fragmented = save_fragmented;
         return;
@@ -3072,10 +3000,10 @@ dissect_dcerpc_cn_stub (tvbuff_t *tvb, int offset, packet_info *pinfo,
 
     /* if we have already seen this packet, see if it was reassembled
        and if so dissect the full pdu.
-       then exit 
+       then exit
     */
     if(pinfo->fd->flags.visited){
-       fd_head=fragment_get(pinfo, frame, dcerpc_co_reassemble_table);
+       fd_head=fragment_get_reassembled(pinfo, frame, dcerpc_co_reassemble_table);
        goto end_cn_stub;
     }
 
@@ -3083,16 +3011,16 @@ dissect_dcerpc_cn_stub (tvbuff_t *tvb, int offset, packet_info *pinfo,
        nor the first fragment then there is nothing more we can do
        so we just have to exit
     */
-    if( !dcerpc_reassemble )
+    if( !dcerpc_reassemble || (tvb_length(tvb)!=tvb_reported_length(tvb)) )
         goto end_cn_stub;
 
     /* if we didnt get 'frame' we dont know where the PDU started and thus
-       it is pointless to continue 
+       it is pointless to continue
     */
     if(!frame)
         goto end_cn_stub;
 
-    /* from now on we must attempt to reassemble the PDU 
+    /* from now on we must attempt to reassemble the PDU
     */
 
     /* if we get here we know it is the first time we see the packet
@@ -3106,69 +3034,20 @@ dissect_dcerpc_cn_stub (tvbuff_t *tvb, int offset, packet_info *pinfo,
       goto end_cn_stub;
     }
 
-    /* defragmentation is a bit tricky here, as there's no offset of the fragment 
+    /* defragmentation is a bit tricky, as there's no offset of the fragment
      * in the protocol data.
      *
-        * The allocation hint will contain the remaining bytes of all fragments following.
-        *
-     * Currently two possible ways:
-     * - the transmitter sends an alloc_hint != 0, use it
-     * - the transmitter sends an alloc_hint == 0, simply append fragments
+     * just use fragment_add_seq_next() and hope that TCP/SMB segments coming
+     * in with the correct sequence.
      */
-
-    /* if this is the first fragment we need to start reassembly
-    */
-    if(hdr->flags&PFC_FIRST_FRAG){
-       fragment_add(decrypted_tvb, 0, pinfo, frame, 
-               dcerpc_co_reassemble_table,
-               0 /* fragment offset */, tvb_length(decrypted_tvb), TRUE /* more_frags */);
-       fragment_set_tot_len(pinfo, frame,
-        dcerpc_co_reassemble_table, alloc_hint ? alloc_hint : tvb_length(decrypted_tvb));
-
-       goto end_cn_stub;
-    }
-
-    /* if this is a middle fragment, just add it and exit */
-    if(!(hdr->flags&PFC_LAST_FRAG)){
-       tot_len = fragment_get_tot_len(pinfo, frame,
-                dcerpc_co_reassemble_table);
-       if(alloc_hint == 0) {
-               fragment_set_tot_len(pinfo, frame,
-                 dcerpc_co_reassemble_table, tot_len + tvb_length(decrypted_tvb));
-       } else {
-               /* the alloc_hint shouldn't be larger than the expected remaining length */
-               if(alloc_hint > (tot_len - tvb_length(decrypted_tvb))) {
-                       THROW(ReportedBoundsError);
-               }
-       }
-       fragment_add(decrypted_tvb, 0, pinfo, frame,
-               dcerpc_co_reassemble_table,
-               tot_len-alloc_hint /* fragment offset */, tvb_length(decrypted_tvb),
-               TRUE /* more_frags */);
-
-       goto end_cn_stub;
-    }
-
-    /* this was the last fragment add it to reassembly
-    */
-    tot_len = fragment_get_tot_len(pinfo, frame,
-               dcerpc_co_reassemble_table);
-       /* the alloc_hint shouldn't be larger than the expected remaining length */
-       if(alloc_hint != 0 && alloc_hint > (tot_len - tvb_length(decrypted_tvb))) {
-               THROW(ReportedBoundsError);
-       }
-    fd_head = fragment_add(decrypted_tvb, 0, pinfo, frame,
-               dcerpc_co_reassemble_table,
-               tot_len-alloc_hint /* fragment offset */, tvb_length(decrypted_tvb),
-               FALSE /* more_frags */);
-       if(alloc_hint == 0) {           
-           fragment_set_tot_len(pinfo, frame,
-                 dcerpc_co_reassemble_table, tot_len + tvb_length(decrypted_tvb));
-       }
+    fd_head = fragment_add_seq_next(decrypted_tvb, 0, pinfo, frame,
+               dcerpc_co_fragment_table, dcerpc_co_reassemble_table,
+               tvb_length(decrypted_tvb),
+               hdr->flags&PFC_LAST_FRAG ? FALSE : TRUE /* more_frags */);
 
 end_cn_stub:
 
-    /* if reassembly is complete and this is the last fragment 
+    /* if reassembly is complete and this is the last fragment
         * (multiple fragments in one PDU are possible!)
         * dissect the full PDU
      */
@@ -3178,8 +3057,12 @@ end_cn_stub:
            tvbuff_t *next_tvb;
         proto_item *frag_tree_item;
 
-           next_tvb = tvb_new_real_data(fd_head->data, fd_head->datalen, fd_head->datalen);
-           tvb_set_child_real_data_tvbuff(decrypted_tvb, next_tvb);
+           next_tvb = tvb_new_real_data(fd_head->data, fd_head->len, fd_head->len);
+           if(decrypted_tvb){
+               tvb_set_child_real_data_tvbuff(decrypted_tvb, next_tvb);
+           } else {
+               tvb_set_child_real_data_tvbuff(payload_tvb, next_tvb);
+           }
            add_new_data_source(pinfo, next_tvb, "Reassembled DCE/RPC");
            show_fragment_tree(fd_head, &dcerpc_frag_items,
                        tree, pinfo, next_tvb, &frag_tree_item);
@@ -3192,15 +3075,21 @@ end_cn_stub:
 
            pinfo->fragmented = FALSE;
 
-               expert_add_info_format(pinfo, NULL, PI_REASSEMBLE, PI_CHAT, 
-                       "%s fragment, reassembled here in #%u", fragment_type(hdr->flags), fd_head->reassembled_in);
+               expert_add_info_format(pinfo, frag_tree_item, PI_REASSEMBLE, PI_CHAT,
+                       "%s fragment, reassembled",
+                       fragment_type(hdr->flags));
 
            dcerpc_try_handoff (pinfo, tree, dcerpc_tree, next_tvb,
                next_tvb, hdr->drep, di, auth_info);
 
        } else {
-           pi = proto_tree_add_uint(dcerpc_tree, hf_dcerpc_reassembled_in,
+           if(decrypted_tvb){
+               pi = proto_tree_add_uint(dcerpc_tree, hf_dcerpc_reassembled_in,
                                decrypted_tvb, 0, 0, fd_head->reassembled_in);
+           } else {
+               pi = proto_tree_add_uint(dcerpc_tree, hf_dcerpc_reassembled_in,
+                               payload_tvb, 0, 0, fd_head->reassembled_in);
+           }
         PROTO_ITEM_SET_GENERATED(pi);
         parent_pi = proto_tree_get_parent(dcerpc_tree);
         if(parent_pi != NULL) {
@@ -3210,7 +3099,7 @@ end_cn_stub:
                col_append_fstr(pinfo->cinfo, COL_INFO,
                        " [DCE/RPC %s fragment, reas: #%u]", fragment_type(hdr->flags), fd_head->reassembled_in);
            }
-               expert_add_info_format(pinfo, NULL, PI_REASSEMBLE, PI_CHAT, 
+               expert_add_info_format(pinfo, NULL, PI_REASSEMBLE, PI_CHAT,
                        "%s fragment, reassembled in #%u", fragment_type(hdr->flags), fd_head->reassembled_in);
        }
     } else {
@@ -3220,7 +3109,7 @@ end_cn_stub:
            col_append_fstr(pinfo->cinfo, COL_INFO,
                        " [DCE/RPC %s fragment]", fragment_type(hdr->flags));
        }
-       expert_add_info_format(pinfo, NULL, PI_REASSEMBLE, PI_CHAT, 
+       expert_add_info_format(pinfo, NULL, PI_REASSEMBLE, PI_CHAT,
                "%s fragment", fragment_type(hdr->flags));
 
        if(decrypted_tvb){
@@ -3237,9 +3126,9 @@ end_cn_stub:
  *  Registers a conversation/UUID binding association, so that
  *  we can invoke the proper sub-dissector for a given DCERPC
  *  conversation.
- * 
+ *
  *  @param binding all values needed to create and bind a new conversation
- * 
+ *
  *  @return Pointer to newly-added UUID/conversation binding.
  */
 struct _dcerpc_bind_value *
@@ -3250,25 +3139,25 @@ dcerpc_add_conv_to_bind_table(decode_dcerpc_bind_values_t *binding)
     conversation_t *conv;
 
     conv = find_conversation (
-        0, 
-        &binding->addr_a, 
-        &binding->addr_b, 
-        binding->ptype, 
-        binding->port_a, 
-        binding->port_b, 
+        0,
+        &binding->addr_a,
+        &binding->addr_b,
+        binding->ptype,
+        binding->port_a,
+        binding->port_b,
         0);
 
     if (!conv) {
         conv = conversation_new (
-            0, 
-            &binding->addr_a, 
-            &binding->addr_b, 
-            binding->ptype, 
-            binding->port_a, 
-            binding->port_b, 
+            0,
+            &binding->addr_a,
+            &binding->addr_b,
+            binding->ptype,
+            binding->port_a,
+            binding->port_b,
             0);
     }
-    
+
     bind_value = se_alloc (sizeof (dcerpc_bind_value));
     bind_value->uuid = binding->uuid;
     bind_value->ver = binding->ver;
@@ -3277,7 +3166,7 @@ dcerpc_add_conv_to_bind_table(decode_dcerpc_bind_values_t *binding)
     key->conv = conv;
     key->ctx_id = binding->ctx_id;
     key->smb_fid = binding->smb_fid;
+
     /* add this entry to the bind table, first removing any
        previous ones that are identical
      */
@@ -3291,18 +3180,16 @@ dcerpc_add_conv_to_bind_table(decode_dcerpc_bind_values_t *binding)
 }
 
 static void
-dissect_dcerpc_cn_rqst (tvbuff_t *tvb, gint offset, packet_info *pinfo, 
+dissect_dcerpc_cn_rqst (tvbuff_t *tvb, gint offset, packet_info *pinfo,
                        proto_tree *dcerpc_tree, proto_tree *tree,
                        e_dce_cn_common_hdr_t *hdr)
 {
     conversation_t *conv;
     guint16 ctx_id;
     guint16 opnum;
-    e_uuid_t obj_id;
+    e_uuid_t obj_id = DCERPC_UUID_NULL;
     dcerpc_auth_info auth_info;
     guint32 alloc_hint;
-    char uuid_str[DCERPC_UUID_STR_LEN]; 
-    int uuid_str_len;
     proto_item *pi;
     proto_item *parent_pi;
 
@@ -3316,6 +3203,16 @@ dissect_dcerpc_cn_rqst (tvbuff_t *tvb, gint offset, packet_info *pinfo,
         proto_item_append_text(parent_pi, " Ctx: %u", ctx_id);
     }
 
+    if (check_col (pinfo->cinfo, COL_DCE_CTX)) {
+               if(pinfo->dcectxid == 0) {
+                       col_append_fstr (pinfo->cinfo, COL_DCE_CTX, "%u", ctx_id);
+               } else {
+                       /* this is not the first DCE-RPC request/response in this (TCP?-)PDU,
+                        * prepend a delimiter */
+                       col_append_fstr (pinfo->cinfo, COL_DCE_CTX, "#%u", ctx_id);
+               }
+    }
+
     offset = dissect_dcerpc_uint16 (tvb, offset, pinfo, dcerpc_tree, hdr->drep,
                                     hf_dcerpc_opnum, &opnum);
 
@@ -3328,24 +3225,11 @@ dissect_dcerpc_cn_rqst (tvbuff_t *tvb, gint offset, packet_info *pinfo,
     }
 
     if (hdr->flags & PFC_OBJECT_UUID) {
-        /* XXX - use "dissect_ndr_uuid_t()"? */
         dcerpc_tvb_get_uuid (tvb, offset, hdr->drep, &obj_id);
         if (dcerpc_tree) {
-           uuid_str_len = g_snprintf(uuid_str, DCERPC_UUID_STR_LEN, 
-                                    "%08x-%04x-%04x-%02x%02x-%02x%02x%02x%02x%02x%02x",
-                                    obj_id.Data1, obj_id.Data2, obj_id.Data3,
-                                    obj_id.Data4[0],
-                                    obj_id.Data4[1],
-                                    obj_id.Data4[2],
-                                    obj_id.Data4[3],
-                                    obj_id.Data4[4],
-                                    obj_id.Data4[5],
-                                    obj_id.Data4[6],
-                                    obj_id.Data4[7]);
-           if (uuid_str_len == -1 || uuid_str_len >= DCERPC_UUID_STR_LEN)
-                 memset(uuid_str, 0, DCERPC_UUID_STR_LEN);
-            proto_tree_add_string_format (dcerpc_tree, hf_dcerpc_obj_id, tvb,
-                                          offset, 16, uuid_str, "Object UUID: %s", uuid_str);
+            proto_tree_add_guid_format (dcerpc_tree, hf_dcerpc_obj_id, tvb,
+                                          offset, 16, (e_guid_t *) &obj_id, "Object UUID: %s",
+                                          guid_to_str((e_guid_t *) &obj_id));
         }
         offset += 16;
     }
@@ -3398,9 +3282,9 @@ dissect_dcerpc_cn_rqst (tvbuff_t *tvb, gint offset, packet_info *pinfo,
                                dcerpc_cn_call_key *call_key;
                                dcerpc_call_value *call_value;
 
-                               /* We found the binding and it is the first fragment 
-                                  (or a complete PDU) of a dcerpc pdu so just add 
-                                  the call to both the call table and the 
+                               /* We found the binding and it is the first fragment
+                                  (or a complete PDU) of a dcerpc pdu so just add
+                                  the call to both the call table and the
                                   matched table
                                */
                                call_key=se_alloc (sizeof (dcerpc_cn_call_key));
@@ -3417,11 +3301,13 @@ dissect_dcerpc_cn_rqst (tvbuff_t *tvb, gint offset, packet_info *pinfo,
                                call_value=se_alloc (sizeof (dcerpc_call_value));
                                call_value->uuid = bind_value->uuid;
                                call_value->ver = bind_value->ver;
+                call_value->object_uuid = obj_id;
                                call_value->opnum = opnum;
                                call_value->req_frame=pinfo->fd->num;
                                call_value->req_time=pinfo->fd->abs_ts;
                                call_value->rep_frame=0;
                                call_value->max_ptr=0;
+                               call_value->se_data = NULL;
                                call_value->private_data = NULL;
                                g_hash_table_insert (dcerpc_cn_calls, call_key, call_value);
 
@@ -3457,8 +3343,14 @@ dissect_dcerpc_cn_rqst (tvbuff_t *tvb, gint offset, packet_info *pinfo,
            dissect_dcerpc_cn_stub (tvb, offset, pinfo, dcerpc_tree, tree,
                                    hdr, di, &auth_info, alloc_hint,
                                    value->req_frame);
-       } else
+        } else {
+        /* no bind information, simply show stub data */
+        pi = proto_tree_add_text(dcerpc_tree, tvb, offset, 0, "No bind info for this interface Context ID - capture start too late?");
+        PROTO_ITEM_SET_GENERATED(pi);
+           expert_add_info_format(pinfo, pi, PI_UNDECODED, PI_NOTE, "No bind info for interface Context ID:%u",
+            ctx_id);
            show_stub_data (tvb, offset, dcerpc_tree, &auth_info, TRUE);
+        }
     }
 
     /* Dissect the verifier */
@@ -3478,6 +3370,7 @@ dissect_dcerpc_cn_resp (tvbuff_t *tvb, gint offset, packet_info *pinfo,
     guint32 alloc_hint;
     proto_item *pi;
     proto_item *parent_pi;
+    e_uuid_t obj_id_null = DCERPC_UUID_NULL;
 
     offset = dissect_dcerpc_uint32 (tvb, offset, pinfo, dcerpc_tree, hdr->drep,
                                     hf_dcerpc_cn_alloc_hint, &alloc_hint);
@@ -3489,6 +3382,17 @@ dissect_dcerpc_cn_resp (tvbuff_t *tvb, gint offset, packet_info *pinfo,
         proto_item_append_text(parent_pi, " Ctx: %u", ctx_id);
     }
 
+    if (check_col (pinfo->cinfo, COL_DCE_CTX)) {
+               if(pinfo->dcectxid == 0) {
+                       col_append_fstr (pinfo->cinfo, COL_DCE_CTX, "%u", ctx_id);
+               } else {
+                       /* this is not the first DCE-RPC request/response in this (TCP?-)PDU,
+                        * prepend a delimiter */
+                       col_append_fstr (pinfo->cinfo, COL_DCE_CTX, "#%u", ctx_id);
+               }
+    }
+
+
     /* save context ID for use with dcerpc_add_conv_to_bind_table() */
     pinfo->dcectxid = ctx_id;
 
@@ -3559,6 +3463,16 @@ dissect_dcerpc_cn_resp (tvbuff_t *tvb, gint offset, packet_info *pinfo,
            di->call_data = value;
 
            proto_tree_add_uint (dcerpc_tree, hf_dcerpc_opnum, tvb, 0, 0, value->opnum);
+
+        /* (optional) "Object UUID" from request */
+        if (value && dcerpc_tree && memcmp(&value->object_uuid, &obj_id_null, sizeof(obj_id_null)) != 0) {
+                pi = proto_tree_add_guid_format (dcerpc_tree, hf_dcerpc_obj_id, tvb,
+                                              offset, 0, (e_guid_t *) &value->object_uuid, "Object UUID: %s",
+                                              guid_to_str((e_guid_t *) &value->object_uuid));
+                PROTO_ITEM_SET_GENERATED(pi);
+        }
+
+        /* request in */
            if(value->req_frame!=0){
                nstime_t delta_ts;
                pi = proto_tree_add_uint(dcerpc_tree, hf_dcerpc_request_in,
@@ -3570,13 +3484,24 @@ dissect_dcerpc_cn_resp (tvbuff_t *tvb, gint offset, packet_info *pinfo,
                nstime_delta(&delta_ts, &pinfo->fd->abs_ts, &value->req_time);
                pi = proto_tree_add_time(dcerpc_tree, hf_dcerpc_time, tvb, offset, 0, &delta_ts);
         PROTO_ITEM_SET_GENERATED(pi);
+        } else {
+                   pi = proto_tree_add_text(dcerpc_tree,
+                                       tvb, 0, 0, "No request to this DCE/RPC call found");
+                   expert_add_info_format(pinfo, pi, PI_SEQUENCE, PI_NOTE,
+                           "No request to this DCE/RPC call found");
            }
 
            dissect_dcerpc_cn_stub (tvb, offset, pinfo, dcerpc_tree, tree,
                                    hdr, di, &auth_info, alloc_hint,
                                    value->rep_frame);
-        } else
+        } else {
+            /* no bind information, simply show stub data */
+            pi = proto_tree_add_text(dcerpc_tree, tvb, offset, 0, "No bind info for this interface Context ID - capture start too late?");
+            PROTO_ITEM_SET_GENERATED(pi);
+               expert_add_info_format(pinfo, pi, PI_UNDECODED, PI_NOTE, "No bind info for interface Context ID:%u",
+                ctx_id);
             show_stub_data (tvb, offset, dcerpc_tree, &auth_info, TRUE);
+        }
     }
 
     /* Dissect the verifier */
@@ -3601,6 +3526,16 @@ dissect_dcerpc_cn_fault (tvbuff_t *tvb, gint offset, packet_info *pinfo,
     offset = dissect_dcerpc_uint16 (tvb, offset, pinfo, dcerpc_tree, hdr->drep,
                                     hf_dcerpc_cn_ctx_id, &ctx_id);
 
+    if (check_col (pinfo->cinfo, COL_DCE_CTX)) {
+               if(pinfo->dcectxid == 0) {
+                       col_append_fstr (pinfo->cinfo, COL_DCE_CTX, "%u", ctx_id);
+               } else {
+                       /* this is not the first DCE-RPC request/response in this (TCP?-)PDU,
+                        * prepend a delimiter */
+                       col_append_fstr (pinfo->cinfo, COL_DCE_CTX, "#%u", ctx_id);
+               }
+    }
+
     offset = dissect_dcerpc_uint8 (tvb, offset, pinfo, dcerpc_tree, hdr->drep,
                                    hf_dcerpc_cn_cancel_count, NULL);
     /* padding */
@@ -3700,11 +3635,16 @@ dissect_dcerpc_cn_fault (tvbuff_t *tvb, gint offset, packet_info *pinfo,
                nstime_delta(&delta_ts, &pinfo->fd->abs_ts, &value->req_time);
                pi = proto_tree_add_time(dcerpc_tree, hf_dcerpc_time, tvb, offset, 0, &delta_ts);
         PROTO_ITEM_SET_GENERATED(pi);
+        } else {
+                   pi = proto_tree_add_text(dcerpc_tree,
+                                       tvb, 0, 0, "No request to this DCE/RPC call found");
+                   expert_add_info_format(pinfo, pi, PI_SEQUENCE, PI_NOTE,
+                           "No request to this DCE/RPC call found");
            }
 
            length = tvb_length_remaining(tvb, offset);
            reported_length = tvb_reported_length_remaining(tvb, offset);
-               /* as we now create a tvb in dissect_dcerpc_cn() containing only the 
+               /* as we now create a tvb in dissect_dcerpc_cn() containing only the
                 * stub_data, the following calculation is no longer valid:
             * stub_length = hdr->frag_len - offset - auth_info.auth_size;
                 * simply use the remaining length of the tvb instead.
@@ -3775,13 +3715,10 @@ dissect_dcerpc_cn_fault (tvbuff_t *tvb, gint offset, packet_info *pinfo,
                }
                if(hdr->flags&PFC_FIRST_FRAG){  /* FIRST fragment */
                    if( (!pinfo->fd->flags.visited) && value->rep_frame ){
-                       fragment_add(tvb, offset, pinfo, value->rep_frame,
-                            dcerpc_co_reassemble_table,
-                            0,
+                       fragment_add_seq_next(tvb, offset, pinfo, value->rep_frame,
+                            dcerpc_co_fragment_table, dcerpc_co_reassemble_table,
                             stub_length,
                             TRUE);
-                       fragment_set_tot_len(pinfo, value->rep_frame,
-                            dcerpc_co_reassemble_table, alloc_hint);
                    }
                    if (check_col(pinfo->cinfo, COL_INFO)) {
                        col_append_fstr(pinfo->cinfo, COL_INFO,
@@ -3790,14 +3727,10 @@ dissect_dcerpc_cn_fault (tvbuff_t *tvb, gint offset, packet_info *pinfo,
                } else if(hdr->flags&PFC_LAST_FRAG){  /* LAST fragment */
                    if( value->rep_frame ){
                        fragment_data *fd_head;
-                       guint32 tot_len;
 
-                       tot_len = fragment_get_tot_len(pinfo, value->rep_frame,
-                                      dcerpc_co_reassemble_table);
-                       fd_head = fragment_add(tvb, offset, pinfo,
+                       fd_head = fragment_add_seq_next(tvb, offset, pinfo,
                             value->rep_frame,
-                            dcerpc_co_reassemble_table,
-                            tot_len-alloc_hint,
+                            dcerpc_co_fragment_table, dcerpc_co_reassemble_table,
                             stub_length,
                             TRUE);
 
@@ -3806,7 +3739,7 @@ dissect_dcerpc_cn_fault (tvbuff_t *tvb, gint offset, packet_info *pinfo,
                            tvbuff_t *next_tvb;
                 proto_item *frag_tree_item;
 
-                           next_tvb = tvb_new_real_data(fd_head->data, fd_head->datalen, fd_head->datalen);
+                           next_tvb = tvb_new_real_data(fd_head->data, fd_head->len, fd_head->len);
                            tvb_set_child_real_data_tvbuff(tvb, next_tvb);
                            add_new_data_source(pinfo, next_tvb, "Reassembled DCE/RPC");
                            show_fragment_tree(fd_head, &dcerpc_frag_items,
@@ -3842,12 +3775,8 @@ dissect_dcerpc_cn_fault (tvbuff_t *tvb, gint offset, packet_info *pinfo,
                    }
                } else {  /* MIDDLE fragment(s) */
                    if( (!pinfo->fd->flags.visited) && value->rep_frame ){
-                       guint32 tot_len;
-                       tot_len = fragment_get_tot_len(pinfo, value->rep_frame,
-                                      dcerpc_co_reassemble_table);
-                       fragment_add(tvb, offset, pinfo, value->rep_frame,
-                            dcerpc_co_reassemble_table,
-                            tot_len-alloc_hint,
+                       fragment_add_seq_next(tvb, offset, pinfo, value->rep_frame,
+                            dcerpc_co_fragment_table, dcerpc_co_reassemble_table,
                             stub_length,
                             TRUE);
                    }
@@ -3899,7 +3828,6 @@ dissect_dcerpc_cn (tvbuff_t *tvb, int offset, packet_info *pinfo,
         offset += 4;
         padding += 4;
     }
-
     /*
      * Check if this looks like a C/O DCERPC call
      */
@@ -3929,7 +3857,13 @@ dissect_dcerpc_cn (tvbuff_t *tvb, int offset, packet_info *pinfo,
     offset += 4;
 
     if (check_col (pinfo->cinfo, COL_DCE_CALL)) {
-        col_append_fstr (pinfo->cinfo, COL_DCE_CALL, "%u", hdr.call_id);
+               if(pinfo->dcectxid == 0) {
+                       col_append_fstr (pinfo->cinfo, COL_DCE_CALL, "%u", hdr.call_id);
+               } else {
+                       /* this is not the first DCE-RPC request/response in this (TCP?-)PDU,
+                        * prepend a delimiter */
+                       col_append_fstr (pinfo->cinfo, COL_DCE_CALL, "#%u", hdr.call_id);
+               }
     }
 
     if (can_desegment && pinfo->can_desegment
@@ -3945,7 +3879,7 @@ dissect_dcerpc_cn (tvbuff_t *tvb, int offset, packet_info *pinfo,
 
     if (check_col (pinfo->cinfo, COL_INFO)) {
         if(pinfo->dcectxid != 0) {
-            /* this is not the first DCE-RPC request/response in this (TCP?-)PDU, 
+            /* this is not the first DCE-RPC request/response in this (TCP?-)PDU,
              * append a delimiter and set a column fence */
             col_append_str (pinfo->cinfo, COL_INFO, " # ");
             col_set_fence(pinfo->cinfo,COL_INFO);
@@ -3956,80 +3890,82 @@ dissect_dcerpc_cn (tvbuff_t *tvb, int offset, packet_info *pinfo,
 
     if(pinfo->dcectxid != 0) {
         /* this is not the first DCE-RPC request/response in this (TCP?-)PDU */
-               expert_add_info_format(pinfo, NULL, PI_SEQUENCE, PI_NOTE, "Multiple DCE/RPC fragments/PDU's in one packet");
+               expert_add_info_format(pinfo, NULL, PI_SEQUENCE, PI_CHAT, "Multiple DCE/RPC fragments/PDU's in one packet");
        }
 
+    offset = start_offset;
+    tvb_ensure_bytes_exist(tvb, offset, 16);
     if (tree) {
-        offset = start_offset;
-        tvb_ensure_bytes_exist(tvb, offset, hdr.frag_len);
         ti = proto_tree_add_item (tree, proto_dcerpc, tvb, offset, hdr.frag_len, FALSE);
-        if (ti) {
-            dcerpc_tree = proto_item_add_subtree (ti, ett_dcerpc);
-        }
-        proto_tree_add_uint (dcerpc_tree, hf_dcerpc_ver, tvb, offset++, 1, hdr.rpc_ver);
-        proto_tree_add_uint (dcerpc_tree, hf_dcerpc_ver_minor, tvb, offset++, 1, hdr.rpc_ver_minor);
-        tf = proto_tree_add_uint (dcerpc_tree, hf_dcerpc_packet_type, tvb, offset++, 1, hdr.ptype);
-       } else {
-               tf = NULL;
-       }
-       
-       /* XXX - too much "output noise", removed for now
-       if(hdr.ptype == PDU_BIND || hdr.ptype == PDU_ALTER ||
-               hdr.ptype == PDU_BIND_ACK || hdr.ptype == PDU_ALTER_ACK)
-               expert_add_info_format(pinfo, tf, PI_SEQUENCE, PI_CHAT, "Context change: %s",
-                       val_to_str(hdr.ptype, pckt_vals, "(0x%x)"));*/
-       if(hdr.ptype == PDU_BIND_NAK)
-               expert_add_info_format(pinfo, tf, PI_SEQUENCE, PI_WARN, "Bind not acknowledged");
+        dcerpc_tree = proto_item_add_subtree (ti, ett_dcerpc);
+    }
+
+    proto_tree_add_uint (dcerpc_tree, hf_dcerpc_ver, tvb, offset, 1, hdr.rpc_ver);
+    offset++;
+
+    proto_tree_add_uint (dcerpc_tree, hf_dcerpc_ver_minor, tvb, offset, 1, hdr.rpc_ver_minor);
+    offset++;
+
+    tf = proto_tree_add_uint (dcerpc_tree, hf_dcerpc_packet_type, tvb, offset, 1, hdr.ptype);
+    offset++;
+
+    /* XXX - too much "output noise", removed for now
+    if(hdr.ptype == PDU_BIND || hdr.ptype == PDU_ALTER ||
+        hdr.ptype == PDU_BIND_ACK || hdr.ptype == PDU_ALTER_ACK)
+        expert_add_info_format(pinfo, tf, PI_SEQUENCE, PI_CHAT, "Context change: %s",
+            val_to_str(hdr.ptype, pckt_vals, "(0x%x)"));*/
+    if(hdr.ptype == PDU_BIND_NAK)
+        expert_add_info_format(pinfo, tf, PI_SEQUENCE, PI_WARN, "Bind not acknowledged");
 
     if (tree) {
-               proto_item_append_text(ti, " %s, Fragment:", val_to_str(hdr.ptype, pckt_vals, "Unknown (0x%02x)"));
+       proto_item_append_text(ti, " %s, Fragment:", val_to_str(hdr.ptype, pckt_vals, "Unknown (0x%02x)"));
 
         tf = proto_tree_add_uint (dcerpc_tree, hf_dcerpc_cn_flags, tvb, offset, 1, hdr.flags);
         cn_flags_tree = proto_item_add_subtree (tf, ett_dcerpc_cn_flags);
-        if (cn_flags_tree) {
-            proto_tree_add_boolean (cn_flags_tree, hf_dcerpc_cn_flags_object, tvb, offset, 1, hdr.flags);
-            proto_tree_add_boolean (cn_flags_tree, hf_dcerpc_cn_flags_maybe, tvb, offset, 1, hdr.flags);
-            proto_tree_add_boolean (cn_flags_tree, hf_dcerpc_cn_flags_dne, tvb, offset, 1, hdr.flags);
-            proto_tree_add_boolean (cn_flags_tree, hf_dcerpc_cn_flags_mpx, tvb, offset, 1, hdr.flags);
-            proto_tree_add_boolean (cn_flags_tree, hf_dcerpc_cn_flags_reserved, tvb, offset, 1, hdr.flags);
-            proto_tree_add_boolean (cn_flags_tree, hf_dcerpc_cn_flags_cancel_pending, tvb, offset, 1, hdr.flags);
-            proto_tree_add_boolean (cn_flags_tree, hf_dcerpc_cn_flags_last_frag, tvb, offset, 1, hdr.flags);
-            proto_tree_add_boolean (cn_flags_tree, hf_dcerpc_cn_flags_first_frag, tvb, offset, 1, hdr.flags);
-            if( (hdr.flags & PFC_FIRST_FRAG) && (hdr.flags & PFC_LAST_FRAG) ) {
-                           proto_item_append_text(ti, " Single");
-            } else {
-                if(hdr.flags & PFC_FIRST_FRAG) {
-                           proto_item_append_text(ti, " 1st");
-                }
-                if(hdr.flags & PFC_LAST_FRAG) {
-                           proto_item_append_text(ti, " Last");
-                }
-                if( !(hdr.flags & PFC_FIRST_FRAG) && !(hdr.flags & PFC_LAST_FRAG) ) {
-                           proto_item_append_text(ti, " Mid");
-                }
-            }
+    }
+    proto_tree_add_boolean (cn_flags_tree, hf_dcerpc_cn_flags_object, tvb, offset, 1, hdr.flags);
+    proto_tree_add_boolean (cn_flags_tree, hf_dcerpc_cn_flags_maybe, tvb, offset, 1, hdr.flags);
+    proto_tree_add_boolean (cn_flags_tree, hf_dcerpc_cn_flags_dne, tvb, offset, 1, hdr.flags);
+    proto_tree_add_boolean (cn_flags_tree, hf_dcerpc_cn_flags_mpx, tvb, offset, 1, hdr.flags);
+    proto_tree_add_boolean (cn_flags_tree, hf_dcerpc_cn_flags_reserved, tvb, offset, 1, hdr.flags);
+    proto_tree_add_boolean (cn_flags_tree, hf_dcerpc_cn_flags_cancel_pending, tvb, offset, 1, hdr.flags);
+    proto_tree_add_boolean (cn_flags_tree, hf_dcerpc_cn_flags_last_frag, tvb, offset, 1, hdr.flags);
+    proto_tree_add_boolean (cn_flags_tree, hf_dcerpc_cn_flags_first_frag, tvb, offset, 1, hdr.flags);
+    if( (hdr.flags & PFC_FIRST_FRAG) && (hdr.flags & PFC_LAST_FRAG) ) {
+        proto_item_append_text(ti, " Single");
+    } else {
+        if(hdr.flags & PFC_FIRST_FRAG) {
+            proto_item_append_text(ti, " 1st");
+        }
+        if(hdr.flags & PFC_LAST_FRAG) {
+            proto_item_append_text(ti, " Last");
+        }
+        if( !(hdr.flags & PFC_FIRST_FRAG) && !(hdr.flags & PFC_LAST_FRAG) ) {
+            proto_item_append_text(ti, " Mid");
         }
-        offset++;
+    }
+    offset++;
 
+    if(dcerpc_tree){
         tf = proto_tree_add_bytes (dcerpc_tree, hf_dcerpc_drep, tvb, offset, 4, hdr.drep);
         drep_tree = proto_item_add_subtree (tf, ett_dcerpc_drep);
-        if (drep_tree) {
-            proto_tree_add_uint(drep_tree, hf_dcerpc_drep_byteorder, tvb, offset, 1, hdr.drep[0] >> 4);
-            proto_tree_add_uint(drep_tree, hf_dcerpc_drep_character, tvb, offset, 1, hdr.drep[0] & 0x0f);
-            proto_tree_add_uint(drep_tree, hf_dcerpc_drep_fp, tvb, offset+1, 1, hdr.drep[1]);
-        }
-        offset += sizeof (hdr.drep);
+    }
+    proto_tree_add_uint(drep_tree, hf_dcerpc_drep_byteorder, tvb, offset, 1, hdr.drep[0] >> 4);
+    proto_tree_add_uint(drep_tree, hf_dcerpc_drep_character, tvb, offset, 1, hdr.drep[0] & 0x0f);
+    proto_tree_add_uint(drep_tree, hf_dcerpc_drep_fp, tvb, offset+1, 1, hdr.drep[1]);
+    offset += sizeof (hdr.drep);
 
-        proto_tree_add_uint (dcerpc_tree, hf_dcerpc_cn_frag_len, tvb, offset, 2, hdr.frag_len);
-        offset += 2;
+    proto_tree_add_uint (dcerpc_tree, hf_dcerpc_cn_frag_len, tvb, offset, 2, hdr.frag_len);
+    offset += 2;
 
-        proto_tree_add_uint (dcerpc_tree, hf_dcerpc_cn_auth_len, tvb, offset, 2, hdr.auth_len);
-        offset += 2;
+    proto_tree_add_uint (dcerpc_tree, hf_dcerpc_cn_auth_len, tvb, offset, 2, hdr.auth_len);
+    offset += 2;
 
-        proto_tree_add_uint (dcerpc_tree, hf_dcerpc_cn_call_id, tvb, offset, 4, hdr.call_id);
-        offset += 4;
+    proto_tree_add_uint (dcerpc_tree, hf_dcerpc_cn_call_id, tvb, offset, 4, hdr.call_id);
+    offset += 4;
 
-               proto_item_append_text(ti, ", FragLen: %u, Call: %u", hdr.frag_len, hdr.call_id);
+    if(ti){
+        proto_item_append_text(ti, ", FragLen: %u, Call: %u", hdr.frag_len, hdr.call_id);
     }
 
     /*
@@ -4049,16 +3985,16 @@ dissect_dcerpc_cn (tvbuff_t *tvb, int offset, packet_info *pinfo,
     if (pkt_len != NULL)
         *pkt_len = hdr.frag_len + padding;
 
-       /* The remaining bytes in the current tvb might contain multiple
-        * DCE/RPC fragments, so create a new tvb subset for this fragment.
-        * Only limit the end of the fragment, but not the offset start, 
-        * as the authentication function dissect_dcerpc_cn_auth() will fail 
-        * (and other functions might fail as well) computing the right start 
-        * offset otherwise.
-        * XXX - I don't understand reported_length completely, is this correct here? */
-       fragment_tvb = tvb_new_subset(tvb, 0, 
-               hdr.frag_len + start_offset /* length */, 
-               hdr.frag_len + start_offset /* reported_length */);
+    /* The remaining bytes in the current tvb might contain multiple
+     * DCE/RPC fragments, so create a new tvb subset for this fragment.
+     * Only limit the end of the fragment, but not the offset start,
+     * as the authentication function dissect_dcerpc_cn_auth() will fail
+     * (and other functions might fail as well) computing the right start
+     * offset otherwise.
+     */
+    fragment_tvb = tvb_new_subset(tvb, 0,
+        MIN((hdr.frag_len + (guint) start_offset), tvb_length(tvb)) /* length */,
+        hdr.frag_len + start_offset /* reported_length */);
 
     /*
      * Packet type specific stuff is next.
@@ -4078,7 +4014,7 @@ dissect_dcerpc_cn (tvbuff_t *tvb, int offset, packet_info *pinfo,
         /*
          * Nothing after the common header other than credentials.
          */
-        dissect_dcerpc_cn_auth (fragment_tvb, offset, pinfo, dcerpc_tree, &hdr, TRUE, 
+        dissect_dcerpc_cn_auth (fragment_tvb, offset, pinfo, dcerpc_tree, &hdr, TRUE,
                                &auth_info);
         break;
 
@@ -4104,7 +4040,7 @@ dissect_dcerpc_cn (tvbuff_t *tvb, int offset, packet_info *pinfo,
          * Nothing after the common header other than an authentication
          * verifier.
          */
-        dissect_dcerpc_cn_auth (fragment_tvb, offset, pinfo, dcerpc_tree, &hdr, FALSE, 
+        dissect_dcerpc_cn_auth (fragment_tvb, offset, pinfo, dcerpc_tree, &hdr, FALSE,
                                &auth_info);
         break;
 
@@ -4117,7 +4053,7 @@ dissect_dcerpc_cn (tvbuff_t *tvb, int offset, packet_info *pinfo,
 
     default:
         /* might as well dissect the auth info */
-        dissect_dcerpc_cn_auth (fragment_tvb, offset, pinfo, dcerpc_tree, &hdr, FALSE, 
+        dissect_dcerpc_cn_auth (fragment_tvb, offset, pinfo, dcerpc_tree, &hdr, FALSE,
                                &auth_info);
         break;
     }
@@ -4187,6 +4123,11 @@ dissect_dcerpc_cn_bs_body (tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
             RETHROW;
         } CATCH(ReportedBoundsError) {
             show_reported_bounds_error(tvb, pinfo, tree);
+            /*
+             * Presumably it looked enough like a DCE RPC PDU that we
+             * dissected enough of it to throw an exception.
+             */
+            dcerpc_pdus++;
         } ENDTRY;
 
         if (!dcerpc_pdus) {
@@ -4210,11 +4151,11 @@ dissect_dcerpc_cn_bs_body (tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
             /*
              * Desegmentation required - bail now, but give the user a hint that desegmentation might be done later.
              */
-            proto_tree_add_uint_format(tree, hf_dcerpc_cn_deseg_req, tvb, offset, 
-                0, 
-                tvb_reported_length_remaining(tvb, offset), 
-                "[DCE RPC: %u byte%s left, desegmentation might follow]", 
-                tvb_reported_length_remaining(tvb, offset), 
+            proto_tree_add_uint_format(tree, hf_dcerpc_cn_deseg_req, tvb, offset,
+                0,
+                tvb_reported_length_remaining(tvb, offset),
+                "[DCE RPC: %u byte%s left, desegmentation might follow]",
+                tvb_reported_length_remaining(tvb, offset),
                 plurality(tvb_reported_length_remaining(tvb, offset), "", "s"));
             break;
        }
@@ -4241,6 +4182,13 @@ dissect_dcerpc_cn_smbpipe (tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
        return dissect_dcerpc_cn_bs_body(tvb, pinfo, tree);
 }
 
+static gboolean
+dissect_dcerpc_cn_smb2 (tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
+{
+       pinfo->dcetransporttype=DCE_TRANSPORT_UNKNOWN;
+       return dissect_dcerpc_cn_bs_body(tvb, pinfo, tree);
+}
+
 
 
 static void
@@ -4429,7 +4377,7 @@ dissect_dcerpc_dg_stub (tvbuff_t *tvb, int offset, packet_info *pinfo,
     proto_item *parent_pi;
 
     if (check_col (pinfo->cinfo, COL_INFO))
-        col_append_fstr (pinfo->cinfo, COL_INFO, " opnum: %u len: %u", 
+        col_append_fstr (pinfo->cinfo, COL_INFO, " opnum: %u len: %u",
             di->call_data->opnum, hdr->frag_len );
 
     length = tvb_length_remaining (tvb, offset);
@@ -4559,11 +4507,13 @@ dissect_dcerpc_dg_rqst (tvbuff_t *tvb, int offset, packet_info *pinfo,
        call_value=se_alloc (sizeof (dcerpc_call_value));
        call_value->uuid = hdr->if_id;
        call_value->ver = hdr->if_ver;
+       call_value->object_uuid = hdr->obj_id;
        call_value->opnum = hdr->opnum;
        call_value->req_frame=pinfo->fd->num;
        call_value->req_time=pinfo->fd->abs_ts;
        call_value->rep_frame=0;
        call_value->max_ptr=0;
+       call_value->se_data = NULL;
        call_value->private_data = NULL;
        g_hash_table_insert (dcerpc_dg_calls, call_key, call_value);
 
@@ -4579,10 +4529,12 @@ dissect_dcerpc_dg_rqst (tvbuff_t *tvb, int offset, packet_info *pinfo,
     if (!value) {
         v.uuid = hdr->if_id;
         v.ver = hdr->if_ver;
+        v.object_uuid = hdr->obj_id;
         v.opnum = hdr->opnum;
         v.req_frame = pinfo->fd->num;
         v.rep_frame = 0;
         v.max_ptr = 0;
+        v.se_data=NULL;
         v.private_data=NULL;
         value = &v;
     }
@@ -4642,9 +4594,11 @@ dissect_dcerpc_dg_resp (tvbuff_t *tvb, int offset, packet_info *pinfo,
     if (!value) {
         v.uuid = hdr->if_id;
         v.ver = hdr->if_ver;
+        v.object_uuid = hdr->obj_id;
         v.opnum = hdr->opnum;
         v.req_frame=0;
         v.rep_frame=pinfo->fd->num;
+        v.se_data=NULL;
         v.private_data=NULL;
         value = &v;
     }
@@ -4667,13 +4621,18 @@ dissect_dcerpc_dg_resp (tvbuff_t *tvb, int offset, packet_info *pinfo,
        nstime_delta(&delta_ts, &pinfo->fd->abs_ts, &value->req_time);
        pi = proto_tree_add_time(dcerpc_tree, hf_dcerpc_time, tvb, offset, 0, &delta_ts);
     PROTO_ITEM_SET_GENERATED(pi);
+    } else {
+               pi = proto_tree_add_text(dcerpc_tree,
+                                   tvb, 0, 0, "No request to this DCE/RPC call found");
+               expert_add_info_format(pinfo, pi, PI_SEQUENCE, PI_NOTE,
+                       "No request to this DCE/RPC call found");
     }
     dissect_dcerpc_dg_stub (tvb, offset, pinfo, dcerpc_tree, tree, hdr, di);
 }
 
 static void
 dissect_dcerpc_dg_ping_ack (tvbuff_t *tvb, int offset, packet_info *pinfo,
-                        proto_tree *dcerpc_tree, proto_tree *tree,
+                        proto_tree *dcerpc_tree,
                         e_dce_dg_common_hdr_t *hdr, conversation_t *conv)
 {
     proto_item *parent_pi;
@@ -4723,8 +4682,8 @@ dissect_dcerpc_dg (tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
     int offset = 0;
     conversation_t *conv;
     int auth_level;
-    char uuid_str[DCERPC_UUID_STR_LEN]; 
-    int uuid_str_len;
+    char *uuid_str;
+    const char *uuid_name = NULL;
 
     /*
      * Check if this looks like a CL DCERPC call.  All dg packets
@@ -4745,8 +4704,8 @@ dissect_dcerpc_dg (tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
     if (hdr.ptype > 19)
         return FALSE;
 
-    /* flags1 has bit 1 and 8 as reserved so if any of them are set, it is 
-       probably not a DCE/RPC packet 
+    /* flags1 has bit 1 and 8 as reserved so if any of them are set, it is
+       probably not a DCE/RPC packet
      */
     hdr.flags1 = tvb_get_guint8 (tvb, offset++);
     if(hdr.flags1&0x81)
@@ -4797,7 +4756,7 @@ dissect_dcerpc_dg (tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
         ti = proto_tree_add_item (tree, proto_dcerpc, tvb, 0, -1, FALSE);
         if (ti) {
             dcerpc_tree = proto_item_add_subtree(ti, ett_dcerpc);
-                   proto_item_append_text(ti, " %s, Seq: %u, Serial: %u, Frag: %u, FragLen: %u", 
+                   proto_item_append_text(ti, " %s, Seq: %u, Serial: %u, Frag: %u, FragLen: %u",
                 val_to_str(hdr.ptype, pckt_vals, "Unknown (0x%02x)"),
                 hdr.seqnum, hdr.serial_hi*256+hdr.serial_lo,
                 hdr.frag_num, hdr.frag_len);
@@ -4826,7 +4785,7 @@ dissect_dcerpc_dg (tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
             proto_tree_add_boolean (dg_flags1_tree, hf_dcerpc_dg_flags1_last_frag, tvb, offset, 1, hdr.flags1);
             proto_tree_add_boolean (dg_flags1_tree, hf_dcerpc_dg_flags1_rsrvd_01, tvb, offset, 1, hdr.flags1);
             if(hdr.flags1) {
-                       proto_item_append_text(tf, " %s%s%s%s%s%s", 
+                       proto_item_append_text(tf, " %s%s%s%s%s%s",
                     (hdr.flags1 & PFCL1_BROADCAST) ? "\"Broadcast\" " : "",
                     (hdr.flags1 & PFCL1_IDEMPOTENT) ? "\"Idempotent\" " : "",
                     (hdr.flags1 & PFCL1_MAYBE) ? "\"Maybe\" " : "",
@@ -4851,7 +4810,7 @@ dissect_dcerpc_dg (tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
             proto_tree_add_boolean (dg_flags2_tree, hf_dcerpc_dg_flags2_cancel_pending, tvb, offset, 1, hdr.flags2);
             proto_tree_add_boolean (dg_flags2_tree, hf_dcerpc_dg_flags2_rsrvd_01, tvb, offset, 1, hdr.flags2);
             if(hdr.flags2) {
-                       proto_item_append_text(tf, " %s", 
+                       proto_item_append_text(tf, " %s",
                     (hdr.flags2 & PFCL2_CANCEL_PENDING) ? "\"Cancel Pending\" " : "");
             }
         }
@@ -4865,7 +4824,7 @@ dissect_dcerpc_dg (tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
             proto_tree_add_uint(drep_tree, hf_dcerpc_drep_byteorder, tvb, offset, 1, hdr.drep[0] >> 4);
             proto_tree_add_uint(drep_tree, hf_dcerpc_drep_character, tvb, offset, 1, hdr.drep[0] & 0x0f);
             proto_tree_add_uint(drep_tree, hf_dcerpc_drep_fp, tvb, offset+1, 1, hdr.drep[1]);
-                   proto_item_append_text(tf, " (Order: %s, Char: %s, Float: %s)", 
+                   proto_item_append_text(tf, " (Order: %s, Char: %s, Float: %s)",
                 val_to_str(hdr.drep[0] >> 4, drep_byteorder_vals, "Unknown"),
                 val_to_str(hdr.drep[0] & 0x0f, drep_character_vals, "Unknown"),
                 val_to_str(hdr.drep[1], drep_fp_vals, "Unknown"));
@@ -4878,62 +4837,29 @@ dissect_dcerpc_dg (tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
     offset++;
 
     if (tree) {
-        /* XXX - use "dissect_ndr_uuid_t()"? */
-       uuid_str_len = g_snprintf(uuid_str, DCERPC_UUID_STR_LEN, 
-                                "%08x-%04x-%04x-%02x%02x-%02x%02x%02x%02x%02x%02x",
-                                hdr.obj_id.Data1, hdr.obj_id.Data2, hdr.obj_id.Data3,
-                                hdr.obj_id.Data4[0],
-                                hdr.obj_id.Data4[1],
-                                hdr.obj_id.Data4[2],
-                                hdr.obj_id.Data4[3],
-                                hdr.obj_id.Data4[4],
-                                hdr.obj_id.Data4[5],
-                                hdr.obj_id.Data4[6],
-                                hdr.obj_id.Data4[7]);
-        if (uuid_str_len == -1 || uuid_str_len >= DCERPC_UUID_STR_LEN)
-               memset(uuid_str, 0, DCERPC_UUID_STR_LEN);
-        proto_tree_add_string_format (dcerpc_tree, hf_dcerpc_obj_id, tvb,
-                                      offset, 16, uuid_str, "Object UUID: %s", uuid_str);
+        proto_tree_add_guid_format (dcerpc_tree, hf_dcerpc_obj_id, tvb,
+            offset, 16, (e_guid_t *) &hdr.obj_id, "Object UUID: %s",
+            guid_to_str((e_guid_t *) &hdr.obj_id));
     }
     offset += 16;
 
     if (tree) {
-        /* XXX - use "dissect_ndr_uuid_t()"? */
-       uuid_str_len = g_snprintf(uuid_str, DCERPC_UUID_STR_LEN, 
-                                "%08x-%04x-%04x-%02x%02x-%02x%02x%02x%02x%02x%02x",
-                                hdr.if_id.Data1, hdr.if_id.Data2, hdr.if_id.Data3,
-                                hdr.if_id.Data4[0],
-                                hdr.if_id.Data4[1],
-                                hdr.if_id.Data4[2],
-                                hdr.if_id.Data4[3],
-                                hdr.if_id.Data4[4],
-                                hdr.if_id.Data4[5],
-                                hdr.if_id.Data4[6],
-                                hdr.if_id.Data4[7]);
-        if (uuid_str_len == -1 || uuid_str_len >= DCERPC_UUID_STR_LEN)
-               memset(uuid_str, 0, DCERPC_UUID_STR_LEN);
-        proto_tree_add_string_format (dcerpc_tree, hf_dcerpc_dg_if_id, tvb,
-                                      offset, 16, uuid_str, "Interface: %s", uuid_str);
+        uuid_str = guid_to_str((e_guid_t*)&hdr.if_id);
+        uuid_name = guids_get_uuid_name(&hdr.if_id);
+        if(uuid_name) {
+                 proto_tree_add_guid_format (dcerpc_tree, hf_dcerpc_dg_if_id, tvb,
+                                        offset, 16, (e_guid_t *) &hdr.if_id, "Interface: %s UUID: %s", uuid_name, uuid_str);
+        } else {
+          proto_tree_add_guid_format (dcerpc_tree, hf_dcerpc_dg_if_id, tvb,
+                                        offset, 16, (e_guid_t *) &hdr.if_id, "Interface UUID: %s", uuid_str);
+        }
     }
     offset += 16;
 
     if (tree) {
-        /* XXX - use "dissect_ndr_uuid_t()"? */
-       uuid_str_len = g_snprintf(uuid_str, DCERPC_UUID_STR_LEN, 
-                                "%08x-%04x-%04x-%02x%02x-%02x%02x%02x%02x%02x%02x",
-                                hdr.act_id.Data1, hdr.act_id.Data2, hdr.act_id.Data3,
-                                hdr.act_id.Data4[0],
-                                hdr.act_id.Data4[1],
-                                hdr.act_id.Data4[2],
-                                hdr.act_id.Data4[3],
-                                hdr.act_id.Data4[4],
-                                hdr.act_id.Data4[5],
-                                hdr.act_id.Data4[6],
-                                hdr.act_id.Data4[7]);
-        if (uuid_str_len == -1 || uuid_str_len >= DCERPC_UUID_STR_LEN)
-               memset(uuid_str, 0, DCERPC_UUID_STR_LEN);
-        proto_tree_add_string_format (dcerpc_tree, hf_dcerpc_dg_act_id, tvb,
-                                      offset, 16, uuid_str, "Activity: %s", uuid_str);
+        proto_tree_add_guid_format (dcerpc_tree, hf_dcerpc_dg_act_id, tvb,
+                                      offset, 16, (e_guid_t *) &hdr.act_id, "Activity: %s",
+                                      guid_to_str((e_guid_t *) &hdr.act_id));
     }
     offset += 16;
 
@@ -5069,7 +4995,10 @@ dissect_dcerpc_dg (tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
         break;
 
     case PDU_FACK:
-        dissect_dcerpc_dg_fack (tvb, offset, pinfo, dcerpc_tree, &hdr);
+        /* Body is optional */
+        /* XXX - we assume "frag_len" is the length of the body */
+        if (hdr.frag_len != 0)
+            dissect_dcerpc_dg_fack (tvb, offset, pinfo, dcerpc_tree, &hdr);
         break;
 
     case PDU_REJECT:
@@ -5088,7 +5017,7 @@ dissect_dcerpc_dg (tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
     /* these requests have no body */
     case PDU_ACK:
     case PDU_PING:
-        dissect_dcerpc_dg_ping_ack (tvb, offset, pinfo, dcerpc_tree, tree, &hdr, conv);
+        dissect_dcerpc_dg_ping_ack (tvb, offset, pinfo, dcerpc_tree, &hdr, conv);
         break;
     case PDU_WORKING:
     default:
@@ -5105,7 +5034,7 @@ dcerpc_init_protocol (void)
        if (dcerpc_binds){
                g_hash_table_destroy (dcerpc_binds);
                dcerpc_binds=NULL;
-       } 
+       }
        if(!dcerpc_binds){
                dcerpc_binds = g_hash_table_new (dcerpc_bind_hash, dcerpc_bind_equal);
        }
@@ -5189,20 +5118,26 @@ proto_register_dcerpc (void)
           { "Assoc Group", "dcerpc.cn_assoc_group", FT_UINT32, BASE_HEX, NULL, 0x0, "", HFILL }},
         { &hf_dcerpc_cn_num_ctx_items,
           { "Num Ctx Items", "dcerpc.cn_num_ctx_items", FT_UINT8, BASE_DEC, NULL, 0x0, "", HFILL }},
+        { &hf_dcerpc_cn_ctx_item,
+          { "Ctx Item", "dcerpc.cn_ctx_item", FT_NONE, BASE_NONE, NULL, 0x0, "", HFILL }},
         { &hf_dcerpc_cn_ctx_id,
           { "Context ID", "dcerpc.cn_ctx_id", FT_UINT16, BASE_DEC, NULL, 0x0, "", HFILL }},
         { &hf_dcerpc_cn_num_trans_items,
           { "Num Trans Items", "dcerpc.cn_num_trans_items", FT_UINT8, BASE_DEC, NULL, 0x0, "", HFILL }},
+        { &hf_dcerpc_cn_bind_abstract_syntax,
+          { "Abstract Syntax", "dcerpc.cn_bind_abstract_syntax", FT_NONE, BASE_NONE, NULL, 0x0, "", HFILL }},
         { &hf_dcerpc_cn_bind_if_id,
-          { "Interface UUID", "dcerpc.cn_bind_to_uuid", FT_STRING, BASE_NONE, NULL, 0x0, "", HFILL }},
+          { "Interface UUID", "dcerpc.cn_bind_to_uuid", FT_GUID, BASE_NONE, NULL, 0x0, "", HFILL }},
         { &hf_dcerpc_cn_bind_if_ver,
           { "Interface Ver", "dcerpc.cn_bind_if_ver", FT_UINT16, BASE_DEC, NULL, 0x0, "", HFILL }},
         { &hf_dcerpc_cn_bind_if_ver_minor,
           { "Interface Ver Minor", "dcerpc.cn_bind_if_ver_minor", FT_UINT16, BASE_DEC, NULL, 0x0, "", HFILL }},
+        { &hf_dcerpc_cn_bind_trans_syntax,
+          { "Transfer Syntax", "dcerpc.cn_bind_trans", FT_NONE, BASE_NONE, NULL, 0x0, "", HFILL }},
         { &hf_dcerpc_cn_bind_trans_id,
-          { "Transfer Syntax", "dcerpc.cn_bind_trans_id", FT_STRING, BASE_NONE, NULL, 0x0, "", HFILL }},
+          { "ID", "dcerpc.cn_bind_trans_id", FT_GUID, BASE_NONE, NULL, 0x0, "", HFILL }},
         { &hf_dcerpc_cn_bind_trans_ver,
-          { "Syntax ver", "dcerpc.cn_bind_trans_ver", FT_UINT32, BASE_DEC, NULL, 0x0, "", HFILL }},
+          { "ver", "dcerpc.cn_bind_trans_ver", FT_UINT32, BASE_DEC, NULL, 0x0, "", HFILL }},
         { &hf_dcerpc_cn_alloc_hint,
           { "Alloc hint", "dcerpc.cn_alloc_hint", FT_UINT32, BASE_DEC, NULL, 0x0, "", HFILL }},
         { &hf_dcerpc_cn_sec_addr_len,
@@ -5216,7 +5151,7 @@ proto_register_dcerpc (void)
         { &hf_dcerpc_cn_ack_reason,
           { "Ack reason", "dcerpc.cn_ack_reason", FT_UINT16, BASE_DEC, VALS(p_provider_reason_vals), 0x0, "", HFILL }},
         { &hf_dcerpc_cn_ack_trans_id,
-          { "Transfer Syntax", "dcerpc.cn_ack_trans_id", FT_STRING, BASE_NONE, NULL, 0x0, "", HFILL }},
+          { "Transfer Syntax", "dcerpc.cn_ack_trans_id", FT_GUID, BASE_NONE, NULL, 0x0, "", HFILL }},
         { &hf_dcerpc_cn_ack_trans_ver,
           { "Syntax ver", "dcerpc.cn_ack_trans_ver", FT_UINT32, BASE_DEC, NULL, 0x0, "", HFILL }},
         { &hf_dcerpc_cn_reject_reason,
@@ -5306,11 +5241,11 @@ proto_register_dcerpc (void)
         { &hf_dcerpc_krb5_av_key_auth_verifier,
           { "Authentication Verifier", "dcerpc.krb5_av.auth_verifier", FT_BYTES, BASE_NONE, NULL, 0x0, "", HFILL }},
         { &hf_dcerpc_obj_id,
-          { "Object", "dcerpc.obj_id", FT_STRING, BASE_NONE, NULL, 0x0, "", HFILL }},
+          { "Object", "dcerpc.obj_id", FT_GUID, BASE_NONE, NULL, 0x0, "", HFILL }},
         { &hf_dcerpc_dg_if_id,
-          { "Interface", "dcerpc.dg_if_id", FT_STRING, BASE_NONE, NULL, 0x0, "", HFILL }},
+          { "Interface", "dcerpc.dg_if_id", FT_GUID, BASE_NONE, NULL, 0x0, "", HFILL }},
         { &hf_dcerpc_dg_act_id,
-          { "Activity", "dcerpc.dg_act_id", FT_STRING, BASE_NONE, NULL, 0x0, "", HFILL }},
+          { "Activity", "dcerpc.dg_act_id", FT_GUID, BASE_NONE, NULL, 0x0, "", HFILL }},
         { &hf_dcerpc_opnum,
           { "Opnum", "dcerpc.opnum", FT_UINT16, BASE_DEC, NULL, 0x0, "", HFILL }},
 
@@ -5358,7 +5293,7 @@ proto_register_dcerpc (void)
 
        { &hf_dcerpc_array_buffer,
          { "Buffer", "dcerpc.array.buffer", FT_BYTES, BASE_NONE, NULL, 0x0, "Buffer: Buffer containing elements of the array", HFILL }},
-               
+
         { &hf_dcerpc_op,
           { "Operation", "dcerpc.op", FT_UINT16, BASE_DEC, NULL, 0x0, "", HFILL }},
 
@@ -5371,34 +5306,34 @@ proto_register_dcerpc (void)
          NULL, 0x0, "DCE/RPC Fragment", HFILL }},
 
        { &hf_dcerpc_fragment_overlap,
-         { "Fragment overlap", "dcerpc.fragment.overlap", FT_BOOLEAN, BASE_NONE, 
+         { "Fragment overlap", "dcerpc.fragment.overlap", FT_BOOLEAN, BASE_NONE,
       NULL, 0x0, "Fragment overlaps with other fragments", HFILL }},
 
        { &hf_dcerpc_fragment_overlap_conflict,
-         { "Conflicting data in fragment overlap", "dcerpc.fragment.overlap.conflict", FT_BOOLEAN, BASE_NONE, 
+         { "Conflicting data in fragment overlap", "dcerpc.fragment.overlap.conflict", FT_BOOLEAN, BASE_NONE,
       NULL, 0x0, "Overlapping fragments contained conflicting data", HFILL }},
 
        { &hf_dcerpc_fragment_multiple_tails,
-         { "Multiple tail fragments found", "dcerpc.fragment.multipletails", FT_BOOLEAN, BASE_NONE, 
+         { "Multiple tail fragments found", "dcerpc.fragment.multipletails", FT_BOOLEAN, BASE_NONE,
       NULL, 0x0, "Several tails were found when defragmenting the packet", HFILL }},
 
        { &hf_dcerpc_fragment_too_long_fragment,
-         { "Fragment too long", "dcerpc.fragment.toolongfragment", FT_BOOLEAN, BASE_NONE, 
+         { "Fragment too long", "dcerpc.fragment.toolongfragment", FT_BOOLEAN, BASE_NONE,
       NULL, 0x0, "Fragment contained data past end of packet", HFILL }},
 
        { &hf_dcerpc_fragment_error,
-         { "Defragmentation error", "dcerpc.fragment.error", FT_FRAMENUM, BASE_NONE, 
+         { "Defragmentation error", "dcerpc.fragment.error", FT_FRAMENUM, BASE_NONE,
       NULL, 0x0, "Defragmentation error due to illegal fragments", HFILL }},
 
-       { &hf_dcerpc_time, 
-         { "Time from request", "dcerpc.time", FT_RELATIVE_TIME, BASE_NONE, 
+       { &hf_dcerpc_time,
+         { "Time from request", "dcerpc.time", FT_RELATIVE_TIME, BASE_NONE,
       NULL, 0, "Time between Request and Response for DCE-RPC calls", HFILL }},
 
        { &hf_dcerpc_reassembled_in,
-      { "Reassembled PDU in frame", "dcerpc.reassembled_in", FT_FRAMENUM, BASE_NONE, 
+      { "Reassembled PDU in frame", "dcerpc.reassembled_in", FT_FRAMENUM, BASE_NONE,
       NULL, 0x0, "The DCE/RPC PDU is completely reassembled in the packet with this number", HFILL }},
 
-       { &hf_dcerpc_unknown_if_id, 
+       { &hf_dcerpc_unknown_if_id,
          { "Unknown DCERPC interface id", "dcerpc.unknown_if_id", FT_BOOLEAN, BASE_NONE, NULL, 0x0, "", HFILL }},
    };
     static gint *ett[] = {
@@ -5406,6 +5341,7 @@ proto_register_dcerpc (void)
         &ett_dcerpc_cn_flags,
         &ett_dcerpc_cn_ctx,
         &ett_dcerpc_cn_iface,
+        &ett_dcerpc_cn_trans_syntax,
         &ett_dcerpc_drep,
         &ett_dcerpc_dg_flags1,
         &ett_dcerpc_dg_flags2,
@@ -5436,7 +5372,7 @@ proto_register_dcerpc (void)
     register_init_routine(dcerpc_reassemble_init);
     dcerpc_uuids = g_hash_table_new (dcerpc_uuid_hash, dcerpc_uuid_equal);
     dcerpc_tap=register_tap("dcerpc");
-    
+
     g_hook_list_init(&dcerpc_hooks_init_protos, sizeof(GHook));
 }
 
@@ -5447,6 +5383,7 @@ proto_reg_handoff_dcerpc (void)
     heur_dissector_add ("netbios", dissect_dcerpc_cn_pk, proto_dcerpc);
     heur_dissector_add ("udp", dissect_dcerpc_dg, proto_dcerpc);
     heur_dissector_add ("smb_transact", dissect_dcerpc_cn_smbpipe, proto_dcerpc);
+    heur_dissector_add ("smb2_heur_subdissectors", dissect_dcerpc_cn_smb2, proto_dcerpc);
     heur_dissector_add ("http", dissect_dcerpc_cn_bs, proto_dcerpc);
     dcerpc_smb_init(proto_dcerpc);
 }