Extract the CRC-32 code from the 802.11 dissector into a separate file,
[obnox/wireshark/wip.git] / packet-isakmp.c
index 6a167812543df40342cbed2537a28b8baf1eaecd..929b12b3a89d291dd43fad5e8bc116bdb97f5e8d 100644 (file)
@@ -1,24 +1,25 @@
 /* packet-isakmp.c
  * Routines for the Internet Security Association and Key Management Protocol
- * (ISAKMP) (RFC 2408)
+ * (ISAKMP) (RFC 2408) and the Internet IP Security Domain of Interpretation
+ * for ISAKMP (RFC 2407)
  * Brad Robel-Forrest <brad.robel-forrest@watchguard.com>
  *
- * $Id: packet-isakmp.c,v 1.43 2001/09/25 18:27:35 guy Exp $
+ * $Id: packet-isakmp.c,v 1.67 2003/08/08 23:08:02 guy Exp $
  *
  * Ethereal - Network traffic analyzer
  * By Gerald Combs <gerald@ethereal.com>
  * Copyright 1998 Gerald Combs
- * 
+ *
  * This program is free software; you can redistribute it and/or
  * modify it under the terms of the GNU General Public License
  * as published by the Free Software Foundation; either version 2
  * of the License, or (at your option) any later version.
- * 
+ *
  * This program is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  * GNU General Public License for more details.
- * 
+ *
  * You should have received a copy of the GNU General Public License
  * along with this program; if not, write to the Free Software
  * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA  02111-1307, USA.
 # include "config.h"
 #endif
 
-#ifdef HAVE_SYS_TYPES_H
-# include <sys/types.h>
-#endif
-
 #include <stdio.h>
 #include <string.h>
 
-#ifdef HAVE_NETINET_IN_H
-#include <netinet/in.h>
-#endif
-
 #include <glib.h>
 
 #ifdef NEED_SNPRINTF_H
 # include "snprintf.h"
 #endif
 
-#include "packet.h"
+#include <epan/packet.h>
 #include "ipproto.h"
 
+#define isakmp_min(a, b)  ((a<b) ? a : b)
+
 static int proto_isakmp = -1;
 
 static gint ett_isakmp = -1;
@@ -155,6 +150,18 @@ static const char *esp_transtypestr[NUM_ESP_TRANS_TYPES] = {
   "AES"
 };
 
+#define NUM_IPCOMP_TRANS_TYPES    5
+#define ipcomp_trans2str(t)  \
+  ((t < NUM_IPCOMP_TRANS_TYPES) ? ipcomp_transtypestr[t] : "UNKNOWN-IPCOMP-TRANS-TYPE")
+
+static const char *ipcomp_transtypestr[NUM_IPCOMP_TRANS_TYPES] = {
+  "RESERVED",
+  "OUI",
+  "DEFLATE",
+  "LZS",
+  "LZJH"
+};
+
 #define NUM_ID_TYPES   12
 #define id2str(t)      \
   ((t < NUM_ID_TYPES) ? idtypestr[t] : "UNKNOWN-ID-TYPE")
@@ -174,6 +181,31 @@ static const char *idtypestr[NUM_ID_TYPES] = {
   "KEY_ID"
 };
 
+#define NUM_GRPDESC_TYPES 19
+#define grpdesc2str(t) ((t < NUM_GRPDESC_TYPES) ? grpdescstr[t] : "UNKNOWN-GROUP-DESCRIPTION")
+
+static const char *grpdescstr[NUM_GRPDESC_TYPES] = {
+  "UNDEFINED - 0",
+  "Default 768-bit MODP group",
+  "Alternate 1024-bit MODP group",
+  "EC2N group on GP[2^155] group",
+  "EC2N group on GP[2^185] group",
+  "1536 bit MODP group",
+  "EC2N group over GF[2^163]",
+  "EC2N group over GF[2^163]",
+  "EC2N group over GF[2^283]",
+  "EC2N group over GF[2^283]",
+  "EC2N group over GF[2^409]",
+  "EC2N group over GF[2^409]",
+  "EC2N group over GF[2^571]",
+  "EC2N group over GF[2^571]",
+  "2048 bit MODP group",
+  "3072 bit MODP group",
+  "4096 bit MODP group",
+  "6144 bit MODP group",
+  "8192 bit MODP group",
+};
+
 struct isakmp_hdr {
   guint8       icookie[8];
   guint8       rcookie[8];
@@ -184,33 +216,32 @@ struct isakmp_hdr {
 #define E_FLAG         0x01
 #define C_FLAG         0x02
 #define A_FLAG         0x04
-  guint8       message_id[4];
-  guint8       length[4];
+  guint32      message_id;
+  guint32      length;
 };
 
 struct udp_encap_hdr {
-  guint8       non_ike_marker[8];
+  guint8       non_esp_marker[4];
   guint32      esp_SPI;
 };
 
 static proto_tree *dissect_payload_header(tvbuff_t *, int, int, guint8,
     guint8 *, guint16 *, proto_tree *);
 
-static void dissect_none(tvbuff_t *, int, int, proto_tree *);
-static void dissect_sa(tvbuff_t *, int, int, proto_tree *);
-static void dissect_proposal(tvbuff_t *, int, int, proto_tree *);
-static void dissect_transform(tvbuff_t *, int, int, proto_tree *, guint8);
-static void dissect_key_exch(tvbuff_t *, int, int, proto_tree *);
-static void dissect_id(tvbuff_t *, int, int, proto_tree *);
-static void dissect_cert(tvbuff_t *, int, int, proto_tree *);
-static void dissect_certreq(tvbuff_t *, int, int, proto_tree *);
-static void dissect_hash(tvbuff_t *, int, int, proto_tree *);
-static void dissect_sig(tvbuff_t *, int, int, proto_tree *);
-static void dissect_nonce(tvbuff_t *, int, int, proto_tree *);
-static void dissect_notif(tvbuff_t *, int, int, proto_tree *);
-static void dissect_delete(tvbuff_t *, int, int, proto_tree *);
-static void dissect_vid(tvbuff_t *, int, int, proto_tree *);
-static void dissect_config(tvbuff_t *, int, int, proto_tree *);
+static void dissect_sa(tvbuff_t *, int, int, proto_tree *, int);
+static void dissect_proposal(tvbuff_t *, int, int, proto_tree *, int);
+static void dissect_transform(tvbuff_t *, int, int, proto_tree *, int);
+static void dissect_key_exch(tvbuff_t *, int, int, proto_tree *, int);
+static void dissect_id(tvbuff_t *, int, int, proto_tree *, int);
+static void dissect_cert(tvbuff_t *, int, int, proto_tree *, int);
+static void dissect_certreq(tvbuff_t *, int, int, proto_tree *, int);
+static void dissect_hash(tvbuff_t *, int, int, proto_tree *, int);
+static void dissect_sig(tvbuff_t *, int, int, proto_tree *, int);
+static void dissect_nonce(tvbuff_t *, int, int, proto_tree *, int);
+static void dissect_notif(tvbuff_t *, int, int, proto_tree *, int);
+static void dissect_delete(tvbuff_t *, int, int, proto_tree *, int);
+static void dissect_vid(tvbuff_t *, int, int, proto_tree *, int);
+static void dissect_config(tvbuff_t *, int, int, proto_tree *, int);
 
 static const char *payloadtype2str(guint8);
 static const char *exchtype2str(guint8);
@@ -224,20 +255,21 @@ static const char *certtype2str(guint8);
 
 static gboolean get_num(tvbuff_t *, int, guint16, guint32 *);
 
+#define LOAD_TYPE_NONE         0       /* payload type for None */
 #define LOAD_TYPE_PROPOSAL     2       /* payload type for Proposal */
 #define        LOAD_TYPE_TRANSFORM     3       /* payload type for Transform */
