aboutsummaryrefslogtreecommitdiffstats
path: root/epan/dissectors
AgeCommit message (Collapse)AuthorFilesLines
2017-11-12ssl-utils: Add export ssl_data_allocAlexis La Goutte2-1/+7
For reuse on QUIC dissector Change-Id: I005fc3fe7a559fdb7a62e3d40450a4a6b25bf2f5 Reviewed-on: https://code.wireshark.org/review/24383 Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2017-11-12Fix "might be clobbered by 'longjmp' or 'vfork' warningUli Heilmeier1-2/+2
Change-Id: I51ec17b5eabe7b0f9fc68f4cf599ca55b8e4631d Reviewed-on: https://code.wireshark.org/review/24385 Reviewed-by: Anders Broman <a.broman58@gmail.com>
2017-11-12[PFCP] Update to 3GPP TS 29.244 V14.1.0 (2017-09)Joakim Karlsson1-36/+347
Change-Id: I69ad8c9a852f400e2918c4eca7365c719df734f5 Reviewed-on: https://code.wireshark.org/review/24380 Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2017-11-12ieee1905: Use BASE_UNIT_STRING and &units_dbm rather than appendingRichard Sharpe1-31/+16
the string 'dBm'. Change-Id: I5f4ddcf98902c53f6f014e64b51eb4c3c263e7e1 Reviewed-on: https://code.wireshark.org/review/24368 Petri-Dish: Richard Sharpe <realrichardsharpe@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Michael Mann <mmann78@netscape.net>
2017-11-11mqtt: Whitespace and indent fixesStig Bjørlykke1-311/+322
Change-Id: Ie6470cd20871b0aac17d19c67aba4b7229f906e6 Reviewed-on: https://code.wireshark.org/review/24367 Petri-Dish: Stig Bjørlykke <stig@bjorlykke.org> Tested-by: Petri Dish Buildbot Reviewed-by: Stig Bjørlykke <stig@bjorlykke.org>
2017-11-11autotools: Be more consistent with variable namesJoão Valverde1-7/+7
Change-Id: I644fbe7796a83c98861a4ebb9200ee41de27c897 Reviewed-on: https://code.wireshark.org/review/24350 Reviewed-by: Anders Broman <a.broman58@gmail.com>
2017-11-11Wi-SUN Protocol dissection and IEEE 802.15.4 IE cleanup.Owen Kirby5-479/+1591
This patch adds dissection for the Wi-SUN Field Area Network standard. Wi-SUN packets are encoded as a collection of IEEE 802.15.4 information elements. This required a bit of refactoring in how the IE dissector table is handled so that external protocols could supply their own dissectors. While I was working with the IEs, I also added support for the TSCH Timeslot and Global Time IEs from the IEEE 802.15.4 standard in addition to some general cleanup. Change-Id: I2858e4ab577756568e33b86adfe282967899abd5 Reviewed-on: https://code.wireshark.org/review/24331 Petri-Dish: Michael Mann <mmann78@netscape.net> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2017-11-11WPS: no previous prototype for ‘proto_reg_handoff_wps’ ↵Alexis La Goutte1-0/+1
[-Wmissing-prototypes] Change-Id: Icb5220af362a25f081f0b5d33f92a9013723bca9 Reviewed-on: https://code.wireshark.org/review/24364 Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com> Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2017-11-11RIP: no previous prototype for ‘proto_reg_handoff_rip’ ↵Alexis La Goutte1-0/+1
[-Wmissing-prototypes] Change-Id: I49e6ecec2e3998ff212069e88618bea36935c894 Reviewed-on: https://code.wireshark.org/review/24363 Reviewed-by: Anders Broman <a.broman58@gmail.com>
2017-11-11[PFCP] OHC: fix Port number field presentJoakim Karlsson1-4/+13
Port number should only be included with UDP/IP header Change-Id: Ie2655d2b6f1b755bff47e74723f36b1ee87170f9 Reviewed-on: https://code.wireshark.org/review/24361 Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com> Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
2017-11-11[PFCP] display changesJoakim Karlsson1-4/+4
For readability, DEC<->HEX and "IE length" Change-Id: I753d8f6cba43a6b34fd06b49b1ffa96f13790c67 Reviewed-on: https://code.wireshark.org/review/24360 Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
2017-11-11[PFCP] F-TEID: add missing TEIDJoakim Karlsson1-1/+12
Change-Id: I4668208b5bf1d2cc168c7aa8f593510d55f27c23 Reviewed-on: https://code.wireshark.org/review/24359 Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com> Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
2017-11-11[PFCP] IPv4, IPv6 address presentJoakim Karlsson1-34/+34
Change-Id: I4efedb43a8a3688931c4815ea5fa82bd01248b98 Reviewed-on: https://code.wireshark.org/review/24358 Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com> Reviewed-by: Anders Broman <a.broman58@gmail.com>
2017-11-11RTLS: Add link to doc about this protocolAlexis La Goutte1-2/+1
Change-Id: Ib0207bb10c948637359f4afb853003065e3ca1bd Reviewed-on: https://code.wireshark.org/review/24344 Reviewed-by: Anders Broman <a.broman58@gmail.com>
2017-11-11[o|t]wamp: Fix Dead Store (Dead assignement/Dead increment) Warning found by ↵Alexis La Goutte1-2/+2
Clang Change-Id: Idfa33bb194a491e2b97cc7e4a95f4da8c30a5f58 Reviewed-on: https://code.wireshark.org/review/24349 Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2017-11-11mqtt: Fix Dead Store (Dead assignement/Dead increment) Warning found by ClangAlexis La Goutte1-1/+1
Change-Id: Ibd0041e0031c92073c89a1535677167e4fc90443 Reviewed-on: https://code.wireshark.org/review/24348 Reviewed-by: Anders Broman <a.broman58@gmail.com>
2017-11-11ieee80211: Fix Dead Store (Dead assignement/Dead increment) Warning found by ↵Alexis La Goutte1-1/+0
Clang Change-Id: I67b2d6f7d6e12f541780dfb17f5bbfe20bae3cf4 Reviewed-on: https://code.wireshark.org/review/24347 Reviewed-by: Anders Broman <a.broman58@gmail.com>
2017-11-11btmesh: Fix Dead Store (Dead assignement/Dead increment) Warning found by ClangAlexis La Goutte1-3/+1
Change-Id: I7218d74e1dc4d014dadd86f55b7805ba1581f9c3 Reviewed-on: https://code.wireshark.org/review/24346 Reviewed-by: Anders Broman <a.broman58@gmail.com>
2017-11-11DVB-CI: ensure that conversation_t stucture exists before using itPascal Quantin1-2/+2
Bug: 14202 Change-Id: Ie0cad9c1cd1d0ea1392a61194567e80bb2b5a566 Reviewed-on: https://code.wireshark.org/review/24345 Petri-Dish: Pascal Quantin <pascal.quantin@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
2017-11-11mqtt: Use guint for offsetStig Bjørlykke1-13/+11
Change-Id: Ie7b4a540e68121dbe5cd14a9aac80a1f9aae90b3 Reviewed-on: https://code.wireshark.org/review/24342 Petri-Dish: Stig Bjørlykke <stig@bjorlykke.org> Reviewed-by: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Roland Knall <rknall@gmail.com>
2017-11-11dissector/mqtt: Dissect the UNSUBACK payloadFlavio Santes1-0/+6
MQTT v5.0/UNSUBACK includes a payload conformed by a list of reason codes. There is no length field for this payload, so it must be computed as the difference between the full message size and the current offset. Change-Id: Ibf7ef4131408e26e2389c3ab89107c22c16412a1 Signed-off-by: Flavio Santes <flavio.santes@1byt3.com> Reviewed-on: https://code.wireshark.org/review/24337 Reviewed-by: Michael Mann <mmann78@netscape.net> Petri-Dish: Michael Mann <mmann78@netscape.net> Petri-Dish: Stig Bjørlykke <stig@bjorlykke.org> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2017-11-11dissector/mqtt: Payload length computationFlavio Santes1-21/+8
This patch modifies how the payload length is computed. With this patch we want to simplify and reduce the number of operations. Change-Id: Ie1c9a3f26c689e92fbbd57c34e4f68abd3ea7d02 Signed-off-by: Flavio Santes <flavio.santes@1byt3.com> Reviewed-on: https://code.wireshark.org/review/24336 Petri-Dish: Stig Bjørlykke <stig@bjorlykke.org> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2017-11-10dissector/amqp: fix TLS/SSL payload dissectionHelmut Buchsbaum1-2/+5
To enable TLS/SSL payload dissection for AMQPS traffic add the AMQP handle as SSL dissector for the actual AMQPS port. Thus we have successfully decrypted our AMQPS traffic using a pre-master-key file. Change-Id: I63dec2217fb5645007da010f651846834abacdbc Reviewed-on: https://code.wireshark.org/review/24335 Reviewed-by: Anders Broman <a.broman58@gmail.com> Petri-Dish: Anders Broman <a.broman58@gmail.com> Reviewed-by: Michael Mann <mmann78@netscape.net>
2017-11-10[PFCP] PDN Type Dissection and minor fixesMatej Tkac1-17/+69
Added PDN Type dissection (3GPP 29.244 Section 8.2.79) Fixed bit order in dissect_ue_ip_address Added "Encoding error" message if MBR or GBR has different format than specified in doc. Change-Id: I3b8b096fa490a2b4a16b3e6e9a640f92e389be1f Reviewed-on: https://code.wireshark.org/review/24334 Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2017-11-10packet-twamp.c: General improvementsMichael Mann1-125/+122
1. Always wait for Greeting packet, otherwise packet must not be TWAMP control 2. Dissect packets on first pass 3. Use value_string for control state Ping-Bug: 14171 Bug: 14196 Change-Id: I642c6fd61d29744ac4fd4185a9081eba27138c0a Reviewed-on: https://code.wireshark.org/review/24329 Petri-Dish: Michael Mann <mmann78@netscape.net> Tested-by: Petri Dish Buildbot Reviewed-by: Michael Mann <mmann78@netscape.net>
2017-11-10ieee802.11: Fix the display of the Cisco Aironet DTPC value.Richard Sharpe1-2/+11
The DTPC value is in dBm, so make it so. There is a second byte that we are looking for a spec on. Bug: 14193 Change-Id: I69f0af295954958fde38b74e965f6fb66063513b Reviewed-on: https://code.wireshark.org/review/24322 Petri-Dish: Richard Sharpe <realrichardsharpe@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Michael Mann <mmann78@netscape.net>
2017-11-09RIP: do not register proto_reg_handoff_rip() callbackPascal Quantin1-4/+1
Bug: 14197 Change-Id: Ib6a46e09622d85fabbf8465a9234af3a959b9663 Reviewed-on: https://code.wireshark.org/review/24327 Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com> Petri-Dish: Pascal Quantin <pascal.quantin@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Michael Mann <mmann78@netscape.net>
2017-11-09dissector/mqtt: Replace proto_tree_add_uint64 by proto_tree_add_uintFlavio Santes1-1/+1
Change-Id: I51d5dfdcbf13cca79ca354d0a63bf7dc328e5dc9 Signed-off-by: Flavio Santes <flavio.santes@1byt3.com> Reviewed-on: https://code.wireshark.org/review/24312 Petri-Dish: Michael Mann <mmann78@netscape.net> Tested-by: Petri Dish Buildbot Reviewed-by: Michael Mann <mmann78@netscape.net>
2017-11-09[PFCP] Dissect Node ID and minor fixesMatej Tkac1-40/+42
Fixed FQDN Field in Dissect Node ID according to 3GPP 29.244 Section 8.2.38 Added "Encoding error" message if the value in field is in dotted form contrary to the doc. Minor fix: Decreased F-TEID Flags spare bits (fix of dc269fcd6f1ebc41d4e164f14dbb2a078bbb6265) Change-Id: I7e9ba53a7deed7af1305f5c445bf4b2a9212e115 Reviewed-on: https://code.wireshark.org/review/24321 Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2017-11-09gvcp:bat2-236/+731
- Added support for new registers: - GVCP_IEEE_1588_EXTENDED_CAPABILITY (0x00000974) - GVCP_IEEE_1588_SUPPORTED_PROFILES (0x00000978) - GVCP_IEEE_1588_SELECTED_PROFILE (0x0000097C) - Added support for missing/new error codes (0x8013 - 0x8017) - Added dissector code for 'current link configuration' part of device mode register - Added support for proper event dissection with the length field in event commands is used as recommended by the GEV 2.1 specification - Added dissector code for new capability/configuration bits (iee1588 profile, multi-part, large leader/trailer, ieee extended capabilities) - BUGFIX: In multi-zone mode, zone directions where decoded as 'Receiver' and 'Transmitter' instead of 'Bottom Up' and 'Top-Down' - BUGFIX: Commands that received at least one 'PENDING_ACK' before the final acknowledge where always reported with '(Failed)' in the Wireshark main window - Fixed some typos gvsp: - Added full multi-part dissection support - Replaced some hard-coded numbers by numerical constants - Added support for missing/new error codes (0x8013 - 0x8017) - BUGFIX: In multi-zone mode, zone directions where decoded as 'Receiver' and 'Transmitter' instead of 'Bottom Up' and 'Top-Down' - Combined 'AddressOffsetHigh/Low' for multi-zone/part into a single more readable 64-bit value Change-Id: I55862204ef71f69bc88c79fe2259f7cb8365699a Reviewed-on: https://code.wireshark.org/review/24299 Petri-Dish: Anders Broman <a.broman58@gmail.com> Reviewed-by: Michael Mann <mmann78@netscape.net>
2017-11-09802.11: Correctly handle Wi-Fi Alliance Multi-AP subtype decoding.Richard Sharpe3-8/+26
Introduce a dissector table and fix a bug so that it now works by calling through a dissector table. Change-Id: Ifa3f01b3f306101b3144604a51806eaccc436373 Reviewed-on: https://code.wireshark.org/review/24319 Petri-Dish: Richard Sharpe <realrichardsharpe@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2017-11-09[PFCP] Improvements to Dissect F-TEID functionMatej Tkac1-15/+42
Added Bit 4 - CHID as described in 3GPP 29.244 Section 8.2.3. Added "Encoding error" msg if documentation conditions aren't met. Change-Id: Ia5e6d8bb2bf635fa695a2bfcba3877c2ecf52d87 Reviewed-on: https://code.wireshark.org/review/24318 Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2017-11-09TWAMP dissector for RFC 5357Anders3-0/+809
From https://github.com/demirten/twamp-dissector cleaned up a bit. Change-Id: I3736e2ef09bd84ba30a63fee8bf07771f07bfe6c Reviewed-on: https://code.wireshark.org/review/24283 Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2017-11-09dissector/mqtt: Rewrite the SUBSCRIBE payload size computationFlavio Santes1-2/+9
- FIX: subtract the property length from the message length - Replace the 'for' loop by a 'while' loop, now that the arithmetic is done before. TODO: It's a protocol error (v5.0)/violation (v3.1.1) not to include the payload in the SUBSCRIBE control packet. It would be nice to display a "malformed packet legend" in such that case. Change-Id: I99ef3862aa19b3a31ea03d1c194e54f489674115 Signed-off-by: Flavio Santes <flavio.santes@1byt3.com> Reviewed-on: https://code.wireshark.org/review/24313 Petri-Dish: Stig Bjørlykke <stig@bjorlykke.org> Tested-by: Petri Dish Buildbot Reviewed-by: Stig Bjørlykke <stig@bjorlykke.org>
2017-11-09ISDN: Convert circuit API to conversation APIMichael Mann1-12/+12
Change-Id: I8752123a740c47f9328e0144624d4fbf4e200165 Reviewed-on: https://code.wireshark.org/review/24314 Petri-Dish: Michael Mann <mmann78@netscape.net> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2017-11-09ISO 14443: Convert circuit API to conversation APIMichael Mann1-18/+15
Change-Id: Id007a3df5442cf4d21263acb17303b6d735aaa7d Reviewed-on: https://code.wireshark.org/review/24311 Petri-Dish: Michael Mann <mmann78@netscape.net> Reviewed-by: Anders Broman <a.broman58@gmail.com>
2017-11-09DVB-CI: Convert circuit API to conversation APIMichael Mann1-65/+60
Add the few necessary conversation APIs to make conversion possible. Change-Id: I775f23005c48cacd2be342bdc704af4738f0789c Reviewed-on: https://code.wireshark.org/review/24310 Petri-Dish: Michael Mann <mmann78@netscape.net> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2017-11-08mqtt: Rename mqtt_conv to mqtt_conv_tStig Bjørlykke1-6/+6
This will clarify that it's a type. Also remove unneeded init values. Change-Id: If61b32a264ff60a26e775dc99f1c06d0aa555a4d Reviewed-on: https://code.wireshark.org/review/24307 Petri-Dish: Stig Bjørlykke <stig@bjorlykke.org> Tested-by: Petri Dish Buildbot Reviewed-by: Michael Mann <mmann78@netscape.net>
2017-11-08btmesh: Remove a debug g_warningStig Bjørlykke1-1/+0
Change-Id: Ifc822efccfff5de32a54ca1f148a09ac3be4d0e3 Reviewed-on: https://code.wireshark.org/review/24308 Reviewed-by: Stig Bjørlykke <stig@bjorlykke.org>
2017-11-08dissector/mqtt: Add 'Subscription Options/Retain Handling' text descriptionsFlavio Santes1-2/+15
Change-Id: I6dea2e9c190095105dc1441197ad00c9b3a66f89 Signed-off-by: Flavio Santes <flavio.santes@1byt3.com> Reviewed-on: https://code.wireshark.org/review/24272 Petri-Dish: Stig Bjørlykke <stig@bjorlykke.org> Tested-by: Petri Dish Buildbot Reviewed-by: Stig Bjørlykke <stig@bjorlykke.org>
2017-11-08dissector/mqtt: Dissect the User Property fieldFlavio Santes1-0/+26
Change-Id: I0a0e484a814a11b785d20fda6eba5c251db98188 Signed-off-by: Flavio Santes <flavio.santes@1byt3.com> Reviewed-on: https://code.wireshark.org/review/24269 Petri-Dish: Stig Bjørlykke <stig@bjorlykke.org> Tested-by: Petri Dish Buildbot Reviewed-by: Stig Bjørlykke <stig@bjorlykke.org>
2017-11-08dissector/mqtt: Dissect MQTT v5.0 String/Binary PropertiesFlavio Santes1-0/+50
Add routines to dissect the following MQTT v5.0 String/Binary Data Property fields: - Content Type - Response Topic - Correlation Data - Assigned Client Identifier - Authentication Method - Authentication Data - Response Information - Server Reference - Reason String Change-Id: Ia0ad0597d32bacc58228376bb738b7092f4a1ad6 Signed-off-by: Flavio Santes <flavio.santes@1byt3.com> Reviewed-on: https://code.wireshark.org/review/24268 Petri-Dish: Stig Bjørlykke <stig@bjorlykke.org> Tested-by: Petri Dish Buildbot Reviewed-by: Stig Bjørlykke <stig@bjorlykke.org>
2017-11-08dissector/mqtt: Add the 'Subscription Options' dissectorFlavio Santes1-8/+56
Change-Id: I400430ca59c7492ae64dba6c4cca2485553988b9 Signed-off-by: Flavio Santes <flavio.santes@1byt3.com> Reviewed-on: https://code.wireshark.org/review/24271 Petri-Dish: Stig Bjørlykke <stig@bjorlykke.org> Tested-by: Petri Dish Buildbot Reviewed-by: Stig Bjørlykke <stig@bjorlykke.org>
2017-11-08smb2: Fix bug 6027 to correct SRT computations.Richard Sharpe1-0/+5
Fix a problem where SMB2 cancel and break messages are included in SRT computations when it should not be. Bug: 6027 Change-Id: I690fe6d9e14f26d85fe5c71830869780046d5d12 Reviewed-on: https://code.wireshark.org/review/24295 Reviewed-by: Richard Sharpe <realrichardsharpe@gmail.com>
2017-11-08dissector/mqtt: Dissect the Subscription Identifier PropertyFlavio Santes1-0/+9
Change-Id: Ia60b9c344fd572193374f57299b1829f17dfa24e Signed-off-by: Flavio Santes <flavio.santes@1byt3.com> Reviewed-on: https://code.wireshark.org/review/24267 Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Stig Bjørlykke <stig@bjorlykke.org>
2017-11-08[Diameter 3GPP] Sub dissect AVP Code: 1677 Group-PLMN-Id.Anders1-0/+23
Change-Id: I69ac8210dfcb43f9fc15e49c75dfb917976f0770 Reviewed-on: https://code.wireshark.org/review/24294 Reviewed-by: Anders Broman <a.broman58@gmail.com> Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Michael Mann <mmann78@netscape.net>
2017-11-08[PFCP] Fixed apply action flags bits referenced in register info.Matej Tkac1-2/+2
In accordance with 3GPP 29.244 8.2.26 Apply Action. Change-Id: Icef9bc9807efc255ece8fbde5e4c244d5f1fc525 Reviewed-on: https://code.wireshark.org/review/24301 Reviewed-by: Michael Mann <mmann78@netscape.net>
2017-11-08btmesh: Fix building without gcryptStig Bjørlykke1-0/+6
Change-Id: If77b36d764c07acd87a553134cebff246b8368fe Reviewed-on: https://code.wireshark.org/review/24296 Petri-Dish: Stig Bjørlykke <stig@bjorlykke.org> Reviewed-by: Richard Sharpe <realrichardsharpe@gmail.com>
2017-11-08MQTT: add a missing breakPascal Quantin1-0/+1
Change-Id: I0a17fd44607a5a7452a7c2789140cce7cdcbd17b Reviewed-on: https://code.wireshark.org/review/24298 Reviewed-by: Stig Bjørlykke <stig@bjorlykke.org>
2017-11-08dissector/mqtt: DISCONNECT and AUTH may not include the Property fieldFlavio Santes1-1/+9
From the spec: 3.14.2.2.1 Property Length: The length of Properties in the DISCONNECT packet Variable Header encoded as a Variable Byte Integer. If the Remaining Length is less than 2, a value of 0 is used In this commit we also assume that AUTH may not include the Property field, given that DISCONNECT and AUTH share the same structure. Change-Id: I5f55151df6b2066d924b2c16fb08a63b0903ef46 Signed-off-by: Flavio Santes <flavio.santes@1byt3.com> Reviewed-on: https://code.wireshark.org/review/24288 Petri-Dish: Stig Bjørlykke <stig@bjorlykke.org> Reviewed-by: Stig Bjørlykke <stig@bjorlykke.org>
generated by cgit v1.2.3 (git 2.25.1) at 2024-06-17 09:43:49 +0000