From Gisle Vanem:
[obnox/wireshark/wip.git] / epan / dissectors / packet-kerberos.c
index e1b339c0ca17562b0b9572f84b4c895576be8c5d..123087097d2d6b86dcff92c5f1da6eb76f806807 100644 (file)
@@ -25,8 +25,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
@@ -60,6 +60,7 @@
 
 #include <stdio.h>
 #include <string.h>
+#include <glib.h>
 #include <ctype.h>
 
 #ifdef HAVE_LIBNETTLE
 #include <nettle/des.h>
 #include <nettle/cbc.h>
 #endif
-#include "crypt-md5.h"
+#include <epan/crypt/crypt-md5.h>
 #include <sys/stat.h>  /* For keyfile manipulation */
 #endif
 
-#include <glib.h>
-
 #include <epan/packet.h>
 
 #include <epan/strutil.h>
 
 #include <epan/conversation.h>
+#include <epan/emem.h>
 #include <epan/dissectors/packet-kerberos.h>
 #include <epan/dissectors/packet-netbios.h>
 #include <epan/dissectors/packet-tcp.h>
@@ -95,7 +95,8 @@
 #include <epan/dissectors/packet-dcerpc.h>
 
 #include <epan/dissectors/packet-gssapi.h>
-#include <epan/emem.h>
+
+#include <wiretap/file_util.h>
 
 #define UDP_PORT_KERBEROS              88
 #define TCP_PORT_KERBEROS              88
@@ -134,7 +135,9 @@ static gint hf_krb_PAC_CREDENTIAL_TYPE = -1;
 static gint hf_krb_PAC_SERVER_CHECKSUM = -1;
 static gint hf_krb_PAC_PRIVSVR_CHECKSUM = -1;
 static gint hf_krb_PAC_CLIENT_INFO_TYPE = -1;
+static gint hf_krb_PAC_CONSTRAINED_DELEGATION = -1;
 static gint hf_krb_encrypted_PA_ENC_TIMESTAMP = -1;
+static gint hf_krb_encrypted_EncKrbCredPart = -1;
 static gint hf_krb_checksum_checksum = -1;
 static gint hf_krb_encrypted_PRIV = -1;
 static gint hf_krb_encrypted_Ticket_data = -1;
@@ -148,8 +151,11 @@ static gint hf_krb_etype_info2_s2kparams = -1;
 static gint hf_krb_SAFE_BODY_user_data = -1;
 static gint hf_krb_PRIV_BODY_user_data = -1;
 static gint hf_krb_realm = -1;
+static gint hf_krb_srealm = -1;
+static gint hf_krb_prealm = -1;
 static gint hf_krb_crealm = -1;
 static gint hf_krb_sname = -1;
+static gint hf_krb_pname = -1;
 static gint hf_krb_cname = -1;
 static gint hf_krb_name_string = -1;
 static gint hf_krb_provsrv_location = -1;
@@ -184,6 +190,8 @@ static gint hf_krb_adtype = -1;
 static gint hf_krb_advalue = -1;
 static gint hf_krb_etype = -1;
 static gint hf_krb_etypes = -1;
+static gint hf_krb_KrbCredInfos = -1;
+static gint hf_krb_sq_tickets = -1;
 static gint hf_krb_LastReqs = -1;
 static gint hf_krb_IF_RELEVANT = -1;
 static gint hf_krb_addr_type = -1;
@@ -201,11 +209,14 @@ static gint hf_krb_seq_number = -1;
 static gint hf_krb_EncTicketPart = -1;
 static gint hf_krb_EncAPRepPart = -1;
 static gint hf_krb_EncKrbPrivPart = -1;
+static gint hf_krb_EncKrbCredPart = -1;
 static gint hf_krb_EncKDCRepPart = -1;
 static gint hf_krb_LastReq = -1;
 static gint hf_krb_Authenticator = -1;
 static gint hf_krb_Checksum = -1;
 static gint hf_krb_s_address = -1;
+static gint hf_krb_r_address = -1;
+static gint hf_krb_KrbCredInfo = -1;
 static gint hf_krb_HostAddress = -1;
 static gint hf_krb_HostAddresses = -1;
 static gint hf_krb_APOptions = -1;
@@ -233,6 +244,7 @@ static gint hf_krb_KDCOptions_proxy = -1;
 static gint hf_krb_KDCOptions_allow_postdate = -1;
 static gint hf_krb_KDCOptions_postdated = -1;
 static gint hf_krb_KDCOptions_renewable = -1;
+static gint hf_krb_KDCOptions_constrained_delegation = -1;
 static gint hf_krb_KDCOptions_canonicalize = -1;
 static gint hf_krb_KDCOptions_opt_hardware_auth = -1;
 static gint hf_krb_KDCOptions_disable_transited_check = -1;
@@ -242,10 +254,25 @@ static gint hf_krb_KDCOptions_renew = -1;
 static gint hf_krb_KDCOptions_validate = -1;
 static gint hf_krb_KDC_REQ_BODY = -1;
 static gint hf_krb_PRIV_BODY = -1;
+static gint hf_krb_CRED_BODY = -1;
 static gint hf_krb_ENC_PRIV = -1;
 static gint hf_krb_authenticator_enc = -1;
+static gint hf_krb_CRED_enc = -1;
 static gint hf_krb_ticket_enc = -1;
 static gint hf_krb_e_checksum = -1;
+static gint hf_krb_gssapi_len = -1;
+static gint hf_krb_gssapi_bnd = -1;
+static gint hf_krb_gssapi_dlgopt = -1;
+static gint hf_krb_gssapi_dlglen = -1;
+static gint hf_krb_gssapi_c_flag_deleg = -1;
+static gint hf_krb_gssapi_c_flag_mutual = -1;
+static gint hf_krb_gssapi_c_flag_replay = -1;
+static gint hf_krb_gssapi_c_flag_sequence = -1;
+static gint hf_krb_gssapi_c_flag_conf = -1;
+static gint hf_krb_gssapi_c_flag_integ = -1;
+static gint hf_krb_gssapi_c_flag_dce_style = -1;
+static gint hf_krb_smb_nt_status = -1;
+static gint hf_krb_smb_unknown = -1;
 
 static gint ett_krb_kerberos = -1;
 static gint ett_krb_TransitedEncoding = -1;
@@ -254,10 +281,12 @@ static gint ett_krb_PAC_CREDENTIAL_TYPE = -1;
 static gint ett_krb_PAC_SERVER_CHECKSUM = -1;
 static gint ett_krb_PAC_PRIVSVR_CHECKSUM = -1;
 static gint ett_krb_PAC_CLIENT_INFO_TYPE = -1;
+static gint ett_krb_PAC_CONSTRAINED_DELEGATION = -1;
 static gint ett_krb_KDC_REP_enc = -1;
 static gint ett_krb_EncTicketPart = -1;
 static gint ett_krb_EncAPRepPart = -1;
 static gint ett_krb_EncKrbPrivPart = -1;
+static gint ett_krb_EncKrbCredPart = -1;
 static gint ett_krb_EncKDCRepPart = -1;
 static gint ett_krb_LastReq = -1;
 static gint ett_krb_Authenticator = -1;
@@ -266,18 +295,24 @@ static gint ett_krb_key = -1;
 static gint ett_krb_subkey = -1;
 static gint ett_krb_AuthorizationData = -1;
 static gint ett_krb_sname = -1;
+static gint ett_krb_pname = -1;
 static gint ett_krb_cname = -1;
 static gint ett_krb_AP_REP_enc = -1;
 static gint ett_krb_padata = -1;
 static gint ett_krb_etypes = -1;
+static gint ett_krb_KrbCredInfos = -1;
+static gint ett_krb_sq_tickets = -1;
 static gint ett_krb_LastReqs = -1;
 static gint ett_krb_IF_RELEVANT = -1;
 static gint ett_krb_PA_DATA_tree = -1;
 static gint ett_krb_PAC = -1;
 static gint ett_krb_s_address = -1;
