aboutsummaryrefslogtreecommitdiffstats
path: root/epan
AgeCommit message (Collapse)AuthorFilesLines
2017-08-18RSVP: fix name of PSMP ID field (SESSION object C-Type 13/14)Alexis La Goutte1-3/+5
Also display PSMP ID in decimal Ping-Bug: 13977 Change-Id: If6b87ab87339038d763dcc1c97353aaf9d69a02c Reviewed-on: https://code.wireshark.org/review/23103 Petri-Dish: Michael Mann <mmann78@netscape.net> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Michael Mann <mmann78@netscape.net>
2017-08-18msdp: Fix Dead Store (Dead assignement/Dead increment) Warning found by ClangAlexis La Goutte1-3/+3
Change-Id: Ie98f0c70190206b4682bf1b1b13add51c52303db Reviewed-on: https://code.wireshark.org/review/23009 Reviewed-by: Anders Broman <a.broman58@gmail.com> Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Michael Mann <mmann78@netscape.net>
2017-08-18ZigBee fix Default Response dissectionChris Brandson1-16/+12
The ZCL Default Response command can be sent in response to any profile-wide or cluster specific command. The Default Response command is itself a profile-wide command so the Default Response frame control is of no help in deciding whether the original request is profile-wide or cluster-specific. The simplest solution is to not attempt interpretation. A more sophisticated solution would be to cache all ZCL frame counters and match Default Responses based on sequence number, but this is problematic because sequence numbers repeat. At least for now we can always display the information correctly. Change-Id: I827e2d2f9d6e5f7c9dfa572d2ee2ac3c9f170d70 Reviewed-on: https://code.wireshark.org/review/22688 Reviewed-by: Anders Broman <a.broman58@gmail.com>
2017-08-18[GSM MAP] Dissect Ext2-QoS-Subscribed, Ext3-QoS-Subscribed andAndersBroman3-33/+656
Ext4-QoS-Subscribed Change-Id: Ic6a0f80d5f2339fd90284d1fddffe57b66c6771d Reviewed-on: https://code.wireshark.org/review/23108 Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Anders Broman <a.broman58@gmail.com>
2017-08-17GTPv2: Fix length of EPS Bearer ID (EBI)Uli Heilmeier1-1/+1
Field is only 1 not 2 bytes long. Bug: 13987 Change-Id: If22e19e917bd41907248a497b30883e89fc9ff4b Reviewed-on: https://code.wireshark.org/review/23109 Reviewed-by: Michael Mann <mmann78@netscape.net>
2017-08-17TLS: Add new cipher suite from draft-ietf-tls-ecdhe-psk-aeadAlexis La Goutte1-0/+5
Change-Id: I256496618e67df70ce02ee90f21bc0cd556b0b5a Reviewed-on: https://code.wireshark.org/review/23105 Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Anders Broman <a.broman58@gmail.com>
2017-08-17802.11: Add dissection of BSS Transition candidate List entriesAlexis La Goutte1-2/+21
It is a list of Neighbor report (for request and response) Missing dissection of Neighbor Sublelement 0x03 (BSS Transition Candidate Preference) Issue reported by Mark Williams Bug: 13985 Change-Id: Ic3871866ba4779ee69e91d6d57b46926466b340c Reviewed-on: https://code.wireshark.org/review/23107 Petri-Dish: Michael Mann <mmann78@netscape.net> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Michael Mann <mmann78@netscape.net>
2017-08-16[GTPv2] Add dissection of Message PriorityAndersBroman1-12/+59
Change-Id: I86e3097a322a4a354235ce941e52e6d4c6baf8ca Reviewed-on: https://code.wireshark.org/review/23097 Reviewed-by: Anders Broman <a.broman58@gmail.com> Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Michael Mann <mmann78@netscape.net>
2017-08-16RADIUS: Remove some shortcuts in how we handle Ascend-Data-FilterJoão Valverde1-21/+24
Do it via type "abinary" like the comment suggests. Note: VSA "abinary" code path untested. Ping-Bug: 11630 Change-Id: Ie8ebbb2fdbc9f04faad40150652277f1396ea030 Reviewed-on: https://code.wireshark.org/review/22973 Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Anders Broman <a.broman58@gmail.com>
2017-08-16RADIUS: Add preference to disable RFC 6929 extended attributesJoão Valverde1-0/+9
This is an ugly workaround for the fact that attributes 241-246 are currently hardcoded as extended type. This is to restore previous functionality to dissect some Ascend AVPs that shouldn't be using the IANA allocation space. Ping-Bug: 11630 Change-Id: I6bebefd21fe5149f5f57b3280c9992a0eca85e62 Reviewed-on: https://code.wireshark.org/review/22972 Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Anders Broman <a.broman58@gmail.com>
2017-08-16 RNSAP: Call RANAP dissector for RANAP-RelocationInformationDarien Spencer3-7/+37
This IE is a transparent container of a RANAP message according to TS 25.413 - Section 11.2 Change-Id: I2080c9a75ce2acc947eade9c60b12c49080f8ee8 Reviewed-on: https://code.wireshark.org/review/23085 Reviewed-by: Michael Mann <mmann78@netscape.net> Petri-Dish: Michael Mann <mmann78@netscape.net> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Anders Broman <a.broman58@gmail.com>
2017-08-16UDP: Bugfix handling UDP checksum of 0.Michael Mann1-1/+1
Copy/pasteo made it "unverified", when it should have been labeled "Not present" Bug: 13955 Change-Id: If3b8dea276ae38ba8169762ed7ed8e764022b5d7 Reviewed-on: https://code.wireshark.org/review/23091 Petri-Dish: Michael Mann <mmann78@netscape.net> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Michael Mann <mmann78@netscape.net>
2017-08-15NBAP over SCTP heuristic dissectorSam Cisneros2-3/+151
Bug: 13866 Change-Id: I895266a6aa7458aa3ab18742bcd981986c2fd17c Reviewed-on: https://code.wireshark.org/review/23074 Reviewed-by: Michael Mann <mmann78@netscape.net> Petri-Dish: Michael Mann <mmann78@netscape.net> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Anders Broman <a.broman58@gmail.com>
2017-08-15TCAP: fix infinite loop introduced in g35f9349396Pascal Quantin2-2/+2
Bug: 13926 Change-Id: I6bb06639f1226d6f5faee614858f5f09953d3ff5 Reviewed-on: https://code.wireshark.org/review/23087 Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
2017-08-15Update RFC and I-D references.Guy Harris2-7/+15
Give draft-frascone-xml-dictionary for the dictionary syntax. Give RFC numbers and I-D names, rather than URLs to the plain text versions of them; that way, you can more easily go to the HTML versions, which give more information. The "more information" for I-Ds may include later drafts and the final RFCs, and for RFCs may give newer RFCs obsoleting the older ones; add the RFC numbers for the I-Ds and the RFC number for the current Diameter RFC. Change-Id: If01ea341af3ae892755a243bd1dd66acfdfd5062 Reviewed-on: https://code.wireshark.org/review/23086 Reviewed-by: Guy Harris <guy@alum.mit.edu>
2017-08-15TCAP: more fixes for SRT analysisPascal Quantin4-30/+46
- when building the END hash entry, always use the BEGIN destination address. If no match is done, we will search using the BEGIN entry anyway - add the source PC to the END hash entry. As it will be used for messages initiated by the host that sent the BEGIN, we probably can assume that the PC will not change Bug: 13926 Change-Id: Iaa8b09bf6821961d1eab22683600f36d77bd2cc5 Reviewed-on: https://code.wireshark.org/review/23048 Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com> Petri-Dish: Pascal Quantin <pascal.quantin@gmail.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Michael Mann <mmann78@netscape.net>
2017-08-14Couchbase: Add doc flag support to dissectorolivermd1-11/+64
This patch makes required changes to be able to dissect packets that use the new subdoc doc flags. This involved modifying the existing subdoc flags, creating the new docs flags and modifying the extras dissector. Change-Id: Iaaf5e58d778ca5f54f76c9d27680df68eee8b7bd Reviewed-on: https://code.wireshark.org/review/23032 Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
2017-08-14ieee802.11: Replace an incorrect fall through with a break.Richard Sharpe1-1/+1
Table 8-216—GAS Initial Request frame body format (page 752 of the 2012 version) suggests that the only thing that comes after the length is the actual request and since the default just inserts the bytes for req_len, it looks like the fall-through should not be there and it should be a break. Change-Id: I8e5afb24fedffea869829dc9f5bf3d42b20121eb Reviewed-on: https://code.wireshark.org/review/23075 Reviewed-by: Michael Mann <mmann78@netscape.net> Petri-Dish: Michael Mann <mmann78@netscape.net> Reviewed-by: Richard Sharpe <realrichardsharpe@gmail.com>
2017-08-14ieee80211: fix dpp doesn't match PROTOABBREV of ieee80211Alexis La Goutte1-1/+1
and also with conflict check: 'dpp' exists multiple times with NOT compatible types: FT_PROTOCOL and FT_UINT8 Change-Id: I98e2f7e8abbee5423e2bfa7b71b5259edd4711e4 Reviewed-on: https://code.wireshark.org/review/23071 Reviewed-by: Michael Mann <mmann78@netscape.net>
2017-08-14wifi-dpp: fix dppi.[init|resp].hash doesn't match PROTOABBREV of wifi-dppAlexis La Goutte1-2/+2
Change-Id: I7132f40aaed8ee217af66ec89a3ccce61a359554 Reviewed-on: https://code.wireshark.org/review/23070 Reviewed-by: Michael Mann <mmann78@netscape.net>
2017-08-14Makefile.am (dissectors): fix align indentAlexis La Goutte1-1/+1
Change-Id: Ifde626095a61bb3aca3f9cc101023aa4bbf3341b Reviewed-on: https://code.wireshark.org/review/23072 Reviewed-by: Michael Mann <mmann78@netscape.net>
2017-08-14ieee80211: fix typo specfic => specificAlexis La Goutte1-1/+1
Change-Id: Ic071dcccf5aab5d3a45dd6354f3cd6eedbc1a84f Reviewed-on: https://code.wireshark.org/review/23073 Reviewed-by: Michael Mann <mmann78@netscape.net>
2017-08-14wifi: fix "multiple definition of wifi_display_ie_handle" linker errorPeter Wu2-2/+2
Change-Id: I4aee15cbf241d31c274dc511be2f86b8bfeae5b8 Fixes: v2.5.0rc0-683-g3c0ff67a9c ("ieee80211: Make a few dissector tables for vendor specific functionality") Reviewed-on: https://code.wireshark.org/review/23069 Petri-Dish: Peter Wu <peter@lekensteyn.nl> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
2017-08-14ieee80211 : Implement Wi-Fi DPP specRichard Sharpe5-31/+545
Including attributes in Public Action frames and those that can appear in GAS frames. Change-Id: I8d2a717984295592952b8fff82879197ace2a4b2 Reviewed-on: https://code.wireshark.org/review/22615 Petri-Dish: Michael Mann <mmann78@netscape.net> Reviewed-by: Richard Sharpe <realrichardsharpe@gmail.com>
2017-08-14ieee80211: Make a few dissector tables for vendor specific functionalityMichael Mann4-94/+170
This allows vendors to more easily expand functionality. Change-Id: Iaa6a0626f3ab3c80a3ea90ca09c90284705d0221 Reviewed-on: https://code.wireshark.org/review/23066 Reviewed-by: Richard Sharpe <realrichardsharpe@gmail.com>
2017-08-14packet-ieee80211.c: Add dissector table for vendor specific actionsMichael Mann3-23/+50
Gives a little more flexibility for vendor implementations https://www.wireshark.org/lists/wireshark-dev/201708/msg00053.html https://ask.wireshark.org/questions/56816/ieee-80211-vendor-specific-action Change-Id: I0345e43140a8ac99b26932b72d08a99ca0036389 Reviewed-on: https://code.wireshark.org/review/23065 Petri-Dish: Michael Mann <mmann78@netscape.net> Reviewed-by: Richard Sharpe <realrichardsharpe@gmail.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
2017-08-14iso14443: fix the request-response trackingMartin Kaiser1-13/+31
Distinguish between Waiting Time Extension (WTX) messages and other messages. Use a composite key for the wmem tree that consists of a boolean wtx flag and the request packet number. Change-Id: I7783aabb231fe145a8d8c0d65c88b553d5a5b533 Reviewed-on: https://code.wireshark.org/review/23063 Reviewed-by: Martin Kaiser <wireshark@kaiser.cx> Petri-Dish: Martin Kaiser <wireshark@kaiser.cx> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Michael Mann <mmann78@netscape.net>
2017-08-13RADIUS: Initialize vendor attribute type to zeroJoão Valverde1-0/+1
Change-Id: Ic43363d93f6d573ddb8d3de3e0bd62624124d104 Reviewed-on: https://code.wireshark.org/review/23064 Reviewed-by: João Valverde <j@v6e.pt>
2017-08-13ZigBee ZCL IAS: decode IAS Zone Enroll RequestJohn Keeping1-0/+13
Add missing decoding for the Enroll Request command in the IAS Zone cluster. Change-Id: I8dc0875912c87601f610173014879e13cad0820c Reviewed-on: https://code.wireshark.org/review/23050 Petri-Dish: Michael Mann <mmann78@netscape.net> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Michael Mann <mmann78@netscape.net>
2017-08-12Sync some CMake and Autotools install behaviors.Gerald Combs1-19/+0
Adjust the following CMake and Autotools behaviors in order to synchronize their respective install behaviors: - Disable tfshark by default in CMakeOptions.txt - Add profiles/Bluetooth/preferences to Makefile.am - Add missing captype and ciscodump entries to doc/Makefile.am - Install help/faq.txt on all platforms in CMakeLists.txt - Add BUILD_corbaidl2wrs, BUILD_dcerpcidl2wrs, and BUILD_xxx2deb options to CMake and use them to adjust the corresponding parts of the build. - Pull the DCERPC idl2wrs build steps into the top-level CMakeLists.txt. This change doesn't sync everything. Some installed content still diverges, including the following: - CMake installs a bunch of modules into lib/wireshark: FindGLIB2.cmake FindWireshark.cmake FindWSWinLibs.cmake LocatePythonModule.cmake UseAsn2Wrs.cmake UseMakeDissectorReg.cmake WiresharkConfig.cmake WiresharkConfigVersion.cmake Do we need any or all of these? If so, should the Autotools behavior be synced accordingly? - Autotools installs libtool .la files. It also installs wireshark-gtk.desktop unconditionally. Change-Id: I7846efe08f7139c31b6ceca6f08a1fa5168b3e22 Reviewed-on: https://code.wireshark.org/review/23041 Reviewed-by: Gerald Combs <gerald@wireshark.org> Petri-Dish: Gerald Combs <gerald@wireshark.org> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Michael Mann <mmann78@netscape.net>
2017-08-12uma: avoid global variableAlexis La Goutte1-3/+1
Change-Id: Icc179c00b5b2bee6dc69d843de784c7519360206 Reviewed-on: https://code.wireshark.org/review/22830 Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Michael Mann <mmann78@netscape.net>
2017-08-12TLS: Add QUIC Transport ParametersAlexis La Goutte3-3/+266
See https://tools.ietf.org/html/draft-ietf-quic-tls Bug: 13881 Change-Id: Id8ae5cdc2f7232e5db28846e528ae378fc0ac58d Reviewed-on: https://code.wireshark.org/review/22780 Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com> Reviewed-by: Michael Mann <mmann78@netscape.net>
2017-08-10Link forward to ftp-data from ftp commandsMartin Mathieson1-22/+102
Change-Id: I57013fefed2e588749a208bb3e3562ca4bbef0ac Reviewed-on: https://code.wireshark.org/review/22953 Petri-Dish: Martin Mathieson <martin.r.mathieson@googlemail.com> Petri-Dish: Graham Bloice <graham.bloice@trihedral.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Martin Mathieson <martin.r.mathieson@googlemail.com>
2017-08-10Add QUIC Transport IETF (draft-05) protocolAlexis La Goutte3-0/+1099
Bug: 13881 Change-Id: I9007800060dfc503a1039f652d53c9cb1cd42211 Reviewed-on: https://code.wireshark.org/review/22366 Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com> Reviewed-by: Peter Wu <peter@lekensteyn.nl>
2017-08-10ua3g: fixed VTA type display for 8018, 8028S and 8008Nicolas Bertin1-2/+3
Change-Id: I24158599831e432b62ff2ec0f36eeb1350c18a4b Reviewed-on: https://code.wireshark.org/review/23031 Reviewed-by: Michael Mann <mmann78@netscape.net> Petri-Dish: Michael Mann <mmann78@netscape.net> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
2017-08-10SML: fix checksum verificationPascal Quantin1-4/+4
The checksum is stored in little endian format. Regression introduced in gad6fc87d64 Change-Id: Ic5dd58f645fbe9d791b367d5777e07acb135d38c Reviewed-on: https://code.wireshark.org/review/23034 Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
2017-08-10Couchbase: Add get error map cmdolivermd1-0/+13
Change-Id: I9ca47fa0f77edd0ce94d2fe4cb098b5ff2786749 Reviewed-on: https://code.wireshark.org/review/23030 Reviewed-by: Michael Mann <mmann78@netscape.net> Petri-Dish: Michael Mann <mmann78@netscape.net> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
2017-08-10MTP2: fix checksum verificationPascal Quantin1-1/+1
The checksum is stored in little endian format. Regression introduced in gad6fc87d64 Change-Id: I0fd7c972eb365e7deddd07a70e5f95fcaf006618 Reviewed-on: https://code.wireshark.org/review/23033 Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
2017-08-09ofcs2: fix 'data' was marked unused but was used [-Wused-but-marked-unused]Alexis La Goutte1-1/+1
Change-Id: Ic04842280e829b30230c7196ab0ba1f49462f295 Reviewed-on: https://code.wireshark.org/review/23016 Reviewed-by: Michael Mann <mmann78@netscape.net>
2017-08-09obex: fix 'pinfo' was marked unused but was used [-Wused-but-marked-unused]Alexis La Goutte1-2/+2
Change-Id: Id791b880d24f3ec948016c6bf00c00ac8f9cd6fb Reviewed-on: https://code.wireshark.org/review/23017 Reviewed-by: Michael Mann <mmann78@netscape.net>
2017-08-09openflow (v1): fix 'pinfo/length' was marked unused but was used ↵Alexis La Goutte1-3/+3
[-Wused-but-marked-unused] Change-Id: Ie74d079cfe63dcdb5f59a576d64c93d4452edeb2 Reviewed-on: https://code.wireshark.org/review/23018 Reviewed-by: Michael Mann <mmann78@netscape.net>
2017-08-09pcp: fix 'pinfo' was marked unused but was used [-Wused-but-marked-unused]Alexis La Goutte1-2/+2
Change-Id: I2cd58b58ff53951044ac13caffdc901d8c1b65bd Reviewed-on: https://code.wireshark.org/review/23019 Reviewed-by: Michael Mann <mmann78@netscape.net>
2017-08-09bootp: fix 'pinfo' was marked unused but was used [-Wused-but-marked-unused]Alexis La Goutte1-6/+6
Change-Id: Ia711b3954aded55470d92d0787368a555481367d Reviewed-on: https://code.wireshark.org/review/23020 Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Michael Mann <mmann78@netscape.net>
2017-08-08We really only need one range_foreach routine.Guy Harris18-88/+70
Rename range_foreach_r() to range_foreach(), getting rid of the old range_foreach(). If your callback doesn't require an additional argument, just pass NULL when calling range_foreach(), and declare the argument as unused. Change-Id: I49a56f90610e39cf2ddc398c9e30ed11a6ca90db Reviewed-on: https://code.wireshark.org/review/23025 Reviewed-by: Guy Harris <guy@alum.mit.edu>
2017-08-08Check for errors running the Lua scripts.Guy Harris1-5/+5
Also, fix error messages reported for lua_load() to speak of precompilation, not execution (the script isn't *executed* until lua_pcall() is called). Change-Id: I7ac9ee6e5df7612f9af141e51958fbfad38a4083 Reviewed-on: https://code.wireshark.org/review/23023 Reviewed-by: Guy Harris <guy@alum.mit.edu>
2017-08-08mrcpv2: fix bug in use of ws_strtou32.Dario Lombardo1-3/+3
Bug: 13952 Change-Id: I374343368c629b7f063c922f0aa8de32e21a95e8 Reviewed-on: https://code.wireshark.org/review/23012 Petri-Dish: Dario Lombardo <lomato@gmail.com> Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
2017-08-08Support multiple labels with bgpJunpei Yoshino1-6/+0
Passing multiple labels information with BGP has come to be used together with segment routing. Change-Id: Ifd45eb0a875ed4a166e44441955e5e42ce84a7ca Reviewed-on: https://code.wireshark.org/review/22944 Petri-Dish: Michael Mann <mmann78@netscape.net> Reviewed-by: Michael Mann <mmann78@netscape.net>
2017-08-07SMTP: show text lines within a PDU fragment.Martin Mathieson1-0/+6
Change-Id: I49ef553c29742ba18d477a61183ce8b6f5e23233 Reviewed-on: https://code.wireshark.org/review/23006 Petri-Dish: Martin Mathieson <martin.r.mathieson@googlemail.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Michael Mann <mmann78@netscape.net>
2017-08-07dns: Fix Dead Store (Dead assignement/Dead increment) Warning found by ClangAlexis La Goutte1-2/+0
Change-Id: I1bb25125d9bd7b62b5784bd43d933dc5ff13987c Reviewed-on: https://code.wireshark.org/review/23005 Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Michael Mann <mmann78@netscape.net>
2017-08-07dns: remove extra semicolonAlexis La Goutte1-4/+4
Change-Id: I710d9c519ccf012d3e74e7c2087d0f9fc5cfca12 Reviewed-on: https://code.wireshark.org/review/23004 Reviewed-by: Michael Mann <mmann78@netscape.net>