For proto_tree_add_item(..., proto_xxx, ...)use ENC_NA as the encoding arg.
[obnox/wireshark/wip.git] / epan / dissectors / packet-coap.c
index daf5449f62fc513358a77853a98c3d47339dfd6b..31c3306f82d2a061a5a708dc239f4b7ef58a6ebc 100644 (file)
@@ -1,14 +1,14 @@
+#include <stdio.h>
 /* packet-coap.c
  * Routines for COAP packet disassembly
+ * draft-ietf-core-coap-07.txt
+ * draft-ietf-core-block-04.txt
+ * draft-ietf-core-observe-02.txt
+ * draft-ietf-core-link-format-06.txt
  * Shoichi Sakane <sakane@tanu.org>
  *
  * $Id$
  *
- * draft-core-coap-02.txt
- * draft-ietf-core-coap-02.txt
- * draft-ietf-core-block-00.txt
- * draft-ietf-core-observe-00.txt
- *
  * Wireshark - Network traffic analyzer
  * By Gerald Combs <gerald@wireshark.org>
  * Copyright 1998 Gerald Combs
@@ -36,6 +36,7 @@
 
 #include <epan/packet.h>
 #include <epan/prefs.h>
+#include <expert.h>
 
 static dissector_table_t media_type_dissector_table;
 
@@ -46,34 +47,42 @@ static int hf_coap_ttype            = -1;
 static int hf_coap_opt_count           = -1;
 static int hf_coap_code                        = -1;
 static int hf_coap_tid                 = -1;
-static int hf_coap_opt_type            = -1;
+static int hf_coap_opt_delta           = -1;
+static int hf_coap_opt_length          = -1;
 static int hf_coap_opt_ctype           = -1;
+static int hf_coap_opt_max_age         = -1;
+static int hf_coap_opt_proxy_uri       = -1;
 static int hf_coap_opt_etag            = -1;
-static int hf_coap_opt_uri_authority   = -1;
-static int hf_coap_opt_location                = -1;
+static int hf_coap_opt_uri_host        = -1;
+static int hf_coap_opt_location_path   = -1;
+static int hf_coap_opt_uri_port                = -1;
+static int hf_coap_opt_location_query  = -1;
 static int hf_coap_opt_uri_path                = -1;
-static int hf_coap_opt_opaque_bytes    = -1;
+static int hf_coap_opt_observe         = -1;
+static int hf_coap_opt_token           = -1;
+static int hf_coap_opt_accept          = -1;
+static int hf_coap_opt_if_match                = -1;
+static int hf_coap_opt_block_number    = -1;
+static int hf_coap_opt_block_mflag     = -1;
+static int hf_coap_opt_block_size      = -1;
+static int hf_coap_opt_uri_query       = -1;
+static int hf_coap_opt_if_none_match   = -1;
 
 static gint ett_coap                   = -1;
-static gint ett_coap_noop              = -1;
-static gint ett_coap_ctype             = -1;
-static gint ett_coap_max_age           = -1;
-static gint ett_coap_uri_scheme                = -1;
-static gint ett_coap_etag              = -1;
-static gint ett_coap_uri_authority     = -1;
-static gint ett_coap_location          = -1;
-static gint ett_coap_uri_path          = -1;
-static gint ett_coap_subscr_lifetime   = -1;
-static gint ett_coap_opaque_bytes      = -1;
-static gint ett_coap_block             = -1;
+static gint ett_coap_option            = -1;
 static gint ett_coap_payload           = -1;
 
-/* TODO: COAP port number will be assigned by IANA after the draft become a RFC */
-#define DEFAULT_COAP_PORT      61616
+/* COAP's IANA-assigned port number */
+#define DEFAULT_COAP_PORT      5683
 
 static const gchar *coap_content_type = NULL;
+static gint coap_content_type_value = ~0;
 static guint global_coap_port_number = DEFAULT_COAP_PORT;
 
+static gint block_number = ~0;
+static guint block_mflag = 0;
+static gchar uri_string[256]; /* 256 is probably enough to display in the screen */
+
 /*
  * Transaction Type
  */
