diff options
| author | Hannes Mezger <hannes.mezger@ascolab.com> | 2014-06-10 11:46:05 +0200 |
|---|---|---|
| committer | Alexis La Goutte <alexis.lagoutte@gmail.com> | 2014-06-11 12:15:39 +0000 |
| commit | e2ae387d9c9d2d38d1da788f39b8e65a22fff8dd (patch) | |
| tree | 7a1bb31c8ea5c4757afdf8d3a68f8d0ac9418b8e /plugins/opcua/opcua_application_layer.c | |
| parent | ddc275c562805757d5d971ccc31ef47dbc6df4bc (diff) | |
Fix NodeId and ExpandedNodeId encoding mask display
There are no 'URI' NodeIds, removed this option and changed GUID/Opaque to their correct values as stated in the UA specification.
Change-Id: Ibf9afdbd72f8ad336f5eb83ece5adbcce51e8365
Reviewed-on: https://code.wireshark.org/review/2122
Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
Diffstat (limited to 'plugins/opcua/opcua_application_layer.c')
| -rw-r--r-- | plugins/opcua/opcua_application_layer.c | 19 |
1 files changed, 8 insertions, 11 deletions
diff --git a/plugins/opcua/opcua_application_layer.c b/plugins/opcua/opcua_application_layer.c index 28920ae5a1..1c29778942 100644 --- a/plugins/opcua/opcua_application_layer.c +++ b/plugins/opcua/opcua_application_layer.c @@ -28,14 +28,12 @@ /** NodeId encoding mask table */ static const value_string g_nodeidmasks[] = { - { 0, "Two byte encoded Numeric" }, - { 1, "Four byte encoded Numeric" }, - { 2, "Numeric of arbitrary length" }, - { 3, "String" }, - { 4, "URI" }, - { 5, "GUID" }, - { 6, "Opaque" }, - { 0x80, "UriMask" }, + { 0x00, "Two byte encoded Numeric" }, + { 0x01, "Four byte encoded Numeric" }, + { 0x02, "Numeric of arbitrary length" }, + { 0x03, "String" }, + { 0x04, "GUID" }, + { 0x05, "Opaque" }, { 0, NULL } }; @@ -101,9 +99,8 @@ int parseServiceNodeId(proto_tree *tree, tvbuff_t *tvb, gint *pOffset) iOffset+=4; break; case 0x03: /* string */ - case 0x04: /* uri */ - case 0x05: /* guid */ - case 0x06: /* byte string */ + case 0x04: /* guid */ + case 0x05: /* opaque*/ /* NOT USED */ break; }; |