+static gint ett_krb_r_address = -1;
+static gint ett_krb_KrbCredInfo = -1;
 static gint ett_krb_HostAddress = -1;
 static gint ett_krb_HostAddresses = -1;
 static gint ett_krb_authenticator_enc = -1;
+static gint ett_krb_CRED_enc = -1;
 static gint ett_krb_AP_Options = -1;
 static gint ett_krb_KDC_Options = -1;
 static gint ett_krb_Ticket_Flags = -1;
@@ -285,6 +320,7 @@ static gint ett_krb_request = -1;
 static gint ett_krb_recordmark = -1;
 static gint ett_krb_ticket = -1;
 static gint ett_krb_ticket_enc = -1;
+static gint ett_krb_CRED = -1;
 static gint ett_krb_PRIV = -1;
 static gint ett_krb_PRIV_enc = -1;
 static gint ett_krb_e_checksum = -1;
@@ -292,7 +328,7 @@ static gint ett_krb_e_checksum = -1;
 guint32 krb5_errorcode;
 
 
-dissector_handle_t krb4_handle=NULL;
+static dissector_handle_t krb4_handle=NULL;
 
 static gboolean do_col_info;
 
@@ -365,7 +401,7 @@ read_keytab_file(const char *filename, krb5_context *context)
        krb5_kt_cursor cursor;
        enc_key_t *new_key;
 
-       /* should use a file in the ethereal users dir */
+       /* should use a file in the wireshark users dir */
        ret = krb5_kt_resolve(*context, filename, &keytab);
        if(ret){
                fprintf(stderr, "KERBEROS ERROR: Could not open keytab file :%s\n",filename);
@@ -389,11 +425,14 @@ read_keytab_file(const char *filename, krb5_context *context)
 
                        /* generate origin string, describing where this key came from */
                        pos=new_key->key_origin;
-                       pos+=g_snprintf(pos, KRB_MAX_ORIG_LEN, "keytab principal ");
+                       pos+=MIN(KRB_MAX_ORIG_LEN,
+                                g_snprintf(pos, KRB_MAX_ORIG_LEN, "keytab principal "));
                        for(i=0;i<key.principal->length;i++){
-                               pos+=g_snprintf(pos, KRB_MAX_ORIG_LEN-(pos-new_key->key_origin), "%s%s",(i?"/":""),(key.principal->data[i]).data);
+                               pos+=MIN(KRB_MAX_ORIG_LEN-(pos-new_key->key_origin),
+                                        g_snprintf(pos, KRB_MAX_ORIG_LEN-(pos-new_key->key_origin), "%s%s",(i?"/":""),(key.principal->data[i]).data));
                        }
-                       pos+=g_snprintf(pos, KRB_MAX_ORIG_LEN-(pos-new_key->key_origin), "@%s",key.principal->realm.data);
+                       pos+=MIN(KRB_MAX_ORIG_LEN-(pos-new_key->key_origin),
+                                g_snprintf(pos, KRB_MAX_ORIG_LEN-(pos-new_key->key_origin), "@%s",key.principal->realm.data));
                        *pos=0;
 /*printf("added key for principal :%s\n", new_key->key_origin);*/
                        new_key->keytype=key.key.enctype;
@@ -485,7 +524,7 @@ read_keytab_file(const char *filename, krb5_context *context)
        krb5_kt_cursor cursor;
        enc_key_t *new_key;
 
-       /* should use a file in the ethereal users dir */
+       /* should use a file in the wireshark users dir */
        ret = krb5_kt_resolve(*context, filename, &keytab);
        if(ret){
                fprintf(stderr, "KERBEROS ERROR: Could not open keytab file :%s\n",filename);
@@ -509,11 +548,14 @@ read_keytab_file(const char *filename, krb5_context *context)
 
                        /* generate origin string, describing where this key came from */
                        pos=new_key->key_origin;
-                       pos+=g_snprintf(pos, KRB_MAX_ORIG_LEN, "keytab principal ");
+                       pos+=MIN(KRB_MAX_ORIG_LEN,
+                                g_snprintf(pos, KRB_MAX_ORIG_LEN, "keytab principal "));
                        for(i=0;i<key.principal->name.name_string.len;i++){
-                               pos+=g_snprintf(pos, KRB_MAX_ORIG_LEN-(pos-new_key->key_origin), "%s%s",(i?"/":""),key.principal->name.name_string.val[i]);
+                               pos+=MIN(KRB_MAX_ORIG_LEN-(pos-new_key->key_origin),
+                                        g_snprintf(pos, KRB_MAX_ORIG_LEN-(pos-new_key->key_origin), "%s%s",(i?"/":""),key.principal->name.name_string.val[i]));
                        }
-                       pos+=g_snprintf(pos, KRB_MAX_ORIG_LEN-(pos-new_key->key_origin), "@%s",key.principal->realm);
+                       pos+=MIN(KRB_MAX_ORIG_LEN-(pos-new_key->key_origin),
+                                g_snprintf(pos, KRB_MAX_ORIG_LEN-(pos-new_key->key_origin), "@%s",key.principal->realm));
                        *pos=0;
                        new_key->keytype=key.keyblock.keytype;
                        new_key->keylength=key.keyblock.keyvalue.length;
@@ -679,7 +721,7 @@ read_keytab_file(const char *service_key_file)
                        }
                }
 