-#define NUM_LOAD_TYPES         15
+#define NUM_LOAD_TYPES         17
 #define loadtype2str(t)        \
   ((t < NUM_LOAD_TYPES) ? strfuncs[t].str : "Unknown payload type")
 
 static struct strfunc {
   const char * str;
-  void          (*func)(tvbuff_t *, int, int, proto_tree *);
+  void          (*func)(tvbuff_t *, int, int, proto_tree *, int);
 } strfuncs[NUM_LOAD_TYPES] = {
-  {"NONE",                     dissect_none      },
+  {"NONE",                     NULL              },
   {"Security Association",     dissect_sa        },
   {"Proposal",                 dissect_proposal  },
-  {"Transform",                        NULL },
+  {"Transform",                        dissect_transform },
   {"Key Exchange",             dissect_key_exch  },
   {"Identification",           dissect_id        },
   {"Certificate",              dissect_cert      },
@@ -248,180 +280,190 @@ static struct strfunc {
   {"Notification",             dissect_notif     },
   {"Delete",                   dissect_delete    },
   {"Vendor ID",                        dissect_vid       },
-  {"Attrib",                   dissect_config    }
+  {"Attrib",                   dissect_config    },
+  {"NAT-Discovery",            NULL              }, /* http://www.ietf.org/internet-drafts/draft-ietf-ipsec-nat-t-ike-05.txt */
+  {"NAT-Original Address",     NULL              }  /* http://www.ietf.org/internet-drafts/draft-ietf-ipsec-nat-t-ike-05.txt */
 };
 
+#define VID_LEN 16
+#define VID_MS_LEN 20
+static const guint8 VID_MS_W2K_WXP[VID_MS_LEN] = {0x1E, 0x2B, 0x51, 0x69, 0x5, 0x99, 0x1C, 0x7D, 0x7C, 0x96, 0xFC, 0xBF, 0xB5, 0x87, 0xE4, 0x61, 0x0, 0x0, 0x0, 0x2}; /* according to http://www.microsoft.com/technet/treeview/default.asp?url=/technet/columns/cableguy/cg0602.asp */
+
+#define VID_CP_LEN 20
+static const guint8 VID_CP[VID_CP_LEN] = {0xF4, 0xED, 0x19, 0xE0, 0xC1, 0x14, 0xEB, 0x51, 0x6F, 0xAA, 0xAC, 0x0E, 0xE3, 0x7D, 0xAF, 0x28, 0x7, 0xB4, 0x38, 0x1F};
+
+static const guint8 VID_CYBERGUARD[VID_LEN] = {0x9A, 0xA1, 0xF3, 0xB4, 0x34, 0x72, 0xA4, 0x5D, 0x5F, 0x50, 0x6A, 0xEB, 0x26, 0xC, 0xF2, 0x14};
+
+static const guint8 VID_draft_ietf_ipsec_nat_t_ike_03[VID_LEN] = {0x7D, 0x94, 0x19, 0xA6, 0x53, 0x10, 0xCA, 0x6F, 0x2C, 0x17, 0x9D, 0x92, 0x15, 0x52, 0x9d, 0x56}; /* according to http://www.ietf.org/internet-drafts/draft-ietf-ipsec-nat-t-ike-03.txt */
+
+/* 
+*  Seen in Netscreen. Suppose to be ASCII HeartBeat_Notify - but I don't know the rest yet. I suspect it then proceeds with
+*  8k10, which means every 8K (?), and version 1.0 of the protocol (?). I won't add it to the code, until I know what it really
+*  means. ykaul-at-netvision.net.il
+*/
+static const guint8 VID_HeartBeat_Notify[VID_LEN] = {0x48, 0x65, 0x61, 0x72, 0x74, 0x42, 0x65, 0x61, 0x74, 0x5f, 0x4e, 0x6f, 0x74, 0x69, 0x66, 0x79}; 
+
 static dissector_handle_t esp_handle;
 static dissector_handle_t ah_handle;
 
+static void
+dissect_payloads(tvbuff_t *tvb, proto_tree *tree, guint8 initial_payload,
+                int offset, int length)
+{
+  guint8 payload, next_payload;
+  guint16              payload_length;
+  proto_tree *         ntree;
+
+  for (payload = initial_payload; length != 0; payload = next_payload) {
+    if (payload == LOAD_TYPE_NONE) {
+      /*
+       * What?  There's more stuff in this chunk of data, but the
+       * previous payload had a "next payload" type of None?
+       */
+      proto_tree_add_text(tree, tvb, offset, length,
+                         "Extra data: %s",
+                         tvb_bytes_to_str(tvb, offset, length));
+      break;
+    }
+    ntree = dissect_payload_header(tvb, offset, length, payload,
+      &next_payload, &payload_length, tree);
+    if (ntree == NULL)
+      break;
+    if (payload_length >= 4) { /* XXX = > 4? */
+      if (payload < NUM_LOAD_TYPES) {
+        (*strfuncs[payload].func)(tvb, offset + 4, payload_length - 4, ntree,
+                                 -1);
+      }
+      else {
+        proto_tree_add_text(ntree, tvb, offset + 4, payload_length - 4,
+            "Payload");
+      }
+    }
+    else {
+        proto_tree_add_text(ntree, tvb, offset + 4, 0,
+            "Payload (bogus, length is %u, must be at least 4)",
+            payload_length);
+        payload_length = 4;
+    }
+    offset += payload_length;
+    length -= payload_length;
+  }
+}
+
 static void
 dissect_isakmp(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
 {
   int                  offset = 0;
-  struct isakmp_hdr *  hdr;
+  struct isakmp_hdr    hdr;
   proto_item *         ti;
   proto_tree *         isakmp_tree = NULL;
-  struct udp_encap_hdr * encap_hdr;
+  struct udp_encap_hdr  encap_hdr;
   guint32              len;
-  guint8               payload, next_payload;
-  guint16              payload_length;
-  proto_tree *         ntree;
-  static const guint8  non_ike_marker[8] = { 0, 0, 0, 0, 0, 0, 0, 0 };
+  static const guint8  non_esp_marker[4] = { 0, 0, 0, 0 };
   tvbuff_t *           next_tvb;
 
-  if (check_col(pinfo->fd, COL_PROTOCOL))
-    col_set_str(pinfo->fd, COL_PROTOCOL, "ISAKMP");
-  if (check_col(pinfo->fd, COL_INFO))
-    col_clear(pinfo->fd, COL_INFO);
+  if (check_col(pinfo->cinfo, COL_PROTOCOL))
+    col_set_str(pinfo->cinfo, COL_PROTOCOL, "ISAKMP");
+  if (check_col(pinfo->cinfo, COL_INFO))
+    col_clear(pinfo->cinfo, COL_INFO);
+
+  hdr.length = tvb_get_ntohl(tvb, offset + sizeof(hdr) - sizeof(hdr.length));
 
-  hdr = (struct isakmp_hdr *)tvb_get_ptr(tvb, 0, sizeof (struct isakmp_hdr));
-  len = pntohl(&hdr->length);
-  
   if (tree) {
-    ti = proto_tree_add_item(tree, proto_isakmp, tvb, offset, len, FALSE);
+    ti = proto_tree_add_item(tree, proto_isakmp, tvb, offset, hdr.length, FALSE);
     isakmp_tree = proto_item_add_subtree(ti, ett_isakmp);
   }
-    
-  encap_hdr = (struct udp_encap_hdr *)tvb_get_ptr(tvb, 0, sizeof(struct udp_encap_hdr));
-  if (memcmp(encap_hdr->non_ike_marker,non_ike_marker,8) == 0) {
-    if (check_col(pinfo->fd, COL_INFO)) {
-      if (encap_hdr->esp_SPI != 0)
-          col_add_str(pinfo->fd, COL_INFO, "UDP encapsulated IPSec - ESP");
-      else
-         col_add_str(pinfo->fd, COL_INFO, "UDP encapsulated IPSec - AH");
-    } 
+
+  tvb_memcpy(tvb, (guint8 *)&encap_hdr, 0, sizeof(encap_hdr));
+
+  if (encap_hdr.non_esp_marker[0] == 0xFF) {
+    if (check_col(pinfo->cinfo, COL_INFO))
+      col_add_str(pinfo->cinfo, COL_INFO, "UDP encapsulated IPSec - NAT Keepalive");
+    return;
+  }
+  if (memcmp(encap_hdr.non_esp_marker,non_esp_marker,4) == 0) {
+    if (check_col(pinfo->cinfo, COL_INFO))
+          col_add_str(pinfo->cinfo, COL_INFO, "UDP encapsulated IPSec - ESP");
     if (tree)
       proto_tree_add_text(isakmp_tree, tvb, offset,
-                         sizeof(encap_hdr->non_ike_marker),
-                         "Non-IKE-Marker");
-    offset += sizeof(encap_hdr->non_ike_marker);
-      
-    if (encap_hdr->esp_SPI != 0) {
-      next_tvb = tvb_new_subset(tvb, offset, -1, -1);
-      call_dissector(esp_handle, next_tvb, pinfo, tree);
-    } else {
-      if (tree)
-        proto_tree_add_text(isakmp_tree, tvb, offset,
-                           sizeof(encap_hdr->esp_SPI),
-                           "Non-ESP-Marker");
-      offset += sizeof(encap_hdr->esp_SPI);
-
-      if (tree)
-        proto_tree_add_text(isakmp_tree, tvb, offset, 1,
-                           "AH Envelope Version: %u",
-                           tvb_get_guint8(tvb, offset) >> 4);
-      offset += 1;
-
-      if (tree)
-        proto_tree_add_text(isakmp_tree, tvb, offset, 1,
-                           "AH Envelope Header Length: %u",
-                           (tvb_get_guint8(tvb, offset) & 0xF)*4);
-      offset += 1;
-
-      if (tree)
-        proto_tree_add_text(isakmp_tree, tvb, offset, 2,
-                           "AH Envelope Identification: 0x%04X",
-                           tvb_get_ntohs(tvb, offset));
-      offset += 2;
-
-      next_tvb = tvb_new_subset(tvb, offset, -1, -1);
-      call_dissector(ah_handle, next_tvb, pinfo, tree);
-    }
+                         sizeof(encap_hdr.non_esp_marker),
+                         "Non-ESP-Marker");
+    offset += sizeof(encap_hdr.non_esp_marker);
+    next_tvb = tvb_new_subset(tvb, offset, -1, -1);
+    call_dissector(esp_handle, next_tvb, pinfo, tree);
     return;
   }
-
-  if (check_col(pinfo->fd, COL_INFO))
-    col_add_str(pinfo->fd, COL_INFO, exchtype2str(hdr->exch_type));
+  hdr.exch_type = tvb_get_guint8(tvb, sizeof(hdr.icookie) + sizeof(hdr.rcookie) + sizeof(hdr.next_payload) + sizeof(hdr.version));
+  if (check_col(pinfo->cinfo, COL_INFO))
+    col_add_str(pinfo->cinfo, COL_INFO, exchtype2str(hdr.exch_type));
 
   if (tree) {
-    proto_tree_add_text(isakmp_tree, tvb, offset, sizeof(hdr->icookie),
-                       "Initiator cookie");
-    offset += sizeof(hdr->icookie);
-    
-    proto_tree_add_text(isakmp_tree, tvb, offset, sizeof(hdr->rcookie),
-                       "Responder cookie");
-    offset += sizeof(hdr->rcookie);
-
-    proto_tree_add_text(isakmp_tree, tvb, offset, sizeof(hdr->next_payload),
+    tvb_memcpy(tvb, (guint8 *)&hdr.icookie, offset, sizeof(hdr.icookie));
+    proto_tree_add_text(isakmp_tree, tvb, offset, sizeof(hdr.icookie),
+                       "Initiator cookie: 0x%s", tvb_bytes_to_str(tvb, offset, sizeof(hdr.icookie)));
+    offset += sizeof(hdr.icookie);
+
+    tvb_memcpy(tvb, (guint8 *)&hdr.rcookie, offset, sizeof(hdr.rcookie));
+    proto_tree_add_text(isakmp_tree, tvb, offset, sizeof(hdr.rcookie),
+                       "Responder cookie: 0x%s", tvb_bytes_to_str(tvb, offset, sizeof(hdr.rcookie)));
+    offset += sizeof(hdr.rcookie);
+
+    hdr.next_payload = tvb_get_guint8(tvb, offset);
+    proto_tree_add_text(isakmp_tree, tvb, offset, sizeof(hdr.next_payload),
                        "Next payload: %s (%u)",
-                       payloadtype2str(hdr->next_payload), hdr->next_payload);
-    offset += sizeof(hdr->next_payload);
+                       payloadtype2str(hdr.next_payload), hdr.next_payload);
+    offset += sizeof(hdr.next_payload);
 
-    proto_tree_add_text(isakmp_tree, tvb, offset, sizeof(hdr->version),
+    hdr.version = tvb_get_guint8(tvb, offset);
+    proto_tree_add_text(isakmp_tree, tvb, offset, sizeof(hdr.version),
                        "Version: %u.%u",
-                       hi_nibble(hdr->version), lo_nibble(hdr->version));
-    offset += sizeof(hdr->version);
-    
-    proto_tree_add_text(isakmp_tree, tvb, offset, sizeof(hdr->exch_type),
+                       hi_nibble(hdr.version), lo_nibble(hdr.version));
+    offset += sizeof(hdr.version);
+
+    hdr.exch_type = tvb_get_guint8(tvb, offset);
+    proto_tree_add_text(isakmp_tree, tvb, offset, sizeof(hdr.exch_type),
                        "Exchange type: %s (%u)",
-                       exchtype2str(hdr->exch_type), hdr->exch_type);
-    offset += sizeof(hdr->exch_type);
-    
+                       exchtype2str(hdr.exch_type), hdr.exch_type);
+    offset += sizeof(hdr.exch_type);
+
     {
       proto_item *     fti;
       proto_tree *     ftree;
-      
-      fti   = proto_tree_add_text(isakmp_tree, tvb, offset, sizeof(hdr->flags), "Flags");
+
+      hdr.flags = tvb_get_guint8(tvb, offset);
+      fti   = proto_tree_add_text(isakmp_tree, tvb, offset, sizeof(hdr.flags), "Flags");
       ftree = proto_item_add_subtree(fti, ett_isakmp_flags);
-      
+
       proto_tree_add_text(ftree, tvb, offset, 1, "%s",
-                         decode_boolean_bitfield(hdr->flags, E_FLAG, sizeof(hdr->flags)*8,
+                         decode_boolean_bitfield(hdr.flags, E_FLAG, sizeof(hdr.flags)*8,
                                                  "Encryption", "No encryption"));
       proto_tree_add_text(ftree, tvb, offset, 1, "%s",
-                         decode_boolean_bitfield(hdr->flags, C_FLAG, sizeof(hdr->flags)*8,
+                         decode_boolean_bitfield(hdr.flags, C_FLAG, sizeof(hdr.flags)*8,
                                                  "Commit", "No commit"));
       proto_tree_add_text(ftree, tvb, offset, 1, "%s",
-                         decode_boolean_bitfield(hdr->flags, A_FLAG, sizeof(hdr->flags)*8,
+                         decode_boolean_bitfield(hdr.flags, A_FLAG, sizeof(hdr.flags)*8,
                                                  "Authentication", "No authentication"));
-      offset += sizeof(hdr->flags);
+      offset += sizeof(hdr.flags);
     }
 
-    proto_tree_add_text(isakmp_tree, tvb, offset, sizeof(hdr->message_id),
-        "Message ID: 0x%02x%02x%02x%02x", hdr->message_id[0],
-        hdr->message_id[1], hdr->message_id[2], hdr->message_id[3]);
-    offset += sizeof(hdr->message_id);
-    
-    proto_tree_add_text(isakmp_tree, tvb, offset, sizeof(hdr->length),
-                       "Length: %u", len);
-    offset += sizeof(hdr->length);
-    len -= sizeof(*hdr);
-
-    if (hdr->flags & E_FLAG) {
+    proto_tree_add_text(isakmp_tree, tvb, offset, sizeof(hdr.message_id),
+        "Message ID: 0x%s", tvb_bytes_to_str(tvb, offset, sizeof(hdr.message_id)));
+    offset += sizeof(hdr.message_id);
+
+    proto_tree_add_text(isakmp_tree, tvb, offset, sizeof(hdr.length),
+                       "Length: %u", hdr.length);
+    offset += sizeof(hdr.length);
+
+    len = hdr.length - sizeof(hdr);
+
+    if (hdr.flags & E_FLAG) {
       if (len && isakmp_tree) {
         proto_tree_add_text(isakmp_tree, tvb, offset, len,
                        "Encrypted payload (%d byte%s)",
                        len, plurality(len, "", "s"));
       }
-    } else {
-      for (payload = hdr->next_payload; len != 0; payload = next_payload) {
-        ntree = dissect_payload_header(tvb, offset, len, payload,
-            &next_payload, &payload_length, isakmp_tree);
-        if (ntree == NULL)
-          break;
-        if (payload_length >= 4) {     /* XXX = > 4? */
-          if (payload < NUM_LOAD_TYPES) {
-            if (next_payload == LOAD_TYPE_TRANSFORM)
-              dissect_transform(tvb, offset + 4, payload_length - 4, ntree, 0);
-               /* XXX - protocol ID? */
-            else
-              (*strfuncs[payload].func)(tvb, offset + 4, payload_length - 4, ntree);
-          }
-          else {
-            proto_tree_add_text(ntree, tvb, offset + 4, payload_length - 4,
-                "Payload");
-          }
-        }
-        else {
-            proto_tree_add_text(ntree, tvb, offset + 4, 0,
-                "Payload (bogus, length is %u, must be at least 4)",
-                payload_length);
-            payload_length = 4;
-        }
-        offset += payload_length;
-        len -= payload_length;
-      }
-    }
+    } else
+      dissect_payloads(tvb, isakmp_tree, hdr.next_payload, offset, len);
   }
 }
 
@@ -457,41 +499,52 @@ dissect_payload_header(tvbuff_t *tvb, int offset, int length, guint8 payload,
 }
 
 static void
-dissect_none(tvbuff_t *tvb, int offset, int length, proto_tree *tree)
-{
-}
-
-static void
-dissect_sa(tvbuff_t *tvb, int offset, int length, proto_tree *tree)
+dissect_sa(tvbuff_t *tvb, int offset, int length, proto_tree *tree,
+    int unused _U_)
 {
   guint32              doi;
   guint32              situation;
-  guint8               next_payload;
-  guint16              payload_length;
-  proto_tree *         ntree;
 
+  if (length < 4) {
+    proto_tree_add_text(tree, tvb, offset, length,
+                       "DOI %s (length is %u, should be >= 4)",
+                       tvb_bytes_to_str(tvb, offset, length), length);
+    return;
+  }
   doi = tvb_get_ntohl(tvb, offset);
   proto_tree_add_text(tree, tvb, offset, 4,
                      "Domain of interpretation: %s (%u)",
                      doitype2str(doi), doi);
   offset += 4;
   length -= 4;
-  
-  situation = tvb_get_ntohl(tvb, offset);
-  proto_tree_add_text(tree, tvb, offset, 4,
-                     "Situation: %s (%u)",
-                     situation2str(situation), situation);
-  offset += 4;
-  length -= 4;
-  
-  ntree = dissect_payload_header(tvb, offset, length, LOAD_TYPE_PROPOSAL,
-    &next_payload, &payload_length, tree);
-  if (ntree != NULL)
-    dissect_proposal(tvb, offset + 4, payload_length - 4, ntree);
+
+  if (doi == 1) {
+    /* IPSEC */
+    if (length < 4) {
+      proto_tree_add_text(tree, tvb, offset, length,
+                         "Situation: %s (length is %u, should be >= 4)",
+                         tvb_bytes_to_str(tvb, offset, length), length);
+      return;
+    }
+    situation = tvb_get_ntohl(tvb, offset);
+    proto_tree_add_text(tree, tvb, offset, 4,
+                       "Situation: %s (%u)",
+                       situation2str(situation), situation);
+    offset += 4;
+    length -= 4;
+
+    dissect_payloads(tvb, tree, LOAD_TYPE_PROPOSAL, offset, length);
+  } else {
+    /* Unknown */
+    proto_tree_add_text(tree, tvb, offset, length,
+                       "Situation: %s",
+                       tvb_bytes_to_str(tvb, offset, length));
+  }
 }
 
 static void
-dissect_proposal(tvbuff_t *tvb, int offset, int length, proto_tree *tree)
+dissect_proposal(tvbuff_t *tvb, int offset, int length, proto_tree *tree,
+    int unused _U_)
 {
   guint8               protocol_id;
   guint8               spi_size;
@@ -499,19 +552,23 @@ dissect_proposal(tvbuff_t *tvb, int offset, int length, proto_tree *tree)
   guint8               next_payload;
   guint16              payload_length;
   proto_tree *         ntree;
+  guint8               proposal_num;
+
+  proposal_num = tvb_get_guint8(tvb, offset);
 
+  proto_item_append_text(tree, " # %d",proposal_num);
   proto_tree_add_text(tree, tvb, offset, 1,
-                     "Proposal number: %u", tvb_get_guint8(tvb, offset));
+                     "Proposal number: %u", proposal_num);
   offset += 1;
   length -= 1;
-  
+
   protocol_id = tvb_get_guint8(tvb, offset);
   proto_tree_add_text(tree, tvb, offset, 1,
                      "Protocol ID: %s (%u)",
                      proto2str(protocol_id), protocol_id);
   offset += 1;
   length -= 1;
-  
+
   spi_size = tvb_get_guint8(tvb, offset);
   proto_tree_add_text(tree, tvb, offset, 1,
                      "SPI size: %u", spi_size);
@@ -525,7 +582,8 @@ dissect_proposal(tvbuff_t *tvb, int offset, int length, proto_tree *tree)
   length -= 1;
 
   if (spi_size) {
-    proto_tree_add_text(tree, tvb, offset, spi_size, "SPI");
+    proto_tree_add_text(tree, tvb, offset, spi_size, "SPI: %s",
+                       tvb_bytes_to_str(tvb, offset, spi_size));
     offset += spi_size;
     length -= spi_size;
   }
@@ -552,18 +610,24 @@ dissect_proposal(tvbuff_t *tvb, int offset, int length, proto_tree *tree)
 
 static void
 dissect_transform(tvbuff_t *tvb, int offset, int length, proto_tree *tree,
-    guint8 protocol_id)
+    int protocol_id)
 {
   guint8               transform_id;
+  guint8               transform_num;
 
+  transform_num = tvb_get_guint8(tvb, offset);
+  proto_item_append_text(tree," # %d",transform_num);
   proto_tree_add_text(tree, tvb, offset, 1,
-                     "Transform number: %u", tvb_get_guint8(tvb, offset));
+                     "Transform number: %u", transform_num);
   offset += 1;
   length -= 1;
 
   transform_id = tvb_get_guint8(tvb, offset);
   switch (protocol_id) {
   default:
+    proto_tree_add_text(tree, tvb, offset, 1,
+                       "Transform ID: %u", transform_id);
+    break;
   case 1:      /* ISAKMP */
     proto_tree_add_text(tree, tvb, offset, 1,
                        "Transform ID: %s (%u)",
@@ -579,10 +643,15 @@ dissect_transform(tvbuff_t *tvb, int offset, int length, proto_tree *tree,
                        "Transform ID: %s (%u)",
                        esp_trans2str(transform_id), transform_id);
     break;
+  case 4:      /* IPCOMP */
+    proto_tree_add_text(tree, tvb, offset, 1,
+                       "Transform ID: %s (%u)",
+                       ipcomp_trans2str(transform_id), transform_id);
+    break;
   }
   offset += 3;
   length -= 3;
-  
+
   while (length>0) {
     const char *str;
     int ike_phase1 = 0;
@@ -629,13 +698,15 @@ dissect_transform(tvbuff_t *tvb, int offset, int length, proto_tree *tree,
 }
 
 static void
-dissect_key_exch(tvbuff_t *tvb, int offset, int length, proto_tree *tree)
+dissect_key_exch(tvbuff_t *tvb, int offset, int length, proto_tree *tree,
+    int unused _U_)
 {
   proto_tree_add_text(tree, tvb, offset, length, "Key Exchange Data");
 }
 
 static void
-dissect_id(tvbuff_t *tvb, int offset, int length, proto_tree *tree)
+dissect_id(tvbuff_t *tvb, int offset, int length, proto_tree *tree,
+    int unused _U_)
 {
   guint8               id_type;
   guint8               protocol_id;
@@ -666,10 +737,9 @@ dissect_id(tvbuff_t *tvb, int offset, int length, proto_tree *tree)
     proto_tree_add_text(tree, tvb, offset, 2, "Port: %u", port);
   offset += 2;
   length -= 2;
-  
+
   switch (id_type) {
     case 1:
-    case 4:
       proto_tree_add_text(tree, tvb, offset, length,
                          "Identification data: %s",
                          ip_to_str(tvb_get_ptr(tvb, offset, 4)));
@@ -680,6 +750,12 @@ dissect_id(tvbuff_t *tvb, int offset, int length, proto_tree *tree)
                          "Identification data: %.*s", length,
                          tvb_get_ptr(tvb, offset, length));
       break;
+    case 4:
+      proto_tree_add_text(tree, tvb, offset, length,
+                         "Identification data: %s/%s",
+                         ip_to_str(tvb_get_ptr(tvb, offset, 4)),
+                         ip_to_str(tvb_get_ptr(tvb, offset+4, 4)));
+      break;
     default:
       proto_tree_add_text(tree, tvb, offset, length, "Identification Data");
       break;
@@ -687,7 +763,8 @@ dissect_id(tvbuff_t *tvb, int offset, int length, proto_tree *tree)
 }
 
 static void
-dissect_cert(tvbuff_t *tvb, int offset, int length, proto_tree *tree)
+dissect_cert(tvbuff_t *tvb, int offset, int length, proto_tree *tree,
+    int unused _U_)
 {
   guint8               cert_enc;
 
@@ -702,7 +779,8 @@ dissect_cert(tvbuff_t *tvb, int offset, int length, proto_tree *tree)
 }
 
 static void
-dissect_certreq(tvbuff_t *tvb, int offset, int length, proto_tree *tree)
+dissect_certreq(tvbuff_t *tvb, int offset, int length, proto_tree *tree,
+    int unused _U_)
 {
   guint8               cert_type;
 
@@ -717,25 +795,29 @@ dissect_certreq(tvbuff_t *tvb, int offset, int length, proto_tree *tree)
 }
 
 static void
-dissect_hash(tvbuff_t *tvb, int offset, int length, proto_tree *tree)
+dissect_hash(tvbuff_t *tvb, int offset, int length, proto_tree *tree,
+    int unused _U_)
 {
   proto_tree_add_text(tree, tvb, offset, length, "Hash Data");
 }
 
 static void
-dissect_sig(tvbuff_t *tvb, int offset, int length, proto_tree *tree)
+dissect_sig(tvbuff_t *tvb, int offset, int length, proto_tree *tree,
+    int unused _U_)
 {
   proto_tree_add_text(tree, tvb, offset, length, "Signature Data");
 }
 
 static void
-dissect_nonce(tvbuff_t *tvb, int offset, int length, proto_tree *tree)
+dissect_nonce(tvbuff_t *tvb, int offset, int length, proto_tree *tree,
+    int unused _U_)
 {
   proto_tree_add_text(tree, tvb, offset, length, "Nonce Data");
 }
 
 static void
-dissect_notif(tvbuff_t *tvb, int offset, int length, proto_tree *tree)
+dissect_notif(tvbuff_t *tvb, int offset, int length, proto_tree *tree,
+    int unused _U_)
 {
   guint32              doi;
   guint8               protocol_id;
@@ -755,13 +837,13 @@ dissect_notif(tvbuff_t *tvb, int offset, int length, proto_tree *tree)
                      proto2str(protocol_id), protocol_id);
   offset += 1;
   length -= 1;
-  
+
   spi_size = tvb_get_guint8(tvb, offset);
   proto_tree_add_text(tree, tvb, offset, 1,
                      "SPI size: %u", spi_size);
   offset += 1;
   length -= 1;
-  
+
   msgtype = tvb_get_ntohs(tvb, offset);
   proto_tree_add_text(tree, tvb, offset, 2,
                      "Message type: %s (%u)", msgtype2str(msgtype), msgtype);
@@ -779,14 +861,15 @@ dissect_notif(tvbuff_t *tvb, int offset, int length, proto_tree *tree)
 }
 
 static void
-dissect_delete(tvbuff_t *tvb, int offset, int length, proto_tree *tree)
+dissect_delete(tvbuff_t *tvb, int offset, int length, proto_tree *tree,
+    int unused _U_)
 {
   guint32              doi;
   guint8               protocol_id;
   guint8               spi_size;
   guint16              num_spis;
   guint16              i;
-  
+
   doi = tvb_get_ntohl(tvb, offset);
   proto_tree_add_text(tree, tvb, offset, 4,
                      "Domain of Interpretation: %s (%u)",
@@ -800,7 +883,7 @@ dissect_delete(tvbuff_t *tvb, int offset, int length, proto_tree *tree)
                      proto2str(protocol_id), protocol_id);
   offset += 1;
   length -= 1;
-  
+
   spi_size = tvb_get_guint8(tvb, offset);
   proto_tree_add_text(tree, tvb, offset, 1,
                      "SPI size: %u", spi_size);
@@ -812,7 +895,7 @@ dissect_delete(tvbuff_t *tvb, int offset, int length, proto_tree *tree)
                      "Number of SPIs: %u", num_spis);
   offset += 2;
   length -= 2;
-  
+
   for (i = 0; i < num_spis; ++i) {
     if (length < spi_size) {
       proto_tree_add_text(tree, tvb, offset, length,
@@ -827,28 +910,83 @@ dissect_delete(tvbuff_t *tvb, int offset, int length, proto_tree *tree)
 }
 
 static void
-dissect_vid(tvbuff_t *tvb, int offset, int length, proto_tree *tree)
+dissect_vid(tvbuff_t *tvb, int offset, int length, proto_tree *tree,
+    int unused _U_)
 {
-  proto_tree_add_text(tree, tvb, offset, length, "Vendor ID");
+  guint32 CPproduct, CPversion;
+  const guint8 * pVID;
+  proto_item * pt;
+  proto_tree * ntree;
+  pVID = tvb_get_ptr(tvb, offset, length);
+  pt = proto_tree_add_text(tree, tvb, offset, length, "Vendor ID: ");
+  if (memcmp(pVID, VID_MS_W2K_WXP, isakmp_min(VID_MS_LEN, length)) == 0)
+       proto_item_append_text(pt, "Microsoft Win2K/WinXP");
+  else
+  if (memcmp(pVID, VID_CP, isakmp_min(VID_CP_LEN, length)) == 0)
+  {
+       proto_item_append_text(pt, "Check Point");
+       offset += VID_CP_LEN;
+       CPproduct = tvb_get_ntohl(tvb, offset);
+       ntree = proto_item_add_subtree(pt, ett_isakmp_payload);
+       pt = proto_tree_add_text(ntree, tvb, offset, sizeof(CPproduct), "Check Point Product: ");
+       switch (CPproduct) {
+               case 1: proto_item_append_text(pt, "VPN-1");
+                       break;
+               case 2: proto_item_append_text(pt, "SecuRemote/SecureClient");
+                       break;
+               default: proto_item_append_text(pt, "Unknown CP product!");
+                       break;
+       }
+       offset += sizeof(CPproduct);
+       CPversion = tvb_get_ntohl(tvb, offset);
+       pt = proto_tree_add_text(ntree, tvb, offset, length, "Version: ");
+       switch (CPversion) {
+               case 2: proto_item_append_text(pt, "4.1");
+                       break;
+               case 3: proto_item_append_text(pt, "4.1 SP-1");
+                       break;
+               case 4002: proto_item_append_text(pt, "4.1 (SP-2 or above)");
+                       break;
+               case 5000: proto_item_append_text(pt, "NG");
+                       break;
+               case 5001: proto_item_append_text(pt, "NG Feature Pack 1");
+                       break;
+               case 5002: proto_item_append_text(pt, "NG Feature Pack 2");
+                       break;
+               case 5003: proto_item_append_text(pt, "NG Feature Pack 3");
+                       break;
+               default: proto_item_append_text(pt, " Uknown CP version!");
+                       break;
+       }
+  }
+  else
+  if (memcmp(pVID, VID_CYBERGUARD, isakmp_min(VID_LEN, length)) == 0)
+        proto_item_append_text(pt, "Cyber Guard");
+  else
+  if (memcmp(pVID,  VID_draft_ietf_ipsec_nat_t_ike_03, isakmp_min(VID_LEN, length)) == 0)
+        proto_item_append_text(pt, "draft-ietf-ipsec-nat-t-ike-03");
+  else
+       proto_item_append_text(pt, "unknown vendor ID: 0x%s",tvb_bytes_to_str(tvb, offset, length));
 }
 
 static void
-dissect_config(tvbuff_t *tvb, int offset, int length, proto_tree *tree)
+dissect_config(tvbuff_t *tvb, int offset, int length, proto_tree *tree,
+    int unused _U_)
 {
   guint8               type;
 
   type = tvb_get_guint8(tvb, offset);
   proto_tree_add_text(tree, tvb, offset, 1,
                      "Type %s (%u)",attrtype2str(type),type);
-  
+
   offset += 2;
   length -= 2;
-  
+
   proto_tree_add_text(tree, tvb, offset, 2,
                       "Identifier: %u", tvb_get_ntohs(tvb, offset));
   offset += 2;
   length -= 2;
-  
+
   while(length>0) {
     guint16 aft     = tvb_get_ntohs(tvb, offset);
     guint16 type    = aft & 0x7fff;
@@ -884,11 +1022,11 @@ dissect_config(tvbuff_t *tvb, int offset, int length, proto_tree *tree)
 static const char *
 payloadtype2str(guint8 type) {
 
-  if (type < NUM_LOAD_TYPES) return strfuncs[type].str;
-  if (type < 128)            return "RESERVED";
-  if (type < 256)            return "Private USE";
-
-  return "Huh? You should never see this! Shame on you!";
+  if (type < NUM_LOAD_TYPES)
+    return strfuncs[type].str;
+  if (type < 128)
+    return "RESERVED";
+  return "Private USE";
 }
 
 static const char *
@@ -904,7 +1042,7 @@ exchtype2str(guint8 type) {
     "Informational",
     "Transaction (Config Mode)"
   };
-  
+
   if (type < NUM_EXCHSTRS) return exchstrs[type];
   if (type < 32)           return "ISAKMP Future Use";
   switch (type) {
@@ -913,10 +1051,9 @@ exchtype2str(guint8 type) {
   case 33:
     return "New Group Mode";
   }
-  if (type < 240)          return "DOI Specific Use";
-  if (type < 256)          return "Private Use";
-  
-  return "Huh? You should never see this! Shame on you!";
+  if (type < 240)
+    return "DOI Specific Use";
+  return "Private Use";
 }
 
 static const char *
@@ -990,11 +1127,12 @@ situation2str(guint32 type) {
   int          n = 0;
   char *       sep = "";
   int          ret;
-  
+
   if (type & SIT_IDENTITY) {
     ret = snprintf(msg, SIT_MSG_NUM-n, "%sIDENTITY", sep);
-    if (ret == -1) {
-      /* Some versions of snprintf return -1 if they'd truncate the output. */
+    if (ret == -1 || ret >= SIT_MSG_NUM-n) {
+      /* Truncated. */
+      msg[SIT_MSG_NUM-1] = '\0';
       return msg;
     }
     n += ret;
@@ -1006,8 +1144,9 @@ situation2str(guint32 type) {
       return msg;
     }
     ret = snprintf(msg, SIT_MSG_NUM-n, "%sSECRECY", sep);
-    if (ret == -1) {
-      /* Some versions of snprintf return -1 if they'd truncate the output. */
+    if (ret == -1 || ret >= SIT_MSG_NUM-n) {
+      /* Truncated. */
+      msg[SIT_MSG_NUM-1] = '\0';
       return msg;
     }
     n += ret;
@@ -1019,8 +1158,9 @@ situation2str(guint32 type) {
       return msg;
     }
     ret = snprintf(msg, SIT_MSG_NUM-n, "%sINTEGRITY", sep);
-    if (ret == -1) {
-      /* Some versions of snprintf return -1 if they'd truncate the output. */
+    if (ret == -1 || ret >= SIT_MSG_NUM-n) {
+      /* Truncated. */
+      msg[SIT_MSG_NUM-1] = '\0';
       return msg;
     }
     n += ret;
@@ -1032,13 +1172,14 @@ situation2str(guint32 type) {
 
 static const char *
 value2str(int ike_p1, guint16 att_type, guint16 value) {
-  
+
   if (value == 0) return "RESERVED";
-  
+
   if (!ike_p1) {
   switch (att_type) {
     case 1:
       switch (value) {
+       case 0: return "RESERVED";
         case 1:  return "Seconds";
         case 2:  return "Kilobytes";
         default: return "UNKNOWN-SA-VALUE";
@@ -1049,8 +1190,11 @@ value2str(int ike_p1, guint16 att_type, guint16 value) {
       return "Group-Value";
     case 4:
       switch (value) {
+       case 0:  return "RESERVED";
         case 1:  return "Tunnel";
         case 2:  return "Transport";
+       case 3:  return "UDP-Encapsulated-Tunnel"; /* http://www.ietf.org/internet-drafts/draft-ietf-ipsec-nat-t-ike-05.txt */
+       case 4:  return "UDP-Encapsulated-Transport"; /* http://www.ietf.org/internet-drafts/draft-ietf-ipsec-nat-t-ike-05.txt */
        case 61440: return "Check Point IPSec UDP Encapsulation";
        case 61443: return "UDP-Encapsulated-Tunnel (draft)";
        case 61444: return "UDP-Encapsulated-Transport (draft)";
@@ -1058,6 +1202,7 @@ value2str(int ike_p1, guint16 att_type, guint16 value) {
       }
     case 5:
       switch (value) {
+       case 0:  return "RESERVED";
         case 1:  return "HMAC-MD5";
         case 2:  return "HMAC-SHA";
         case 3:  return "DES-MAC";
@@ -1072,7 +1217,9 @@ value2str(int ike_p1, guint16 att_type, guint16 value) {
     case 7:
       return "Key-Rounds";
     case 8:
-      return "log2-size";
+      return "Compress-Dictionary-size";
+    case 9:
+      return "Compress Private Algorithm";
     default: return "UNKNOWN-ATTRIBUTE-TYPE";
   }
   }
@@ -1106,6 +1253,9 @@ value2str(int ike_p1, guint16 att_type, guint16 value) {
           case 3:  return "RSA-SIG";
           case 4:  return "RSA-ENC";
           case 5:  return "RSA-Revised-ENC";
+         case 6:  return "Encryption with El-Gamal";
+         case 7:  return "Revised encryption with El-Gamal";
+         case 8:  return "ECDSA signatures";
          case 64221: return "HybridInitRSA";
          case 64222: return "HybridRespRSA";
          case 64223: return "HybridInitDSS";
@@ -1122,7 +1272,7 @@ value2str(int ike_p1, guint16 att_type, guint16 value) {
           case 65010: return "XAUTHRespRSARevisedEncryption";
          default: return "UNKNOWN-AUTH-METHOD";
         }
-      case 4:
+      case 4: return grpdesc2str(value);
       case 6:
       case 7:
       case 8:
@@ -1156,7 +1306,7 @@ value2str(int ike_p1, guint16 att_type, guint16 value) {
   }
 }
 
-static const char * 
+static const char *
 attrtype2str(guint8 type) {
   switch (type) {
   case 0: return "Reserved";
@@ -1170,7 +1320,7 @@ attrtype2str(guint8 type) {
   return "Private use";
 }
 
-static const char * 
+static const char *
 cfgattrident2str(guint16 ident) {
 #define NUM_ATTR_DEFINED       12
   static const char *msgs[NUM_PREDEFINED] = {
@@ -1187,7 +1337,7 @@ cfgattrident2str(guint16 ident) {
     "INTERNAL_IP6_DNS",
     "INTERNAL_IP6_NBNS",
     "INTERNAL_IP6_DHCP",
-  }; 
+  };
   if(ident < NUM_ATTR_DEFINED)
     return msgs[ident];
   if(ident < 16383)
@@ -1201,6 +1351,8 @@ cfgattrident2str(guint16 ident) {
   case 16525: return "XAUTH_CHALLANGE";
   case 16526: return "XAUTH_DOMAIN";
   case 16527: return "XAUTH_STATUS";
+  case 16528: return "XAUTH_NEXT_PIN";
+  case 16529: return "XAUTH_ANSWER";
   default: return "Private use";
   }
 }
@@ -1266,17 +1418,22 @@ proto_register_isakmp(void)
                                               "ISAKMP", "isakmp");
 /*  proto_register_field_array(proto_isakmp, hf, array_length(hf));*/
   proto_register_subtree_array(ett, array_length(ett));
+
+  register_dissector("isakmp", dissect_isakmp, proto_isakmp);
 }
 
 void
 proto_reg_handoff_isakmp(void)
 {
+  dissector_handle_t isakmp_handle;
+
   /*
    * Get handle for the AH & ESP dissectors.
    */
   esp_handle = find_dissector("esp");
   ah_handle = find_dissector("ah");
 
-  dissector_add("udp.port", UDP_PORT_ISAKMP, dissect_isakmp, proto_isakmp);
-  dissector_add("tcp.port", TCP_PORT_ISAKMP, dissect_isakmp, proto_isakmp);
+  isakmp_handle = find_dissector("isakmp");
+  dissector_add("udp.port", UDP_PORT_ISAKMP, isakmp_handle);
+  dissector_add("tcp.port", TCP_PORT_ISAKMP, isakmp_handle);
 }