@@ -90,6 +99,8 @@ static const value_string vals_ttype[] = {
  * Response Code
  */
 static const value_string vals_code[] = {
+       { 0, "Empty Message" },
+
        /* method code */
        { 1, "GET" },
        { 2, "POST" },
@@ -97,18 +108,28 @@ static const value_string vals_code[] = {
        { 4, "DELETE" },
 
        /* response code */
-       { 40,  "100 Continue" },
-       { 80,  "200 OK"},
-       { 81,  "201 Created"},
-       { 124, "304 Not Modified"},
-       { 160, "400 Bad Request"},
-       { 164, "404 Not Found"},
-       { 165, "405 Method Not Allowed"},
-       { 175, "415 Unsupported Media Type"},
-       { 200, "500 Internal Server Error"},
-       { 202, "502 Bad Gateway"},
-       { 203, "503 Service Unavailable"},
-       { 204, "504 Gateway Timeout"},
+       {  65, "2.01 Created" },
+       {  66, "2.02 Deleted" },
+       {  67, "2.03 Valid" },
+       {  68, "2.04 Changed" },
+       {  69, "2.05 Content" },
+       { 128, "4.00 Bad Request" },
+       { 129, "4.01 Unauthorized" },
+       { 130, "4.02 Bad Option" },
+       { 131, "4.03 Forbidden" },
+       { 132, "4.04 Not Found" },
+       { 133, "4.05 Method Not Allowed" },
+       { 136, "4.08 Request Entity Incomplete" },      /* core-block-03 */
+       { 140, "4.12 Precondition Failed" },
+       { 141, "4.13 Request Entity Too Large" },
+       { 143, "4.15 Unsupported Media Type" },
+       { 160, "5.00 Internal Server Error" },
+       { 161, "5.01 Not Implemented" },
+       { 162, "5.02 Bad Gateway" },
+       { 163, "5.03 Service Unavailable" },
+       { 164, "5.04 Gateway Timeout" },
+       { 165, "5.05 Proxying Not Supported" },
+
        { 0, NULL },
 };
 
@@ -119,93 +140,140 @@ static const value_string vals_code[] = {
  */
 #define COAP_OPT_CONTENT_TYPE  1
 #define COAP_OPT_MAX_AGE       2
+#define COAP_OPT_PROXY_URI     3
 #define COAP_OPT_ETAG          4
-#define COAP_OPT_URI_AUTHORITY 5
-#define COAP_OPT_LOCATION      6
+#define COAP_OPT_URI_HOST      5
+#define COAP_OPT_LOCATION_PATH 6
+#define COAP_OPT_URI_PORT      7
+#define COAP_OPT_LOCATION_QUERY        8
 #define COAP_OPT_URI_PATH      9
-#define COAP_OPT_SUBSCR_LIFETIME       10
-#define COAP_OPT_OPAQUE_BYTES  11
-#define COAP_OPT_BLOCK         13
+#define COAP_OPT_OBSERVE       10      /* core-observe */
+#define COAP_OPT_TOKEN         11
+#define COAP_OPT_ACCEPT                12
+#define COAP_OPT_IF_MATCH      13
+#define COAP_OPT_URI_QUERY     15
+#define COAP_OPT_BLOCK2                17      /* core-block-03 */
+#define COAP_OPT_BLOCK1                19      /* core-block-03 */
+#define COAP_OPT_IF_NONE_MATCH 21
 
 static const value_string vals_opt_type[] = {
        { COAP_OPT_CONTENT_TYPE, "Content-Type" },
        { COAP_OPT_MAX_AGE, "Max-age" },
+       { COAP_OPT_PROXY_URI, "Proxy-Uri" },
        { COAP_OPT_ETAG, "Etag" },
-       { COAP_OPT_URI_AUTHORITY, "Uri-Authority" },
-       { COAP_OPT_LOCATION, "Location" },
+       { COAP_OPT_URI_HOST, "Uri-Host" },
+       { COAP_OPT_LOCATION_PATH, "Location-Path" },
+       { COAP_OPT_URI_PORT, "Uri-Port" },
+       { COAP_OPT_LOCATION_QUERY, "Location-Query" },
        { COAP_OPT_URI_PATH, "Uri-Path" },
-       { COAP_OPT_SUBSCR_LIFETIME, "Subscription Lifetime" },
-       { COAP_OPT_OPAQUE_BYTES, "Opaque Bytes" },
-       { COAP_OPT_BLOCK, "Block" },
+       { COAP_OPT_OBSERVE, "Observe" },
+       { COAP_OPT_TOKEN, "Token" },
+       { COAP_OPT_ACCEPT, "Accept" },
+       { COAP_OPT_IF_MATCH, "If-Match" },
+       { COAP_OPT_URI_QUERY, "Uri-Query" },
+       { COAP_OPT_BLOCK2, "Block2" },
+       { COAP_OPT_BLOCK1, "Block1" },
+       { COAP_OPT_IF_NONE_MATCH, "If-None-Match" },
        { 0, NULL },
 };
 
 static const value_string vals_ctype[] = {
        { 0, "text/plain" },
-       { 1, "text/xml" },
-       { 2, "text/csv" },
-       { 3, "text/html" },
-       { 21, "image/gif" },
-       { 22, "image/jpeg" },
-       { 23, "image/png" },
-       { 24, "image/tiff" },
-       { 25, "audio/raw" },
-       { 26, "video/raw" },
        { 40, "application/link-format" },
        { 41, "application/xml" },
        { 42, "application/octet-stream" },
-       { 43, "application/rdf+xml" },
-       { 44, "application/soap+xml" },
-       { 45, "application/atom+xml" },
-       { 46, "application/xmpp+xml" },
        { 47, "application/exi" },
-       { 48, "application/x-bxml" },
-       { 49, "application/fastinfoset" },
-       { 50, "application/soap+fastinfoset" },
-       { 51, "application/json" },
+       { 50, "application/json" },
        { 0, NULL },
 };
 
 void proto_reg_handoff_coap(void);
 
-/* the value of opt_length should be checked out of this function */
-static void
-dissect_coap_opt_time(tvbuff_t *tvb, proto_tree *subtree, int offset, gint opt_length, char *str)
+static int
+coap_is_str_ipv6addr(guint8 *str)
 {
-       guint time = 0;
+       size_t len = strlen(str);
+       int colon = 0;
 
-       switch (opt_length) {
-       case 0:
-               time = 0;
-               break;
-       case 1:
-               time = (guint)tvb_get_guint8(tvb, offset);
-               break;
-       case 2:
-               time = (guint)tvb_get_ntohs(tvb, offset);
-               break;
-       case 3:
-               time = (guint)tvb_get_ntoh24(tvb, offset);
+       while (len--) {
+               if (*str++ == ':')
+                       colon++;
+       }
+
+       return colon > 1 ? 1 : 0;
+}
+
+static void
+dissect_coap_opt_string(tvbuff_t *tvb, packet_info *pinfo _U_, proto_tree *subtree, gint offset, gint opt_length, int hfindex, int opt_code)
+{
+       guint8 *hoststr = NULL;
+
+       proto_tree_add_item(subtree, hfindex, tvb, offset, opt_length, FALSE);
+
+       /* forming a uri-string */
+       switch (opt_code) {
+       case COAP_OPT_URI_HOST:
+               g_strlcat(uri_string, "coap://", sizeof(uri_string));
+               hoststr = tvb_get_ephemeral_string(tvb, offset, opt_length);
+               /* if the string looks an IPv6 address, it has to be enclosed by brackets. */
+               if (coap_is_str_ipv6addr(hoststr)) {
+                       g_strlcat(uri_string, "[", sizeof(uri_string));
+                       g_strlcat(uri_string, hoststr, sizeof(uri_string));
+                       g_strlcat(uri_string, "]", sizeof(uri_string));
+               } else
+                       g_strlcat(uri_string, hoststr, sizeof(uri_string));
                break;
-       case 4:
-               time = (guint)tvb_get_ntohl(tvb, offset);
+       case COAP_OPT_URI_PATH:
+               g_strlcat(uri_string, "/", sizeof(uri_string));
+               g_strlcat(uri_string, tvb_get_ephemeral_string(tvb, offset, opt_length), sizeof(uri_string));
                break;
-       default:
-               proto_tree_add_text(subtree, tvb, 0, 0, "Invalid length: %d", opt_length);
+       case COAP_OPT_URI_QUERY:
+               g_strlcat(uri_string, "/?", sizeof(uri_string));
+               g_strlcat(uri_string, tvb_get_ephemeral_string(tvb, offset, opt_length), sizeof(uri_string));
                break;
        }
-       proto_tree_add_text(subtree, tvb, offset, opt_length, "%s: %d (s)", str, time);
+}
+
+static void
+dissect_coap_opt_ctype(tvbuff_t *tvb, packet_info *pinfo, proto_tree *subtree, gint offset, gint opt_length, int hfindex)
+{
+       guint32 opt_ctype = 0;
+
+       if (opt_length != 1) {
+               expert_add_info_format(pinfo, subtree, PI_MALFORMED, PI_WARN, "Invalid Option Length: %d", opt_length);
+               return;
+       }
+
+       opt_ctype = tvb_get_guint8(tvb, offset);
+       coap_content_type_value = (gint)opt_ctype;
+       coap_content_type = val_to_str(opt_ctype, vals_ctype, "Unknown %d");
+
+       proto_tree_add_item(subtree, hfindex, tvb, offset, 1, FALSE);
+}
+
+static void
+dissect_coap_opt_time(tvbuff_t *tvb, packet_info *pinfo, proto_tree *subtree, gint offset, gint opt_length, int hfindex)
+{
+       proto_item *item = NULL;
+
+       if (opt_length <= 0 || opt_length > 4) {
+               expert_add_info_format(pinfo, subtree, PI_MALFORMED, PI_WARN, "Invalid Option Length: %d", opt_length);
+               return;
+       }
+
+       item = proto_tree_add_item(subtree, hfindex, tvb, offset, opt_length, FALSE);
+       proto_item_append_text(item, " (s)");
 
        return;
 }
 
 static void
-dissect_coap_opt_block(tvbuff_t *tvb, proto_tree *subtree, int offset, gint opt_length)
+dissect_coap_opt_block(tvbuff_t *tvb, packet_info *pinfo, proto_tree *subtree, gint offset, gint opt_length, int hfindex)
 {
-       guint block_number = 0;
-       guint more_flag = 0;
-       guint block_size = 0;
        guint8 val = 0;
+       guint encoded_block_size = 0;
+       guint block_size;
+       proto_item *item = NULL;
 
        switch (opt_length) {
        case 1:
@@ -218,16 +286,54 @@ dissect_coap_opt_block(tvbuff_t *tvb, proto_tree *subtree, int offset, gint opt_
                block_number = (guint)(tvb_get_ntoh24(tvb, offset) >> 4);
                break;
        default:
-               proto_tree_add_text(subtree, tvb, 0, 0, "Invalid length: %d", opt_length);
+               expert_add_info_format(pinfo, subtree, PI_MALFORMED, PI_WARN, "Invalid Option Length: %d", opt_length);
                return;
        }
 
        val = tvb_get_guint8(tvb, offset + opt_length - 1) & 0x0f;
-       more_flag = (val & 0x08) >> 3;
-       block_size = val & 0x07;
-       block_size = 2^(block_size + 4);
+       encoded_block_size = val & 0x07;
+       block_mflag = val & 0x08;
+
+       proto_tree_add_int(subtree, hf_coap_opt_block_number, tvb, offset, opt_length, block_number);
+       proto_tree_add_item(subtree, hfindex, tvb, offset + opt_length - 1, 1, FALSE);
 
-       proto_tree_add_text(subtree, tvb, offset, opt_length, "Block Number:%d, More Flag:%d, Block Size:%d", block_number, more_flag, block_size);
+       block_size = 1 << (encoded_block_size + 4);
+       item = proto_tree_add_item(subtree, hf_coap_opt_block_size, tvb, offset + opt_length - 1, 1, ENC_BIG_ENDIAN);
+       proto_item_append_text(item, ", Result: %d", block_size);
+}
+
+static void
+dissect_coap_opt_port(tvbuff_t *tvb, packet_info *pinfo, proto_tree *subtree, gint offset, gint opt_length, int hfindex)
+{
+       proto_item *item = NULL;
+       char portstr[6];
+
+       memset(portstr, '\0', sizeof(portstr));
+
+       switch (opt_length) {
+       case 0:
+               item = proto_tree_add_uint(subtree, hfindex, tvb, offset, opt_length, pinfo->destport);
+               proto_item_append_text(item, " (default)");
+               return;
+       case 1:
+               g_snprintf(portstr, sizeof(portstr), "%u", tvb_get_guint8(tvb, offset));
+               break;
+       case 2:
+               g_snprintf(portstr, sizeof(portstr), "%u", tvb_get_ntohs(tvb, offset));
+               break;
+       default:
+               expert_add_info_format(pinfo, subtree, PI_MALFORMED, PI_WARN, "Invalid Option Length: %d", opt_length);
+               return;
+       }
+       (void)proto_tree_add_item(subtree, hfindex, tvb, offset, opt_length, FALSE);
+
+       /* forming a uri-string */
+       if (uri_string[0] == '\0')
+               g_strlcat(uri_string, ep_address_to_str(&pinfo->net_dst), sizeof(uri_string));
+       g_strlcat(uri_string, ":", sizeof(uri_string));
+       g_strlcat(uri_string, portstr, sizeof(uri_string));
+
+       return;
 }
 
 /*
@@ -235,17 +341,25 @@ dissect_coap_opt_block(tvbuff_t *tvb, proto_tree *subtree, int offset, gint opt_
  * return the total length of the option including the header (e.g. delta and length).
  */
 static int
-dissect_coap_options(tvbuff_t *tvb, proto_tree *coap_tree, proto_tree *parent_tree _U_, int offset, guint8 opt_count, guint8 *opt_code)
+dissect_coap_options(tvbuff_t *tvb, packet_info *pinfo, proto_tree *coap_tree, gint offset, guint8 opt_count, guint8 *opt_code)
 {
        guint8 opt_delta;
-       guint32 opt_ctype = 0;
        gint opt_length;
        proto_tree *subtree = NULL;
        proto_item *item = NULL;
-       int opt_hlen = 0;
+       gint opt_hlen = 0;
+       tvbuff_t *tvb_lenbuf = NULL;
 
        opt_delta = (tvb_get_guint8(tvb, offset) & 0xf0) >> 4;
        *opt_code += opt_delta;
+
+       /*
+        * Length:
+        *   Normally Length is a 4-bit unsigned integer
+        *   allowing values of 0-14 octets.  When the length is 15 or more,
+        *   another byte is added as an 8-bit unsigned integer plus 15
+        *   allowing values of 15-270 octets.
+        */
        opt_length = (tvb_get_guint8(tvb, offset) & 0x0f);
        opt_hlen = 1;
        if (opt_length == 0x0f) {
@@ -254,57 +368,73 @@ dissect_coap_options(tvbuff_t *tvb, proto_tree *coap_tree, proto_tree *parent_tr
        }
 
        item = proto_tree_add_text(coap_tree, tvb, offset, opt_hlen + opt_length,
-                                  "Option #%u (Length: %u) %s", opt_count, opt_length,
-                                  val_to_str(*opt_code, vals_opt_type, "Unknown Option Type %u"));
+                                  "Option #%u: %s (Type: %u)",
+                                  opt_count, val_to_str(*opt_code, vals_opt_type, *opt_code % 14 == 0 ? "No-Op" : "Unknown Option"), *opt_code);
+
+       subtree = proto_item_add_subtree(item, ett_coap_option);
+       proto_tree_add_item(subtree, hf_coap_opt_delta, tvb, offset, 1, ENC_BIG_ENDIAN);
+
+       tvb_lenbuf = tvb_new_subset(tvb, offset, opt_hlen, opt_hlen);
+       proto_tree_add_uint_bits_format_value(subtree, hf_coap_opt_length, tvb_lenbuf, 4, opt_hlen == 1 ? 4 : 12, opt_length, "%d", opt_length);
        offset += opt_hlen;
 
-       /* if opt_code is a multiple of 14, that means the option is a noop option */
-       if (*opt_code % 14 == 0) {
-               subtree = proto_item_add_subtree(item, ett_coap_noop);
-               proto_tree_add_text(subtree, tvb, 0, 0, "No-Op option");
-       } else {
-               switch (*opt_code) {
-               case COAP_OPT_CONTENT_TYPE:
-                       subtree = proto_item_add_subtree(item, ett_coap_ctype);
-                       opt_ctype = tvb_get_guint8(tvb, offset);
-                       coap_content_type = val_to_str(opt_ctype, vals_ctype, "Unknown %d");
-                       proto_tree_add_item(subtree, hf_coap_opt_ctype, tvb, offset, 1, FALSE);
-                       break;
-               case COAP_OPT_MAX_AGE:
-                       subtree = proto_item_add_subtree(item, ett_coap_max_age);
-                       dissect_coap_opt_time(tvb, subtree, offset, opt_length, "Max-age");
-                       break;
-               case COAP_OPT_SUBSCR_LIFETIME:
-                       subtree = proto_item_add_subtree(item, ett_coap_subscr_lifetime);
-                       dissect_coap_opt_time(tvb, subtree, offset, opt_length, "Subscription Lifetime");
-                       break;
-               case COAP_OPT_ETAG:
-                       subtree = proto_item_add_subtree(item, ett_coap_etag);
-                       proto_tree_add_item(subtree, hf_coap_opt_etag, tvb, offset, opt_length, FALSE);
-                       break;
-               case COAP_OPT_URI_AUTHORITY:
-                       subtree = proto_item_add_subtree(item, ett_coap_uri_authority);
-                       proto_tree_add_item(subtree, hf_coap_opt_uri_authority, tvb, offset, opt_length, FALSE);
-                       break;
-               case COAP_OPT_LOCATION:
-                       subtree = proto_item_add_subtree(item, ett_coap_location);
-                       proto_tree_add_item(subtree, hf_coap_opt_location, tvb, offset, opt_length, FALSE);
-                       break;
-               case COAP_OPT_URI_PATH:
-                       subtree = proto_item_add_subtree(item, ett_coap_uri_path);
-                       proto_tree_add_item(subtree, hf_coap_opt_uri_path, tvb, offset, opt_length, FALSE);
-                       break;
-               case COAP_OPT_OPAQUE_BYTES:
-                       /* TODO: implement it after a draft will be published */
-                       break;
-               case COAP_OPT_BLOCK:
-                       subtree = proto_item_add_subtree(item, ett_coap_block);
-                       dissect_coap_opt_block(tvb, subtree, offset, opt_length);
-                       break;
-               default:
-                       proto_tree_add_text(subtree, tvb, 0, 0, "Unkown Option Type");
-               }
+       switch (*opt_code) {
+       case COAP_OPT_CONTENT_TYPE:
+               dissect_coap_opt_ctype(tvb, pinfo, subtree, offset, opt_length, hf_coap_opt_ctype);
+               break;
+       case COAP_OPT_MAX_AGE:
+               dissect_coap_opt_time(tvb, pinfo, subtree, offset, opt_length, hf_coap_opt_max_age);
+               break;
+       case COAP_OPT_PROXY_URI:
+               dissect_coap_opt_string(tvb, pinfo, subtree, offset, opt_length, hf_coap_opt_proxy_uri, COAP_OPT_PROXY_URI);
+               break;
+       case COAP_OPT_ETAG:
+               dissect_coap_opt_string(tvb, pinfo, subtree, offset, opt_length, hf_coap_opt_etag, COAP_OPT_ETAG);
+               break;
+       case COAP_OPT_URI_HOST:
+               dissect_coap_opt_string(tvb, pinfo, subtree, offset, opt_length, hf_coap_opt_uri_host, COAP_OPT_URI_HOST);
+               break;
+       case COAP_OPT_LOCATION_PATH:
+               dissect_coap_opt_string(tvb, pinfo, subtree, offset, opt_length, hf_coap_opt_location_path, COAP_OPT_LOCATION_PATH);
+               break;
+       case COAP_OPT_URI_PORT:
+               dissect_coap_opt_port(tvb, pinfo, subtree, offset, opt_length, hf_coap_opt_uri_port);
+               break;
+       case COAP_OPT_LOCATION_QUERY:
+               dissect_coap_opt_string(tvb, pinfo, subtree, offset, opt_length, hf_coap_opt_location_query, COAP_OPT_LOCATION_QUERY);
+               break;
+       case COAP_OPT_URI_PATH:
+               dissect_coap_opt_string(tvb, pinfo, subtree, offset, opt_length, hf_coap_opt_uri_path, COAP_OPT_URI_PATH);
+               break;
+       case COAP_OPT_OBSERVE:
+               dissect_coap_opt_time(tvb, pinfo, subtree, offset, opt_length, hf_coap_opt_observe);
+               break;
+       case COAP_OPT_TOKEN:
+               dissect_coap_opt_string(tvb, pinfo, subtree, offset, opt_length, hf_coap_opt_token, COAP_OPT_TOKEN);
+               break;
+       case COAP_OPT_ACCEPT:
+               dissect_coap_opt_ctype(tvb, pinfo, subtree, offset, opt_length, hf_coap_opt_accept);
+               break;
+       case COAP_OPT_IF_MATCH:
+               dissect_coap_opt_string(tvb, pinfo, subtree, offset, opt_length, hf_coap_opt_if_match, COAP_OPT_IF_MATCH);
+               break;
+       case COAP_OPT_URI_QUERY:
+               dissect_coap_opt_string(tvb, pinfo, subtree, offset, opt_length, hf_coap_opt_uri_query, COAP_OPT_URI_QUERY);
+               break;
+       case COAP_OPT_BLOCK2:
+               dissect_coap_opt_block(tvb, pinfo, subtree, offset, opt_length, hf_coap_opt_block_mflag);
+               break;
+       case COAP_OPT_BLOCK1:
+               dissect_coap_opt_block(tvb, pinfo, subtree, offset, opt_length, hf_coap_opt_block_mflag);
+               break;
+       case COAP_OPT_IF_NONE_MATCH:
+               dissect_coap_opt_string(tvb, pinfo, subtree, offset, opt_length, hf_coap_opt_if_none_match, COAP_OPT_IF_NONE_MATCH);
+               break;
+       default:
+               /* In case of unknown opt_code, just ignore it here. A message is displayed beforehand. */
+               break;
        }
+
        return offset + opt_length;
 }
 
@@ -336,41 +466,53 @@ dissect_coap(tvbuff_t *tvb, packet_info *pinfo, proto_tree *parent_tree)
         */
        coap_length = tvb_reported_length(tvb);
        coap_content_type = NULL;
+       coap_content_type_value = ~0;
 
-       coap_root = proto_tree_add_item(parent_tree, proto_coap, tvb, offset, -1, FALSE);
+       coap_root = proto_tree_add_item(parent_tree, proto_coap, tvb, offset, -1, ENC_NA);
        coap_tree = proto_item_add_subtree(coap_root, ett_coap);
 
-       proto_tree_add_item(coap_tree, hf_coap_version, tvb, offset, 1, FALSE);
+       proto_tree_add_item(coap_tree, hf_coap_version, tvb, offset, 1, ENC_BIG_ENDIAN);
 
-       proto_tree_add_item(coap_tree, hf_coap_ttype, tvb, offset, 1, FALSE);
+       proto_tree_add_item(coap_tree, hf_coap_ttype, tvb, offset, 1, ENC_BIG_ENDIAN);
        ttype = (tvb_get_guint8(tvb, offset) & 0x30) >> 4;
        col_add_fstr(pinfo->cinfo, COL_INFO, "%s", val_to_str(ttype, vals_ttype, "Unknown %d"));
 
-       proto_tree_add_item(coap_tree, hf_coap_opt_count, tvb, offset, 1, FALSE);
+       proto_tree_add_item(coap_tree, hf_coap_opt_count, tvb, offset, 1, ENC_BIG_ENDIAN);
        opt_count = tvb_get_guint8(tvb, offset) & 0x0f;
        offset += 1;
 
-       proto_tree_add_item(coap_tree, hf_coap_code, tvb, offset, 1, FALSE);
+       proto_tree_add_item(coap_tree, hf_coap_code, tvb, offset, 1, ENC_BIG_ENDIAN);
        code = tvb_get_guint8(tvb, offset);
        col_append_fstr(pinfo->cinfo, COL_INFO, ", %s", val_to_str(code, vals_code, "Unknown %d"));
        offset += 1;
 
-       proto_tree_add_item(coap_tree, hf_coap_tid, tvb, offset, 2, FALSE);
+       proto_tree_add_item(coap_tree, hf_coap_tid, tvb, offset, 2, ENC_BIG_ENDIAN);
        tid = tvb_get_ntohs(tvb, offset);
        offset += 2;
 
        /* append the header information */
        proto_item_append_text(coap_tree, ", TID: %u, Length: %u", tid, coap_length);
 
+       /* initialize the external value */
+       block_number = ~0;
+       block_mflag = 0;
+       uri_string[0] = 0;
+
        /* dissect the options */
        for (i = 1; i <= opt_count; i++) {
-               offset = dissect_coap_options(tvb, coap_tree, parent_tree, offset, i, &opt_code);
+               offset = dissect_coap_options(tvb, pinfo, coap_tree, offset, i, &opt_code);
                if (coap_length < offset) {
                        /* error */
                        proto_tree_add_text(coap_tree, tvb, 0, 0, "Invalid length: coap_length(%d) < offset(%d)", coap_length, offset);
                        return;
                }
        }
+       if (block_number != ~0) {
+               col_append_fstr(pinfo->cinfo, COL_INFO, ", %sBlock #%d", block_mflag ? "" : "End of ", block_number);
+       }
+       if (uri_string[0] != '\0') {
+               col_append_fstr(pinfo->cinfo, COL_INFO, ", %s", uri_string);
+       }
 
        /* dissect the payload */
        if (coap_length > offset) {
@@ -381,24 +523,17 @@ dissect_coap(tvbuff_t *tvb, packet_info *pinfo, proto_tree *parent_tree)
                char *ctype_str_default = "";
 
                /*
-                * TODO:
-                * currently, coap_content_type is used to distinguish whether
-                * the content-type was specified.  If we need to properly handle
-                * the case when the type was unknown, we need another flag.
+                * TODO: should the content type be canonicalized ?
+                * currently assuming it be small.
                 */
-               if (coap_content_type == NULL) {
+               if (coap_content_type_value == ~0) {
                        /* default: coap-02 section 3.2.1 */
-                       /* when it's NULL, "text/plain" is set anyway */
                        coap_content_type = "text/plain";
                        ctype_str_default = " (default)";
                }
-               /*
-                * TODO: should the content type be canonicalized,
-                * currently assuming it be small ?
-                */
 
                payload_item = proto_tree_add_text(coap_tree, tvb, offset, -1, "Payload Content-Type: %s%s, Length: %u, offset: %u",
-                                                  coap_content_type, ctype_str_default, payload_length, offset);
+                                                  coap_content_type, ctype_str_default, payload_length, offset);
                payload_tree = proto_item_add_subtree(payload_item, ett_coap_payload);
                payload_tvb = tvb_new_subset(tvb, offset, payload_length, payload_length);
 
@@ -413,33 +548,36 @@ void
 proto_register_coap(void)
 {
        static hf_register_info hf[] = {
-           { &hf_coap_version, { "Version", "coap.version", FT_UINT8, BASE_DEC, NULL, 0xc0, "COAP Version", HFILL }},
-           { &hf_coap_ttype, { "Type", "coap.type", FT_UINT8, BASE_DEC, VALS(&vals_ttype), 0x30, "COAP Transaction Type", HFILL }},
-           { &hf_coap_opt_count, { "Option Count", "coap.optcount", FT_UINT8, BASE_DEC, NULL, 0x0f, "COAP Option Count", HFILL }},
-           { &hf_coap_code, { "Code", "coap.code", FT_UINT8, BASE_DEC, VALS(&vals_code), 0x0, "COAP Method or Response Code", HFILL }},
-           { &hf_coap_tid, { "Transaction ID", "coap.tid", FT_UINT16, BASE_DEC, NULL, 0x0, "COAP Transaction ID", HFILL }},
-           { &hf_coap_opt_type, { "Option Type", "coap.opt.opt_type", FT_UINT8, BASE_DEC, VALS(&vals_opt_type), 0x0, "COAP Option Type", HFILL }},
-           { &hf_coap_opt_ctype, { "Content-type", "coap.opt.ctype", FT_UINT8, BASE_DEC, VALS(&vals_ctype), 0x0, "COAP Media Type", HFILL }},
-           { &hf_coap_opt_etag, { "Etag", "coap.opt.etag", FT_BYTES, BASE_NONE, NULL, 0x0, "COAP Etag", HFILL }},
-           { &hf_coap_opt_uri_authority, { "Uri-Authority", "coap.opt.uri_auth", FT_STRING, BASE_NONE, NULL, 0x0, "COAP Uri-Authority", HFILL }},
-           { &hf_coap_opt_location, { "Location", "coap.opt.location", FT_STRING, BASE_NONE, NULL, 0x0, "COAP Location", HFILL }},
-           { &hf_coap_opt_uri_path, { "Uri-Path", "coap.opt.uri_path", FT_STRING, BASE_NONE, NULL, 0x0, "COAP Uri-Path", HFILL }},
-           { &hf_coap_opt_opaque_bytes, { "Opaque Bytes", "coap.opt.opaquebytes", FT_BYTES, BASE_NONE, NULL, 0x0, "COAP Opaque Bytes", HFILL }},
+               { &hf_coap_version, { "Version", "coap.version", FT_UINT8, BASE_DEC, NULL, 0xc0, "COAP Version", HFILL }},
+               { &hf_coap_ttype, { "Type", "coap.type", FT_UINT8, BASE_DEC, VALS(vals_ttype), 0x30, "COAP Transaction Type", HFILL }},
+               { &hf_coap_opt_count, { "Option Count", "coap.optcount", FT_UINT8, BASE_DEC, NULL, 0x0f, "COAP Option Count", HFILL }},
+               { &hf_coap_code, { "Code", "coap.code", FT_UINT8, BASE_DEC, VALS(vals_code), 0x0, "COAP Method or Response Code", HFILL }},
+               { &hf_coap_tid, { "Transaction ID", "coap.tid", FT_UINT16, BASE_DEC, NULL, 0x0, "COAP Transaction ID", HFILL }},
+               { &hf_coap_opt_delta, { "Delta", "coap.opt.delta", FT_UINT8, BASE_DEC, NULL, 0xf0, "COAP Option Delta", HFILL }},
+               { &hf_coap_opt_length, { "Length", "coap.opt.length", FT_UINT16, BASE_DEC, NULL, 0x0, "COAP Option Length", HFILL }},
+               { &hf_coap_opt_ctype, { "Content-type", "coap.opt.ctype", FT_UINT8, BASE_DEC, VALS(vals_ctype), 0x0, "COAP Content Type", HFILL }},
+               { &hf_coap_opt_max_age, { "Max-age", "coap.opt.max_age", FT_UINT32, BASE_DEC, NULL, 0x0, "COAP Max-age", HFILL }},
+               { &hf_coap_opt_proxy_uri, { "Proxy-Uri", "coap.opt.proxy_uri", FT_STRING, BASE_NONE, NULL, 0x0, "COAP Proxy-Uri", HFILL }},
+               { &hf_coap_opt_etag, { "Etag", "coap.opt.etag", FT_BYTES, BASE_NONE, NULL, 0x0, "COAP Etag", HFILL }},
+               { &hf_coap_opt_uri_host, { "Uri-Host", "coap.opt.uri_host", FT_STRING, BASE_NONE, NULL, 0x0, "COAP Uri-Host", HFILL }},
+               { &hf_coap_opt_location_path, { "Location-Path", "coap.opt.location_path", FT_STRING, BASE_NONE, NULL, 0x0, "COAP URI Path", HFILL }},
+               { &hf_coap_opt_uri_port, { "Uri-Port", "coap.opt.uri_port", FT_UINT16, BASE_DEC, NULL, 0x0, "COAP Uri-Port", HFILL }},
+               { &hf_coap_opt_location_query, { "Location-Query", "coap.opt.location_query", FT_STRING, BASE_NONE, NULL, 0x0, "COAP URI Query", HFILL }},
+               { &hf_coap_opt_uri_path, { "Uri-Path", "coap.opt.uri_path", FT_STRING, BASE_NONE, NULL, 0x0, "COAP Uri-Path", HFILL }},
+               { &hf_coap_opt_observe, { "Lifetime", "coap.opt.subscr_lifetime", FT_INT32, BASE_DEC, NULL, 0x0, "COAP Observe", HFILL }},
+               { &hf_coap_opt_token, { "Token", "coap.opt.token", FT_BYTES, BASE_NONE, NULL, 0x0, "COAP Token", HFILL }},
+               { &hf_coap_opt_accept, { "Accept", "coap.opt.accept", FT_UINT8, BASE_DEC, VALS(vals_ctype), 0x0, "COAP Acceptable Content Type", HFILL }},
+               { &hf_coap_opt_if_match, { "If-Match", "coap.opt.if_match", FT_BYTES, BASE_NONE, NULL, 0x0, "COAP If-Match", HFILL }},
+               { &hf_coap_opt_block_number, { "Block Number", "coap.opt.block_number", FT_INT32, BASE_DEC, NULL, 0x0, "COAP Block Number", HFILL }},
+               { &hf_coap_opt_block_mflag, { "More Flag", "coap.opt.block_mflag", FT_UINT8, BASE_DEC, NULL, 0x08, "COAP Block More Size", HFILL }},
+               { &hf_coap_opt_block_size, { "Encoded Block Size", "coap.opt.block_size", FT_UINT8, BASE_DEC, NULL, 0x07, "COAP Encoded Block Size", HFILL }},
+               { &hf_coap_opt_uri_query, { "Uri-Query", "coap.opt.uri_query", FT_STRING, BASE_NONE, NULL, 0x0, "COAP Uri-Query", HFILL }},
+               { &hf_coap_opt_if_none_match, { "If-None-Match", "coap.opt.if_none_match", FT_BYTES, BASE_NONE, NULL, 0x0, "COAP If-None-Match", HFILL }},
        };
 
        static gint *ett[] = {
                &ett_coap,
-               &ett_coap_noop,
-               &ett_coap_ctype,
-               &ett_coap_max_age,
-               &ett_coap_uri_scheme,
-               &ett_coap_etag,
-               &ett_coap_uri_authority,
-               &ett_coap_location,
-               &ett_coap_uri_path,
-               &ett_coap_subscr_lifetime,
-               &ett_coap_opaque_bytes,
-               &ett_coap_block,
+               &ett_coap_option,
                &ett_coap_payload,
        };
 
@@ -465,18 +603,18 @@ proto_reg_handoff_coap(void)
 {
        static gboolean coap_prefs_initialized = FALSE;
        static dissector_handle_t coap_handle;
-       static guint    coap_port_number;
+       static guint coap_port_number;
 
        if (!coap_prefs_initialized) {
                coap_handle = find_dissector("coap");
                media_type_dissector_table = find_dissector_table("media_type");
                coap_prefs_initialized = TRUE;
        } else {
-               dissector_delete("udp.port", coap_port_number, coap_handle);
-               dissector_delete("tcp.port", coap_port_number, coap_handle);
+               dissector_delete_uint("udp.port", coap_port_number, coap_handle);
+               dissector_delete_uint("tcp.port", coap_port_number, coap_handle);
        }
 
        coap_port_number = global_coap_port_number;
-       dissector_add("udp.port", coap_port_number, coap_handle);
-       dissector_add("tcp.port", coap_port_number, coap_handle);
+       dissector_add_uint("udp.port", coap_port_number, coap_handle);
+       dissector_add_uint("tcp.port", coap_port_number, coap_handle);
 }