diff options
| author | Anish Bhatt <anish@chelsio.com> | 2015-01-05 22:50:44 -0800 |
|---|---|---|
| committer | Alexis La Goutte <alexis.lagoutte@gmail.com> | 2015-01-06 07:02:21 +0000 |
| commit | 124272ec097852c0c0efa587cb1cb1bf8047ff73 (patch) | |
| tree | 35c0b33d70826f55ef605651ab7b12278c60bf00 /plugins | |
| parent | eb98b04ac12efe01d1abddf866b18f31101533ee (diff) | |
[PROFINET] v2: Fix swapped MaintenanceRequired & MaintenanceDemanded bits
Fix the correct code path this time.
Bug: 10753
Change-Id: I7bc2e74c91868986303e8b41b25e43a665be1860
Reviewed-on: https://code.wireshark.org/review/6335
Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
Diffstat (limited to 'plugins')
| -rw-r--r-- | plugins/profinet/packet-dcerpc-pn-io.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/plugins/profinet/packet-dcerpc-pn-io.c b/plugins/profinet/packet-dcerpc-pn-io.c index 7d870ed783..1c95189995 100644 --- a/plugins/profinet/packet-dcerpc-pn-io.c +++ b/plugins/profinet/packet-dcerpc-pn-io.c @@ -2244,10 +2244,13 @@ static const value_string pn_io_channel_properties_accumulative_vals[] = { { 0, NULL } }; +/* We are reading this as a two bit value, but the spec specifies each bit + * separately. Beware endianness when reading spec + */ static const value_string pn_io_channel_properties_maintenance[] = { { 0x0000, "Failure" }, - { 0x0001, "Maintenance demanded" }, - { 0x0002, "Maintenance required" }, + { 0x0001, "Maintenance required" }, + { 0x0002, "Maintenance demanded" }, { 0x0003, "see QualifiedChannelQualifier" }, { 0, NULL } }; |