-               skf = fopen(service_key_file, "rb");
+               skf = eth_fopen(service_key_file, "rb");
                if (! skf) return;
 
                while (fread(buf, SERVICE_KEY_SIZE, 1, skf) == 1) {
@@ -817,7 +859,8 @@ g_warning("woohoo decrypted keytype:%d in frame:%d\n", keytype, pinfo->fd->num);
 #define KRB5_MSG_ENC_AS_REP_PART       25      /* EncASRepPart */
 #define KRB5_MSG_ENC_TGS_REP_PART      26      /* EncTGSRepPart */
 #define KRB5_MSG_ENC_AP_REP_PART       27      /* EncAPRepPart */
-#define KRB5_MSG_ENC_KRB_PRIV_PART             28      /* EncAPRepPart */
+#define KRB5_MSG_ENC_KRB_PRIV_PART             28      /* EncKrbPrivPart */
+#define KRB5_MSG_ENC_KRB_CRED_PART             29      /* EncKrbCredPart */
 #define KRB5_MSG_ERROR                 30      /* KRB-ERROR type */
 
 /* address type constants */
@@ -1111,12 +1154,14 @@ static const value_string krb5_error_codes[] = {
 #define PAC_SERVER_CHECKSUM    6
 #define PAC_PRIVSVR_CHECKSUM   7
 #define PAC_CLIENT_INFO_TYPE   10
+#define PAC_CONSTRAINED_DELEGATION 11
 static const value_string w2k_pac_types[] = {
     { PAC_LOGON_INFO           , "Logon Info" },
     { PAC_CREDENTIAL_TYPE      , "Credential Type" },
     { PAC_SERVER_CHECKSUM      , "Server Checksum" },
     { PAC_PRIVSVR_CHECKSUM     , "Privsvr Checksum" },
     { PAC_CLIENT_INFO_TYPE     , "Client Info Type" },
+    { PAC_CONSTRAINED_DELEGATION, "Constrained Delegation" },
     { 0, NULL },
 };
 
@@ -1303,6 +1348,7 @@ static const value_string krb5_msg_types[] = {
        { KRB5_MSG_ENC_TGS_REP_PART,    "EncTGSRepPart" },
        { KRB5_MSG_ENC_AP_REP_PART,     "EncAPRepPart" },
        { KRB5_MSG_ENC_KRB_PRIV_PART,   "EncKrbPrivPart" },
+       { KRB5_MSG_ENC_KRB_CRED_PART,   "EncKrbCredPart" },
        { KRB5_MSG_ERROR,               "KRB-ERROR" },
         { 0, NULL },
 };
@@ -1316,6 +1362,7 @@ static int dissect_krb5_Authenticator(packet_info *pinfo, proto_tree *tree, tvbu
 static int dissect_krb5_EncTicketPart(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offset);
 static int dissect_krb5_EncAPRepPart(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offset);
 static int dissect_krb5_EncKrbPrivPart(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offset);
+static int dissect_krb5_EncKrbCredPart(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offset);
 static int dissect_krb5_EncKDCRepPart(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offset);
 static int dissect_krb5_KDC_REQ(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offset);
 static int dissect_krb5_KDC_REP(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offset);
@@ -1323,6 +1370,7 @@ static int dissect_krb5_AP_REQ(packet_info *pinfo, proto_tree *tree, tvbuff_t *t
 static int dissect_krb5_AP_REP(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offset);
 static int dissect_krb5_SAFE(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offset);
 static int dissect_krb5_PRIV(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offset);
+static int dissect_krb5_CRED(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offset);
 static int dissect_krb5_ERROR(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offset);
 
 static const ber_choice_t kerberos_applications_choice[] = {
@@ -1339,8 +1387,10 @@ static const ber_choice_t kerberos_applications_choice[] = {
        { KRB5_MSG_ENC_TGS_REP_PART, BER_CLASS_APP, KRB5_MSG_ENC_TGS_REP_PART, 0, dissect_krb5_EncKDCRepPart },
        { KRB5_MSG_ENC_AP_REP_PART, BER_CLASS_APP, KRB5_MSG_ENC_AP_REP_PART, 0, dissect_krb5_EncAPRepPart },
        { KRB5_MSG_ENC_KRB_PRIV_PART, BER_CLASS_APP, KRB5_MSG_ENC_KRB_PRIV_PART, 0, dissect_krb5_EncKrbPrivPart },
+       { KRB5_MSG_ENC_KRB_CRED_PART, BER_CLASS_APP, KRB5_MSG_ENC_KRB_CRED_PART, 0, dissect_krb5_EncKrbCredPart },
        { KRB5_MSG_SAFE,        BER_CLASS_APP,  KRB5_MSG_SAFE,          0,      dissect_krb5_SAFE },
        { KRB5_MSG_PRIV,        BER_CLASS_APP,  KRB5_MSG_PRIV,          0,      dissect_krb5_PRIV },
+       { KRB5_MSG_CRED,        BER_CLASS_APP,  KRB5_MSG_CRED,          0,      dissect_krb5_CRED },
        { KRB5_MSG_ERROR,       BER_CLASS_APP,  KRB5_MSG_ERROR,         0,      dissect_krb5_ERROR },
        { 0, 0, 0, 0, NULL }
 };
@@ -1405,6 +1455,10 @@ static const true_false_string krb5_kdcoptions_renewable = {
        "This ticket is RENEWABLE",
        "This ticket is NOT renewable"
 };
+static const true_false_string krb5_kdcoptions_constrained_delegation = {
+       "This is a request for a CONSTRAINED DELEGATION PAC",
+       "This is a normal request (no constrained delegation)"
+};
 static const true_false_string krb5_kdcoptions_canonicalize = {
        "This is a request for a CANONICALIZED ticket",
        "This is NOT a canonicalized ticket request"
@@ -1439,6 +1493,7 @@ static int* KDCOptions_bits[] = {
   &hf_krb_KDCOptions_postdated,
   &hf_krb_KDCOptions_renewable,
   &hf_krb_KDCOptions_opt_hardware_auth,
+  &hf_krb_KDCOptions_constrained_delegation,
   &hf_krb_KDCOptions_canonicalize,
   &hf_krb_KDCOptions_disable_transited_check,
   &hf_krb_KDCOptions_renewable_ok,
@@ -1640,8 +1695,9 @@ static int dissect_krb5_address(packet_info *pinfo, proto_tree *tree, tvbuff_t *
                {
                char netbios_name[(NETBIOS_NAME_LEN - 1)*4 + 1];
                int netbios_name_type;
+               int netbios_name_len = (NETBIOS_NAME_LEN - 1)*4 + 1;
 
-               netbios_name_type = process_netbios_name(tvb_get_ptr(tvb, offset, 16), netbios_name);
+               netbios_name_type = process_netbios_name(tvb_get_ptr(tvb, offset, 16), netbios_name, netbios_name_len);
                g_snprintf(address_str, 255, "%s<%02x>", netbios_name, netbios_name_type);
                it=proto_tree_add_string_format(tree, hf_krb_address_netbios, tvb, offset, 16, netbios_name, "NetBIOS Name: %s (%s)", address_str, netbios_name_type_descr(netbios_name_type));
                }
@@ -1682,6 +1738,15 @@ dissect_krb5_s_address(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int
        return offset;
 }
 
+static int
+dissect_krb5_r_address(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offset)
+{
+
+       offset=dissect_ber_sequence(FALSE, pinfo, tree, tvb, offset, HostAddress_sequence, hf_krb_r_address, ett_krb_r_address);
+
+       return offset;
+}
+
 /*
  *  HostAddresses ::=   SEQUENCE OF SEQUENCE {
  *                      addr-type[0]             INTEGER,
@@ -1701,6 +1766,17 @@ dissect_krb5_HostAddresses(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb,
 }
 
 
+/* sequence of tickets */
+static ber_sequence_t sequence_of_tickets[1] = {
+  { BER_CLASS_APP, 1, 0, dissect_krb5_Application_1},
+};
+static int
+dissect_krb5_sq_tickets(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offset)
+{
+       offset=dissect_ber_sequence_of(FALSE, pinfo, tree, tvb, offset, sequence_of_tickets, hf_krb_sq_tickets, ett_krb_sq_tickets);
+
+       return offset;
+}
 
 static int
 dissect_krb5_msg_type(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offset)
@@ -1790,6 +1866,14 @@ dissect_krb5_sname(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offs
 
        return offset;
 }
+static int
+dissect_krb5_pname(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offset)
+{
+
+       offset=dissect_ber_sequence(FALSE, pinfo, tree, tvb, offset, PrincipalName_sequence, hf_krb_pname, ett_krb_pname);
+
+       return offset;
+}
 int
 dissect_krb5_cname(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offset)
 {
@@ -1800,6 +1884,20 @@ dissect_krb5_cname(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offs
 }
 
 
+int
+dissect_krb5_prealm(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offset)
+{
+       offset=dissect_ber_GeneralString(pinfo, tree, tvb, offset, hf_krb_prealm, NULL, 0);
+       return offset;
+}
+
+int
+dissect_krb5_srealm(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offset)
+{
+       offset=dissect_ber_GeneralString(pinfo, tree, tvb, offset, hf_krb_srealm, NULL, 0);
+       return offset;
+}
+
 int
 dissect_krb5_realm(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offset)
 {
@@ -1960,7 +2058,7 @@ dissect_krb5_etype_info_salt(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb
        return offset;
 }
 
-int
+static int
 dissect_krb5_etype_info2_salt(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offset)
 {
        offset=dissect_ber_GeneralString(pinfo, tree, tvb, offset, hf_krb_etype_info2_salt, NULL, 0);
@@ -2028,6 +2126,33 @@ dissect_krb5_PA_ENCTYPE_INFO2(packet_info *pinfo, proto_tree *tree, tvbuff_t *tv
        return offset;
 }
 
+
+static int
+dissect_krb5_PW_SALT(packet_info *pinfo _U_, proto_tree *tree, tvbuff_t *tvb, int offset)
+{
+       /* Microsoft stores a special 12 byte blob here
+        * guint32 NT_status
+        * guint32 unknown
+        * guint32 unknown
+        * decode everything as this blob for now until we see if anyone
+        * else ever uses it   or we learn how to tell wether this
+        * is such an MS blob or not.
+        */
+       proto_tree_add_item(tree, hf_krb_smb_nt_status, tvb, offset, 4,
+                       TRUE);
+       offset += 4;
+
+       proto_tree_add_item(tree, hf_krb_smb_unknown, tvb, offset, 4,
+                       TRUE);
+       offset += 4;
+
+       proto_tree_add_item(tree, hf_krb_smb_unknown, tvb, offset, 4,
+                       TRUE);
+       offset += 4;
+
+       return offset;
+}
+
 /*
  * PA-DATA ::=        SEQUENCE {
  *          padata-type[1]        INTEGER,
@@ -2084,6 +2209,9 @@ dissect_krb5_PA_DATA_value(packet_info *pinfo, proto_tree *parent_tree, tvbuff_t
        case KRB5_PA_ENCTYPE_INFO2:
                offset=dissect_ber_octet_string_wcb(FALSE, pinfo, tree, tvb, offset,hf_krb_PA_DATA_value, dissect_krb5_PA_ENCTYPE_INFO2);
                break;
+       case KRB5_PA_PW_SALT:
+               offset=dissect_ber_octet_string_wcb(FALSE, pinfo, tree, tvb, offset,hf_krb_PA_DATA_value, dissect_krb5_PW_SALT);
+               break;
        default:
                offset=dissect_ber_octet_string_wcb(FALSE, pinfo, tree, tvb, offset,hf_krb_PA_DATA_value, NULL);
        }
@@ -2124,7 +2252,6 @@ dissect_krb5_padata(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int off
 
 
 
-
 static const true_false_string krb5_ticketflags_forwardable = {
        "FORWARDABLE tickets are allowed/requested",
        "Do NOT use forwardable tickets"
@@ -2280,10 +2407,11 @@ dissect_krb5_PAC_LOGON_INFO(packet_info *pinfo, proto_tree *parent_tree, tvbuff_
                tree=proto_item_add_subtree(item, ett_krb_PAC_LOGON_INFO);
        }
 
-       /* skip the first 20 bytes, they look like a unique ndr pointer
-          followed by (where did it come from?) a contect_handle ?*/
-       proto_tree_add_text(tree, tvb, offset, 20, "unknown: is this an undocumented policy handle?");
-       offset+=20;
+       /* skip the first 16 bytes, they are some magic created by the idl
+        * compiler   the first 4 bytes might be flags?
+        */
+       proto_tree_add_text(tree, tvb, offset, 16, "unknown blob");
+       offset+=16;
 
 
        /* the PAC_LOGON_INFO blob */
@@ -2294,13 +2422,49 @@ dissect_krb5_PAC_LOGON_INFO(packet_info *pinfo, proto_tree *parent_tree, tvbuff_
        pinfo->private_data=&di;
        init_ndr_pointer_list(pinfo);
        offset = dissect_ndr_pointer(tvb, offset, pinfo, tree, drep,
-               netlogon_dissect_PAC_LOGON_INFO, NDR_POINTER_REF,
+               netlogon_dissect_PAC_LOGON_INFO, NDR_POINTER_UNIQUE,
                "PAC_LOGON_INFO:", -1);
        pinfo->private_data=old_private_data;
 
        return offset;
 }
 
+static int
+dissect_krb5_PAC_CONSTRAINED_DELEGATION(packet_info *pinfo, proto_tree *parent_tree, tvbuff_t *tvb, int offset)
+{
+       proto_item *item=NULL;
+       proto_tree *tree=NULL;
+       guint8 drep[4] = { 0x10, 0x00, 0x00, 0x00}; /* fake DREP struct */
+       dcerpc_info di; /* fake dcerpc_info struct */
+       void *old_private_data;
+
+       item=proto_tree_add_item(parent_tree, hf_krb_PAC_CONSTRAINED_DELEGATION, tvb, offset, tvb_length_remaining(tvb, offset), FALSE);
+       if(parent_tree){
+               tree=proto_item_add_subtree(item, ett_krb_PAC_CONSTRAINED_DELEGATION);
+       }
+
+       /* skip the first 16 bytes, they are some magic created by the idl
+        * compiler   the first 4 bytes might be flags?
+        */
+       proto_tree_add_text(tree, tvb, offset, 16, "unknown blob");
+       offset+=16;
+
+
+       /* the PAC_CONSTRAINED_DELEGATION blob */
+       /* fake whatever state the dcerpc runtime support needs */
+       di.conformant_run=0;
+       di.call_data=NULL;
+       old_private_data=pinfo->private_data;
+       pinfo->private_data=&di;
+       init_ndr_pointer_list(pinfo);
+       offset = dissect_ndr_pointer(tvb, offset, pinfo, tree, drep,
+               netlogon_dissect_PAC_CONSTRAINED_DELEGATION, NDR_POINTER_UNIQUE,
+               "PAC_CONSTRAINED_DELEGATION:", -1);
+       pinfo->private_data=old_private_data;
+
+       return offset;
+}
+
 static int
 dissect_krb5_PAC_CREDENTIAL_TYPE(packet_info *pinfo _U_, proto_tree *parent_tree, tvbuff_t *tvb, int offset)
 {
@@ -2435,6 +2599,9 @@ dissect_krb5_AD_WIN2K_PAC_struct(packet_info *pinfo, proto_tree *tree, tvbuff_t
        case PAC_CLIENT_INFO_TYPE:
                dissect_krb5_PAC_CLIENT_INFO_TYPE(pinfo, tr, next_tvb, 0);
                break;
+       case PAC_CONSTRAINED_DELEGATION:
+               dissect_krb5_PAC_CONSTRAINED_DELEGATION(pinfo, tr, next_tvb, 0);
+               break;
        default:;
 /*qqq*/
        }
@@ -2825,17 +2992,124 @@ dissect_krb5_authenticator_vno(packet_info *pinfo, proto_tree *tree, tvbuff_t *t
 }
 
 
+#define KRB5_GSS_C_DELEG_FLAG          0x01
+#define KRB5_GSS_C_MUTUAL_FLAG         0x02
+#define KRB5_GSS_C_REPLAY_FLAG         0x04
+#define KRB5_GSS_C_SEQUENCE_FLAG       0x08
+#define KRB5_GSS_C_CONF_FLAG           0x10
+#define KRB5_GSS_C_INTEG_FLAG          0x20
+#define KRB5_GSS_C_DCE_STYLE           0x1000
+static const true_false_string tfs_gss_flags_deleg = {
+       "Delegate credantials to remote peer",
+       "Do NOT delegate"
+};
+static const true_false_string tfs_gss_flags_mutual = {
+       "Request that remote peer authenticates itself",
+       "Mutual authentication NOT required"
+};
+static const true_false_string tfs_gss_flags_replay = {
+       "Enable replay protection for signed or sealed messages",
+       "Do NOT enable replay protection"
+};
+static const true_false_string tfs_gss_flags_sequence = {
+       "Enable Out-of-sequence detection for sign or sealed messages",
+       "Do NOT enable out-of-sequence detection"
+};
+static const true_false_string tfs_gss_flags_conf = {
+       "Confidentiality (sealing) may be invoked",
+       "Do NOT use Confidentiality (sealing)"
+};
+static const true_false_string tfs_gss_flags_integ = {
+       "Integrity protection (signing) may be invoked",
+       "Do NOT use integrity protection"
+};
+
+static const true_false_string tfs_gss_flags_dce_style = {
+       "DCE-STYLE",
+       "Not using DCE-STYLE"
+};
+
+/* Dissect a GSSAPI checksum as per RFC1964. This is NOT ASN.1 encoded.
+ */
+static int
+dissect_krb5_rfc1964_checksum(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb)
+{
+       int offset=0;
+       guint32 len;
+       guint16 dlglen;
+
+       /* Length of Bnd field */
+       len=tvb_get_letohl(tvb, offset);
+       proto_tree_add_item(tree, hf_krb_gssapi_len, tvb, offset, 4, TRUE);
+       offset += 4;
+
+       /* Bnd field */
+       proto_tree_add_item(tree, hf_krb_gssapi_bnd, tvb, offset, len, TRUE);
+       offset += len;
+
+
+       /* flags */
+       proto_tree_add_item(tree, hf_krb_gssapi_c_flag_dce_style, tvb, offset, 4, TRUE);
+       proto_tree_add_item(tree, hf_krb_gssapi_c_flag_integ, tvb, offset, 4, TRUE);
+       proto_tree_add_item(tree, hf_krb_gssapi_c_flag_conf, tvb, offset, 4, TRUE);
+       proto_tree_add_item(tree, hf_krb_gssapi_c_flag_sequence, tvb, offset, 4, TRUE);
+       proto_tree_add_item(tree, hf_krb_gssapi_c_flag_replay, tvb, offset, 4, TRUE);
+       proto_tree_add_item(tree, hf_krb_gssapi_c_flag_mutual, tvb, offset, 4, TRUE);
+       proto_tree_add_item(tree, hf_krb_gssapi_c_flag_deleg, tvb, offset, 4, TRUE);
+       offset += 4;
+
+       /* the next fields are optional so we have to check that we have
+        * more data in our buffers */
+       if(tvb_length_remaining(tvb, offset)<2){
+               return offset;
+       }
+       /* dlgopt identifier */
+       proto_tree_add_item(tree, hf_krb_gssapi_dlgopt, tvb, offset, 2, TRUE);
+       offset += 2;
+
+       if(tvb_length_remaining(tvb, offset)<2){
+               return offset;
+       }
+       /* dlglen identifier */
+       dlglen=tvb_get_letohs(tvb, offset);
+       proto_tree_add_item(tree, hf_krb_gssapi_dlglen, tvb, offset, 2, TRUE);
+       offset += 2;
+
+       if(dlglen!=tvb_length_remaining(tvb, offset)){
+               proto_tree_add_text(tree, tvb, 0, 0, "Error: DlgLen:%d is not the same as number of bytes remaining:%d", dlglen, tvb_length_remaining(tvb, offset));
+               return offset;
+       }
+
+       /* this should now be a KRB_CRED message */
+       offset=dissect_ber_choice(pinfo, tree, tvb, offset, kerberos_applications_choice, -1, -1, NULL);
+
+
+       return offset;
+}
+
+static guint32 checksum_type;
+
 static int
 dissect_krb5_checksum_type(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offset)
 {
-       offset=dissect_ber_integer(FALSE, pinfo, tree, tvb, offset, hf_krb_checksum_type, NULL);
+       offset=dissect_ber_integer(FALSE, pinfo, tree, tvb, offset, hf_krb_checksum_type, &checksum_type);
 
        return offset;
 }
+
 static int
 dissect_krb5_checksum_checksum(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offset)
 {
-       offset=dissect_ber_octet_string(FALSE, pinfo, tree, tvb, offset, hf_krb_checksum_checksum, NULL);
+       tvbuff_t *next_tvb;
+
+       switch(checksum_type){
+       case KRB5_CHKSUM_GSSAPI:
+               offset=dissect_ber_octet_string(FALSE, pinfo, tree, tvb, offset, hf_krb_checksum_checksum, &next_tvb);
+               dissect_krb5_rfc1964_checksum(pinfo, tree, next_tvb);
+               break;
+       default:
+               offset=dissect_ber_octet_string(FALSE, pinfo, tree, tvb, offset, hf_krb_checksum_checksum, NULL);
+       }
        return offset;
 }
 
@@ -3030,6 +3304,155 @@ dissect_krb5_PRIV(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offse
        return offset;
 }
 
+static guint32 EncKrbCredPart_etype;
+static int
+dissect_krb5_EncKrbCredPart_etype(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offset)
+{
+       offset=dissect_ber_integer(FALSE, pinfo, tree, tvb, offset, hf_krb_etype, &EncKrbCredPart_etype);
+       if(tree){
+               proto_item_append_text(tree, " %s",
+                       val_to_str(EncKrbCredPart_etype, krb5_encryption_types,
+                       "%#x"));
+       }
+       return offset;
+}
+
+
+
+
+
+static ber_sequence_t KrbCredInfo_sequence[] = {
+       { BER_CLASS_CON, 0, 0, dissect_krb5_key },
+       { BER_CLASS_CON, 1, BER_FLAGS_OPTIONAL, dissect_krb5_prealm },
+       { BER_CLASS_CON, 2, BER_FLAGS_OPTIONAL, dissect_krb5_pname },
+       { BER_CLASS_CON, 3, BER_FLAGS_OPTIONAL, dissect_krb5_TicketFlags },
+       { BER_CLASS_CON, 4, BER_FLAGS_OPTIONAL, dissect_krb5_authtime },
+       { BER_CLASS_CON, 5, BER_FLAGS_OPTIONAL, dissect_krb5_starttime },
+       { BER_CLASS_CON, 6, BER_FLAGS_OPTIONAL, dissect_krb5_endtime },
+       { BER_CLASS_CON, 7, BER_FLAGS_OPTIONAL, dissect_krb5_renew_till },
+       { BER_CLASS_CON, 8, BER_FLAGS_OPTIONAL, dissect_krb5_srealm },
+       { BER_CLASS_CON, 9, BER_FLAGS_OPTIONAL, dissect_krb5_sname },
+       { BER_CLASS_CON, 10, BER_FLAGS_OPTIONAL, dissect_krb5_HostAddresses },
+       { 0, 0, 0, NULL }
+};
+static int
+dissect_krb5_KrbCredInfo(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offset)
+{
+
+       offset=dissect_ber_sequence(FALSE, pinfo, tree, tvb, offset, KrbCredInfo_sequence, hf_krb_KrbCredInfo, ett_krb_KrbCredInfo);
+
+       return offset;
+}
+
+static ber_sequence_t KrbCredInfo_sequence_of[1] = {
+  { BER_CLASS_UNI, BER_UNI_TAG_SEQUENCE, BER_FLAGS_NOOWNTAG, dissect_krb5_KrbCredInfo },
+};
+static int
+dissect_krb5_KrbCredInfo_sequence_of(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offset)
+{
+       offset=dissect_ber_sequence_of(FALSE, pinfo, tree, tvb, offset, KrbCredInfo_sequence_of, hf_krb_KrbCredInfos, ett_krb_KrbCredInfos);
+
+       return offset;
+}
+static const ber_sequence_t EncKrbCredPart_sequence[] = {
+       { BER_CLASS_CON, 0, 0, dissect_krb5_KrbCredInfo_sequence_of },
+       { BER_CLASS_CON, 1, BER_FLAGS_OPTIONAL, dissect_krb5_nonce },
+       { BER_CLASS_CON, 2, BER_FLAGS_OPTIONAL, dissect_krb5_ctime },
+       { BER_CLASS_CON, 3, BER_FLAGS_OPTIONAL, dissect_krb5_cusec },
+       { BER_CLASS_CON, 4, BER_FLAGS_OPTIONAL, dissect_krb5_s_address },
+       { BER_CLASS_CON, 5, BER_FLAGS_OPTIONAL, dissect_krb5_r_address },
+       { 0, 0, 0, NULL }
+};
+
+static int
+dissect_krb5_EncKrbCredPart(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offset)
+{
+       offset=dissect_ber_sequence(FALSE, pinfo, tree, tvb, offset, EncKrbCredPart_sequence, hf_krb_EncKrbCredPart, ett_krb_EncKrbCredPart);
+
+       return offset;
+}
+
+#ifdef HAVE_KERBEROS
+static int
+dissect_krb5_decrypt_EncKrbCredPart (packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offset)
+{
+       guint8 *plaintext=NULL;
+       int length;
+
+       length=tvb_length_remaining(tvb, offset);
+
+       /* RFC4120 :
+        * EncKrbCredPart encrypted with usage
+        * == 14
+        */
+       if(!plaintext){
+               plaintext=decrypt_krb5_data(tree, pinfo, 14, length, tvb_get_ptr(tvb, offset, length), EncKrbCredPart_etype);
+       }
+
+       if(plaintext){
+               tvbuff_t *next_tvb;
+               next_tvb = tvb_new_real_data (plaintext,
+                                          length,
+                                          length);
+               tvb_set_child_real_data_tvbuff(tvb, next_tvb);
+
+               /* Add the decrypted data to the data source list. */
+               add_new_data_source(pinfo, next_tvb, "EncKrbCredPart");
+
+               offset=dissect_ber_choice(pinfo, tree, next_tvb, 0, kerberos_applications_choice, -1, -1, NULL);
+       }
+       return offset;
+}
+#endif
+
+static int
+dissect_krb5_encrypted_CRED_data(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offset)
+{
+#ifdef HAVE_KERBEROS
+       offset=dissect_ber_octet_string_wcb(FALSE, pinfo, tree, tvb, offset, hf_krb_encrypted_EncKrbCredPart, dissect_krb5_decrypt_EncKrbCredPart);
+#else
+       offset=dissect_ber_octet_string_wcb(FALSE, pinfo, tree, tvb, offset, hf_krb_encrypted_EncKrbCredPart, NULL);
+#endif
+       return offset;
+}
+
+static ber_sequence_t encrypted_CRED_sequence[] = {
+       { BER_CLASS_CON, 0, 0,
+               dissect_krb5_EncKrbCredPart_etype },
+       { BER_CLASS_CON, 1, BER_FLAGS_OPTIONAL,
+               dissect_krb5_kvno },
+       { BER_CLASS_CON, 2, 0,
+               dissect_krb5_encrypted_CRED_data },
+       { 0, 0, 0, NULL }
+};
+static int
+dissect_krb5_encrypted_CRED(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offset)
+{
+       offset=dissect_ber_sequence(FALSE, pinfo, tree, tvb, offset, encrypted_CRED_sequence, hf_krb_CRED_enc, ett_krb_CRED_enc);
+
+       return offset;
+}
+
+static ber_sequence_t CRED_BODY_sequence[] = {
+       { BER_CLASS_CON, 0, 0,
+               dissect_krb5_pvno },
+       { BER_CLASS_CON, 1, 0,
+               dissect_krb5_msg_type },
+       { BER_CLASS_CON, 2, 0,
+               dissect_krb5_sq_tickets },
+       { BER_CLASS_CON, 3, 0,
+               dissect_krb5_encrypted_CRED },
+       { 0, 0, 0, NULL }
+};
+static int
+dissect_krb5_CRED(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offset)
+{
+
+       offset=dissect_ber_sequence(FALSE, pinfo, tree, tvb, offset, CRED_BODY_sequence, hf_krb_CRED_BODY, ett_krb_CRED);
+
+       return offset;
+}
+
 
 static int
 dissect_krb5_SAFE_BODY_user_data(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offset)
@@ -3149,7 +3572,9 @@ static ber_sequence_t KDC_REQ_BODY_sequence[] = {
                dissect_krb5_etype_sequence_of },
        { BER_CLASS_CON, 9, BER_FLAGS_OPTIONAL,
                dissect_krb5_HostAddresses },
-/* XXX [10] and [11] enc-authorization-data and additional-tickets should be added */
+/* XXX [10] enc-authorization-data should be added */
+       { BER_CLASS_CON, 11, BER_FLAGS_OPTIONAL,
+               dissect_krb5_sq_tickets },
        { 0, 0, 0, NULL }
 };
 static int
@@ -3652,7 +4077,18 @@ static int
 dissect_krb5_e_data(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offset)
 {
        switch(krb5_errorcode){
+       case KRB5_ET_KRB5KDC_ERR_BADOPTION:
+       case KRB5_ET_KRB5KDC_ERR_CLIENT_REVOKED:
+       case KRB5_ET_KRB5KDC_ERR_KEY_EXP:
+       case KRB5_ET_KRB5KDC_ERR_POLICY:
+               /* ms windows kdc sends e-data of this type containing a "salt"
+                * that contains the nt_status code for these error codes.
+                */
+               offset=dissect_ber_octet_string_wcb(FALSE, pinfo, tree, tvb, offset, hf_krb_e_data, dissect_krb5_PA_DATA);
+               break;
        case KRB5_ET_KRB5KDC_ERR_PREAUTH_REQUIRED:
+       case KRB5_ET_KRB5KDC_ERR_PREAUTH_FAILED:
+       case KRB5_ET_KRB5KDC_ERR_ETYPE_NOSUPP:
                offset=dissect_ber_octet_string_wcb(FALSE, pinfo, tree, tvb, offset, hf_krb_e_data, dissect_krb5_padata);
 
                break;
@@ -3752,11 +4188,8 @@ static gint dissect_kerberos_common(tvbuff_t *tvb, packet_info *pinfo,
                                        gboolean do_col_protocol,
                                        gboolean have_rm,
                                        kerberos_callbacks *cb);
-static gint kerberos_rm_to_reclen(guint krb_rm);
 static void dissect_kerberos_tcp_pdu(tvbuff_t *tvb, packet_info *pinfo,
                                proto_tree *tree);
-static guint get_krb_pdu_len(tvbuff_t *tvb, int offset);
-
 
 
 gint
@@ -3765,7 +4198,7 @@ dissect_kerberos_main(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, int d
     return (dissect_kerberos_common(tvb, pinfo, tree, do_col_info, FALSE, FALSE, cb));
 }
 
-guint32 
+guint32
 kerberos_output_keytype(void)
 {
   return keytype;
@@ -3775,15 +4208,18 @@ static gint
 dissect_kerberos_udp(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
 {
     /* Some weird kerberos implementation apparently do krb4 on the krb5 port.
-       Since all (except weirdo transarc krb4 stuff) use 
+       Since all (except weirdo transarc krb4 stuff) use
        an opcode <=16 in the first byte, use this to see if it might
        be krb4.
        All krb5 commands start with an APPL tag and thus is >=0x60
        so if first byte is <=16  just blindly assume it is krb4 then
     */
     if(tvb_bytes_exist(tvb, 0, 1) && tvb_get_guint8(tvb, 0)<=0x10){
-      if(krb4_handle){ 
-       return call_dissector(krb4_handle, tvb, pinfo, tree);
+      if(krb4_handle){
+       gboolean res;
+
+       res=call_dissector_only(krb4_handle, tvb, pinfo, tree);
+       return res;
       }else{
         return 0;
       }
@@ -3793,14 +4229,14 @@ dissect_kerberos_udp(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
     return dissect_kerberos_common(tvb, pinfo, tree, TRUE, TRUE, FALSE, NULL);
 }
 
-static gint
+gint
 kerberos_rm_to_reclen(guint krb_rm)
 {
     return (krb_rm & KRB_RM_RECLEN);
 }
 
-static guint
-get_krb_pdu_len(tvbuff_t *tvb, int offset)
+guint
+get_krb_pdu_len(packet_info *pinfo _U_, tvbuff_t *tvb, int offset)
 {
     guint krb_rm;
     gint pdulen;
@@ -3840,7 +4276,7 @@ dissect_kerberos_tcp(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
 /*
  * Display the TCP record mark.
  */
-static void
+void
 show_krb_recordmark(proto_tree *tree, tvbuff_t *tvb, gint start, guint32 krb_rm)
 {
     gint rec_len;
@@ -3898,7 +4334,7 @@ dissect_kerberos_common(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree,
        show_krb_recordmark(kerberos_tree, tvb, offset, krb_rm);
        offset += 4;
     } else {
-        /* Do some sanity checking here, 
+        /* Do some sanity checking here,
          * All krb5 packets start with a TAG class that is BER_CLASS_APP
          * and a tag value that is either of the values below:
          * If it doesnt look like kerberos, return 0 and let someone else have
@@ -3926,6 +4362,7 @@ dissect_kerberos_common(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree,
             case KRB5_MSG_ENC_TGS_REP_PART:
             case KRB5_MSG_ENC_AP_REP_PART:
             case KRB5_MSG_ENC_KRB_PRIV_PART:
+            case KRB5_MSG_ENC_KRB_CRED_PART:
             case KRB5_MSG_SAFE:
             case KRB5_MSG_PRIV:
             case KRB5_MSG_ERROR:
@@ -4088,6 +4525,12 @@ proto_register_kerberos(void)
        { &hf_krb_realm, {
            "Realm", "kerberos.realm", FT_STRING, BASE_NONE,
            NULL, 0, "Name of the Kerberos Realm", HFILL }},
+       { &hf_krb_srealm, {
+           "SRealm", "kerberos.srealm", FT_STRING, BASE_NONE,
+           NULL, 0, "Name of the Kerberos SRealm", HFILL }},
+       { &hf_krb_prealm, {
+           "Delegated Principal Realm", "kerberos.prealm", FT_STRING, BASE_NONE,
+           NULL, 0, "Name of the Kerberos PRealm", HFILL }},
        { &hf_krb_crealm, {
            "Client Realm", "kerberos.crealm", FT_STRING, BASE_NONE,
            NULL, 0, "Name of the Clients Kerberos Realm", HFILL }},
@@ -4157,6 +4600,9 @@ proto_register_kerberos(void)
        { &hf_krb_PRIV_BODY, {
            "PRIV_BODY", "kerberos.priv_body", FT_NONE, BASE_NONE,
            NULL, 0, "Kerberos PRIVate BODY", HFILL }},
+       { &hf_krb_CRED_BODY, {
+           "CRED_BODY", "kerberos.cred_body", FT_NONE, BASE_NONE,
+           NULL, 0, "Kerberos CREDential BODY", HFILL }},
        { &hf_krb_encrypted_PRIV, {
            "Encrypted PRIV", "kerberos.enc_priv", FT_NONE, BASE_NONE,
            NULL, 0, "Kerberos Encrypted PRIVate blob data", HFILL }},
@@ -4181,6 +4627,9 @@ proto_register_kerberos(void)
        { &hf_krb_KDCOptions_renewable, {
            "Renewable", "kerberos.kdcoptions.renewable", FT_BOOLEAN, 32,
            TFS(&krb5_kdcoptions_renewable), 0x00800000, "Whether this ticket is renewable or not", HFILL }},
+       { &hf_krb_KDCOptions_constrained_delegation, {
+           "Constrained Delegation", "kerberos.kdcoptions.constrained_delegation", FT_BOOLEAN, 32,
+           TFS(&krb5_kdcoptions_constrained_delegation), 0x00020000, "Do we want a PAC containing constrained delegation info or not", HFILL }},
        { &hf_krb_KDCOptions_canonicalize, {
            "Canonicalize", "kerberos.kdcoptions.canonicalize", FT_BOOLEAN, 32,
            TFS(&krb5_kdcoptions_canonicalize), 0x00010000, "Do we want the KDC to canonicalize the principal or not", HFILL }},
@@ -4217,6 +4666,9 @@ proto_register_kerberos(void)
        { &hf_krb_encrypted_authenticator_data, {
            "Authenticator data", "kerberos.authenticator.data", FT_BYTES, BASE_HEX,
            NULL, 0, "Data content of an encrypted authenticator", HFILL }},
+       { &hf_krb_encrypted_EncKrbCredPart, {
+           "enc EncKrbCredPart", "kerberos.EncKrbCredPart.encrypted", FT_BYTES, BASE_HEX,
+           NULL, 0, "Encrypted EncKrbCredPart blob", HFILL }},
        { &hf_krb_encrypted_PA_ENC_TIMESTAMP, {
            "enc PA_ENC_TIMESTAMP", "kerberos.PA_ENC_TIMESTAMP.encrypted", FT_BYTES, BASE_HEX,
            NULL, 0, "Encrypted PA-ENC-TIMESTAMP blob", HFILL }},
@@ -4235,6 +4687,9 @@ proto_register_kerberos(void)
        { &hf_krb_PAC_CLIENT_INFO_TYPE, {
            "PAC_CLIENT_INFO_TYPE", "kerberos.PAC_CLIENT_INFO_TYPE", FT_BYTES, BASE_HEX,
            NULL, 0, "PAC_CLIENT_INFO_TYPE structure", HFILL }},
+       { &hf_krb_PAC_CONSTRAINED_DELEGATION, {
+           "PAC_CONSTRAINED_DELEGATION", "kerberos.PAC_CONSTRAINED_DELEGATION", FT_BYTES, BASE_HEX,
+           NULL, 0, "PAC_CONSTRAINED_DELEGATION structure", HFILL }},
        { &hf_krb_checksum_checksum, {
            "checksum", "kerberos.checksum.checksum", FT_BYTES, BASE_HEX,
            NULL, 0, "Kerberos Checksum", HFILL }},
@@ -4280,12 +4735,18 @@ proto_register_kerberos(void)
        { &hf_krb_tkt_vno, {
            "Tkt-vno", "kerberos.tkt_vno", FT_UINT32, BASE_DEC,
            NULL, 0, "Version number for the Ticket format", HFILL }},
+       { &hf_krb_KrbCredInfo, {
+           "KrbCredInfo", "kerberos.KrbCredInfo", FT_NONE, BASE_DEC,
+           NULL, 0, "This is a Kerberos KrbCredInfo", HFILL }},
        { &hf_krb_HostAddress, {
            "HostAddress", "kerberos.hostaddress", FT_NONE, BASE_DEC,
            NULL, 0, "This is a Kerberos HostAddress sequence", HFILL }},
        { &hf_krb_s_address, {
            "S-Address", "kerberos.s_address", FT_NONE, BASE_DEC,
            NULL, 0, "This is the Senders address", HFILL }},
+       { &hf_krb_r_address, {
+           "R-Address", "kerberos.r_address", FT_NONE, BASE_DEC,
+           NULL, 0, "This is the Recipient address", HFILL }},
        { &hf_krb_key, {
            "key", "kerberos.key", FT_NONE, BASE_DEC,
            NULL, 0, "This is a Kerberos EncryptionKey sequence", HFILL }},
@@ -4307,6 +4768,9 @@ proto_register_kerberos(void)
        { &hf_krb_EncKrbPrivPart, {
            "EncKrbPrivPart", "kerberos.EncKrbPrivPart", FT_NONE, BASE_DEC,
            NULL, 0, "This is a decrypted Kerberos EncKrbPrivPart sequence", HFILL }},
+       { &hf_krb_EncKrbCredPart, {
+           "EncKrbCredPart", "kerberos.EncKrbCredPart", FT_NONE, BASE_DEC,
+           NULL, 0, "This is a decrypted Kerberos EncKrbCredPart sequence", HFILL }},
        { &hf_krb_EncKDCRepPart, {
            "EncKDCRepPart", "kerberos.EncKDCRepPart", FT_NONE, BASE_DEC,
            NULL, 0, "This is a decrypted Kerberos EncKDCRepPart sequence", HFILL }},
@@ -4328,18 +4792,30 @@ proto_register_kerberos(void)
        { &hf_krb_etypes, {
            "Encryption Types", "kerberos.etypes", FT_NONE, BASE_DEC,
            NULL, 0, "This is a list of Kerberos encryption types", HFILL }},
+       { &hf_krb_KrbCredInfos, {
+           "Sequence of KrbCredInfo", "kerberos.KrbCredInfos", FT_NONE, BASE_DEC,
+           NULL, 0, "This is a list of KrbCredInfo", HFILL }},
+       { &hf_krb_sq_tickets, {
+           "Tickets", "kerberos.sq.tickets", FT_NONE, BASE_DEC,
+           NULL, 0, "This is a list of Kerberos Tickets", HFILL }},
        { &hf_krb_LastReqs, {
            "LastReqs", "kerberos.LastReqs", FT_NONE, BASE_DEC,
            NULL, 0, "This is a list of LastReq structures", HFILL }},
        { &hf_krb_sname, {
            "Server Name", "kerberos.sname", FT_NONE, BASE_DEC,
            NULL, 0, "This is the name part server's identity", HFILL }},
+       { &hf_krb_pname, {
+           "Delegated Principal Name", "kerberos.pname", FT_NONE, BASE_DEC,
+           NULL, 0, "Identity of the delegated principal", HFILL }},
        { &hf_krb_cname, {
            "Client Name", "kerberos.cname", FT_NONE, BASE_DEC,
            NULL, 0, "The name part of the client principal identifier", HFILL }},
        { &hf_krb_authenticator_enc, {
            "Authenticator", "kerberos.authenticator", FT_NONE, BASE_DEC,
            NULL, 0, "Encrypted authenticator blob", HFILL }},
+       { &hf_krb_CRED_enc, {
+           "EncKrbCredPart", "kerberos.encrypted_cred", FT_NONE, BASE_DEC,
+           NULL, 0, "Encrypted Cred blob", HFILL }},
        { &hf_krb_ticket_enc, {
            "enc-part", "kerberos.ticket.enc_part", FT_NONE, BASE_DEC,
            NULL, 0, "The structure holding the encrypted part of a ticket", HFILL }},
@@ -4388,23 +4864,69 @@ proto_register_kerberos(void)
        { &hf_krb_e_checksum, {
            "e-checksum", "kerberos.e_checksum", FT_NONE, BASE_DEC,
            NULL, 0, "This is a Kerberos e-checksum", HFILL }},
+       { &hf_krb_gssapi_len, {
+           "Length", "kerberos.gssapi.len", FT_UINT32, BASE_DEC,
+           NULL, 0, "Length of GSSAPI Bnd field", HFILL }},
+       { &hf_krb_gssapi_bnd, {
+           "Bnd", "kerberos.gssapi.bdn", FT_BYTES, BASE_HEX,
+           NULL, 0, "GSSAPI Bnd field", HFILL }},
+       { &hf_krb_gssapi_c_flag_deleg, {
+           "Deleg", "kerberos.gssapi.checksum.flags.deleg", FT_BOOLEAN, 32,
+           TFS(&tfs_gss_flags_deleg), KRB5_GSS_C_DELEG_FLAG, "", HFILL }},
+       { &hf_krb_gssapi_c_flag_mutual, {
+           "Mutual", "kerberos.gssapi.checksum.flags.mutual", FT_BOOLEAN, 32,
+           TFS(&tfs_gss_flags_mutual), KRB5_GSS_C_MUTUAL_FLAG, "", HFILL }},
+       { &hf_krb_gssapi_c_flag_replay, {
+           "Replay", "kerberos.gssapi.checksum.flags.replay", FT_BOOLEAN, 32,
+           TFS(&tfs_gss_flags_replay), KRB5_GSS_C_REPLAY_FLAG, "", HFILL }},
+       { &hf_krb_gssapi_c_flag_sequence, {
+           "Sequence", "kerberos.gssapi.checksum.flags.sequence", FT_BOOLEAN, 32,
+           TFS(&tfs_gss_flags_sequence), KRB5_GSS_C_SEQUENCE_FLAG, "", HFILL }},
+       { &hf_krb_gssapi_c_flag_conf, {
+           "Conf", "kerberos.gssapi.checksum.flags.conf", FT_BOOLEAN, 32,
+           TFS(&tfs_gss_flags_conf), KRB5_GSS_C_CONF_FLAG, "", HFILL }},
+       { &hf_krb_gssapi_c_flag_integ, {
+           "Integ", "kerberos.gssapi.checksum.flags.integ", FT_BOOLEAN, 32,
+           TFS(&tfs_gss_flags_integ), KRB5_GSS_C_INTEG_FLAG, "", HFILL }},
+       { &hf_krb_gssapi_c_flag_dce_style, {
+           "DCE-style", "kerberos.gssapi.checksum.flags.dce-style", FT_BOOLEAN, 32,
+           TFS(&tfs_gss_flags_dce_style), KRB5_GSS_C_DCE_STYLE, "", HFILL }},
+       { &hf_krb_gssapi_dlgopt, {
+           "DlgOpt", "kerberos.gssapi.dlgopt", FT_UINT16, BASE_DEC,
+           NULL, 0, "GSSAPI DlgOpt", HFILL }},
+       { &hf_krb_gssapi_dlglen, {
+           "DlgLen", "kerberos.gssapi.dlglen", FT_UINT16, BASE_DEC,
+           NULL, 0, "GSSAPI DlgLen", HFILL }},
+       { &hf_krb_smb_nt_status,
+               { "NT Status", "kerberos.smb.nt_status", FT_UINT32, BASE_HEX,
+               VALS(NT_errors), 0, "NT Status code", HFILL }},
+       { &hf_krb_smb_unknown,
+               { "Unknown", "kerberos.smb.unknown", FT_UINT32, BASE_HEX,
+               NULL, 0, "unknown", HFILL }},
+
     };
 
     static gint *ett[] = {
         &ett_krb_kerberos,
        &ett_krb_KDC_REP_enc,
         &ett_krb_sname,
+        &ett_krb_pname,
         &ett_krb_cname,
        &ett_krb_AP_REP_enc,
         &ett_krb_padata,
         &ett_krb_etypes,
+        &ett_krb_KrbCredInfos,
+        &ett_krb_sq_tickets,
         &ett_krb_LastReqs,
         &ett_krb_IF_RELEVANT,
        &ett_krb_PA_DATA_tree,
         &ett_krb_s_address,
+        &ett_krb_r_address,
+        &ett_krb_KrbCredInfo,
         &ett_krb_HostAddress,
         &ett_krb_HostAddresses,
        &ett_krb_authenticator_enc,
+       &ett_krb_CRED_enc,
         &ett_krb_AP_Options,
         &ett_krb_KDC_Options,
         &ett_krb_Ticket_Flags,
@@ -4412,11 +4934,13 @@ proto_register_kerberos(void)
         &ett_krb_recordmark,
         &ett_krb_ticket,
        &ett_krb_ticket_enc,
+        &ett_krb_CRED,
         &ett_krb_PRIV,
         &ett_krb_PRIV_enc,
         &ett_krb_EncTicketPart,
         &ett_krb_EncAPRepPart,
         &ett_krb_EncKrbPrivPart,
+        &ett_krb_EncKrbCredPart,
         &ett_krb_EncKDCRepPart,
         &ett_krb_LastReq,
         &ett_krb_Authenticator,
@@ -4431,6 +4955,7 @@ proto_register_kerberos(void)
        &ett_krb_PAC_SERVER_CHECKSUM,
        &ett_krb_PAC_PRIVSVR_CHECKSUM,
        &ett_krb_PAC_CLIENT_INFO_TYPE,
+       &ett_krb_PAC_CONSTRAINED_DELEGATION,
        &ett_krb_e_checksum,
     };
     module_t *krb_module;