aboutsummaryrefslogtreecommitdiffstats
AgeCommit message (Collapse)AuthorFilesLines
2018-01-30SDP/RTP: Support for "bundled" media.Robert Jongbloed5-32/+86
Modern SDP usage (e.g. SIP, WebRTC) can "bundle" multiple RTP media streams on a single port. Thus the RTP dissector has to be able to handle audio and video at the same time, so the gboolean flag in _rtp_info was changed to a bit mask. The SDP parsing was then changed to detect multiple "m=" lines using the same port, and combine their audio/video bit masks, and the rtp_dyn_payload used has all the audio and video payload descriptions. Change-Id: Ifa3c034260f892ed005fe28647d28f3b0b1b05cf Reviewed-on: https://code.wireshark.org/review/25431 Petri-Dish: Jaap Keuter <jaap.keuter@xs4all.nl> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2018-01-30wifi-dpp: A number of further changes from the recent testing.Richard Sharpe1-6/+33
1. Add the crypto suite that came in with a spec change (v171212). 2. Add two additional fields that need handling. 3. Make the attribute value a separate sub-tree. Change-Id: Ic01527bcd0361bf2522d2efbc91cd8191d7b2e27 Reviewed-on: https://code.wireshark.org/review/25514 Petri-Dish: Richard Sharpe <realrichardsharpe@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
2018-01-30Remove some unused images.Gerald Combs9-9/+2
The NSIS checkboxes were last used in 2003. I'm not sure if the KDE icons ({hi,lo}??-app-wireshark.png) were ever used. Change-Id: Ib548b0fcfb650b279e8d475f350cdba663f903c0 Reviewed-on: https://code.wireshark.org/review/25512 Reviewed-by: Gerald Combs <gerald@wireshark.org>
2018-01-30dct2000 wiretap: expect all lines to have exactly 4 subsecond digits.Martin Mathieson2-3/+3
Should fix clang warning created by https://code.wireshark.org/review/#/c/25492. Change-Id: Iafa31e24cd786a510f3a953d615df4cbc3930fa6 Reviewed-on: https://code.wireshark.org/review/25508 Petri-Dish: Martin Mathieson <martin.r.mathieson@googlemail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Jakub Zawadzki <darkjames-ws@darkjames.pl> Reviewed-by: Michael Mann <mmann78@netscape.net>
2018-01-29codecs: register SBC codec once.Jaap Keuter1-4/+0
Somehow the registration of the SBC codec was duplicated, which does not (yet) cause an error message, but also doesn't work. Remove the duplicate registration. Change-Id: Ic22ba66fd65b326fb2688300dad8f9b81081c7e5 Signed-off-by: Jaap Keuter <jaap.keuter@xs4all.nl> Reviewed-on: https://code.wireshark.org/review/25506 Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com> Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Michael Mann <mmann78@netscape.net>
2018-01-29Avoid #ifdef'ing out translation unitsJoão Valverde4-9/+13
Change-Id: Ibef0120184ae577f11059fcaf0eaa24a32820273 Reviewed-on: https://code.wireshark.org/review/25502 Petri-Dish: João Valverde <j@v6e.pt> Tested-by: Petri Dish Buildbot Reviewed-by: João Valverde <j@v6e.pt>
2018-01-29DNP3: Limit the amount of "empty" objects/fields can be present in a packet.Michael Mann1-0/+15
Chances are if there are a large number of "empty objects" (that don't increment packet counter) it's an intentionally malicious packet and we should break the loop. Bug: 14362 Change-Id: Id9a6f4270cc47188becdf4652f903d0ba4478dcb Reviewed-on: https://code.wireshark.org/review/25497 Petri-Dish: Michael Mann <mmann78@netscape.net> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2018-01-29dct2000: for speed, avoid ws_strtoi32() while reading timestampMartin Mathieson1-6/+11
Change-Id: I5d8797b68c53168d4c00be8c3c3a3325b370e38c Reviewed-on: https://code.wireshark.org/review/25492 Petri-Dish: Martin Mathieson <martin.r.mathieson@googlemail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Michael Mann <mmann78@netscape.net> Reviewed-by: Anders Broman <a.broman58@gmail.com>
2018-01-29Display JSON in compact formMichail Koreshkov1-2/+178
JSON will be display like in browswer developer tool (Firefox or Chrome). Change-Id: Ib504f4828d9fd8d25d9564b93717007ac021713c Reviewed-on: https://code.wireshark.org/review/25474 Petri-Dish: Stig Bjørlykke <stig@bjorlykke.org> Tested-by: Petri Dish Buildbot Reviewed-by: Stig Bjørlykke <stig@bjorlykke.org>
2018-01-29ieee80211: Implement two more extended IE from IEEE STD 802.11.Richard Sharpe1-0/+135
Handle the Estimated Service Parameters tagged element and the Future channel guidance one. The second may need more work in future. These are defined in IEEE STD 802.11-2016 but may have been defined earlier. Change-Id: I1c67a0ea6df9c1cc89bb3a34da921f3938e0a012 Reviewed-on: https://code.wireshark.org/review/25407 Reviewed-by: Michael Mann <mmann78@netscape.net> Petri-Dish: Michael Mann <mmann78@netscape.net> Tested-by: Petri Dish Buildbot Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
2018-01-29radiotap: Mark unknown fields as unknown instead of reserved.Richard Sharpe1-132/+135
Change-Id: Iecfb705b37f54119eaec75ab8df8c7ee3c76bfec Reviewed-on: https://code.wireshark.org/review/25503 Petri-Dish: Richard Sharpe <realrichardsharpe@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Michael Mann <mmann78@netscape.net>
2018-01-29cli: free memory on exit in tap-comparestat (found by clang).Dario Lombardo1-2/+5
Change-Id: Ie4dfc68583fa6fc742baeaf9d9fd1c7d2f783f38 Reviewed-on: https://code.wireshark.org/review/25435 Petri-Dish: Michael Mann <mmann78@netscape.net> Tested-by: Petri Dish Buildbot Reviewed-by: Michael Mann <mmann78@netscape.net>
2018-01-29Protect UAT color "datatype" from an empty stringMichael Mann1-1/+5
UAT color "datatype" has the format of #XXXXXX so the XXXXXX is strduped to pass to strtol(). The "pointer math" assumed the # was always present and would result in large memory allocation if string was empty. Bug: 14357 Change-Id: Idc43b17f0e07705880d0d77f106991d10e09f072 Reviewed-on: https://code.wireshark.org/review/25504 Petri-Dish: Michael Mann <mmann78@netscape.net> Tested-by: Petri Dish Buildbot Reviewed-by: Michael Mann <mmann78@netscape.net>
2018-01-28TDS: Dissect TDS4.2 response packetCraig Jackson1-270/+1030
Add dissection of the TDS response packet for TDS 4.2. In order to share code, this required parameterizing TDS 7.x token-handling routines for things such as endian-ness and one-byte vs two-byte character encodings. This required ascertaining accurately when TDS 7.x is in use as early in the conversation as possible. This in turn required knowing the program versions downloaded in the prelogin packet in the case where the login packet is encrypted. (Listening to the LoginAck token is a little too late.) Add more support routines to parameterize the endian nature of each connection. Although the particular tokens decoded here are documented for TDS 4.2, it has only been tested with a trace from TDS 4.6. TDS 4.6 didn't change much, but there may be a few minor errors. Change-Id: I6f8f136bcc565640fbea4302cb79ea29a118d9a1 Reviewed-on: https://code.wireshark.org/review/25464 Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Michael Mann <mmann78@netscape.net>
2018-01-28androiddump: Raise ADB timeout to 2s for slower devicesFlorian Bezold1-1/+1
Running "tcpdump -D" on target seems to be a expensive operation in some cases, not finishing within the 500ms timeout on several devices (~1000 ms in one of our cases). Change-Id: I57e4d31f12c4e393ff84e79b64cb024b74a11f0e Reviewed-on: https://code.wireshark.org/review/24719 Reviewed-by: Michal Labedzki <michal.labedzki@wireshark.org> Reviewed-by: Michael Mann <mmann78@netscape.net>
2018-01-28wsutil/filesystem.c: fix memory leak in init_progfile_dir()Jakub Zawadzki1-6/+0
If file wasn't found in PATH, there is a leak of path. Don't break the loop after reaching NUL character. Check for NUL character will be done in while() start condition. Found by clang. Change-Id: I0111a71853ffd485ee1096296f92de4472409c66 Reviewed-on: https://code.wireshark.org/review/25445 Petri-Dish: Jakub Zawadzki <darkjames-ws@darkjames.pl> Tested-by: Petri Dish Buildbot Reviewed-by: Michael Mann <mmann78@netscape.net>
2018-01-28Minor Makefile.am cleanup.João Valverde1-5/+0
Change-Id: I3670c86a695cbea00be39d0c274a5bef0ef31cbf Reviewed-on: https://code.wireshark.org/review/25501 Petri-Dish: João Valverde <j@v6e.pt> Tested-by: Petri Dish Buildbot Reviewed-by: João Valverde <j@v6e.pt>
2018-01-28plugins: Remove nordic_ble.dll exceptionJoão Valverde1-10/+0
This should no longer be necessary with the sub-version directory struture. If the user is installing a DLL for a built-in plugin, an error message is appropriate. Change-Id: I80842f1ea03a08594321a674650a1373aaa48712 Reviewed-on: https://code.wireshark.org/review/25500 Reviewed-by: João Valverde <j@v6e.pt>
2018-01-28Rename plugins_scan_dir()João Valverde1-5/+5
This makes it more obvious that it is a static function. Change-Id: I7348c4326b1e388870cb6657d9c67bd1086357b0 Reviewed-on: https://code.wireshark.org/review/25499 Reviewed-by: João Valverde <j@v6e.pt>
2018-01-28Use g_str_has_suffix()João Valverde1-3/+1
Change-Id: Ida335462efdfeef88a7afb3f27caf274632ca879 Reviewed-on: https://code.wireshark.org/review/25498 Reviewed-by: João Valverde <j@v6e.pt>
2018-01-28plugins: Use per-library data structuresJoão Valverde1-55/+41
This is cleaner and allows the "duplicate plugin name check" to be performed correctly (per module and not globally). Change-Id: I2b2122495dc047af5b7043dcf020a525766b7c30 Reviewed-on: https://code.wireshark.org/review/25496 Petri-Dish: João Valverde <j@v6e.pt> Tested-by: Petri Dish Buildbot Reviewed-by: João Valverde <j@v6e.pt>
2018-01-28[Automatic update for 2018-01-28]Gerald Combs10-69/+813
Update manuf, services enterprise numbers, translations, and other items. Change-Id: I0b47c83ca6e62a24043fb5d546141e3e9af1e7b8 Reviewed-on: https://code.wireshark.org/review/25493 Reviewed-by: Gerald Combs <gerald@wireshark.org>
2018-01-28Infiniband: Decode AETH syndrome byteUli Heilmeier1-1/+118
Used reference: Book "InfiniBand Network Architecture" by Tom Shanley; page 369 ff Bug: 14359 Change-Id: I77e64ca16ccc5f193eac34b304165f722ffb0748 Reviewed-on: https://code.wireshark.org/review/25489 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>
2018-01-27DCT2000: after profiling, speed up some partsMartin Mathieson1-12/+31
Change-Id: Ia335aaca3368608fe495c887d846fe3f057b74dd Reviewed-on: https://code.wireshark.org/review/25486 Petri-Dish: Martin Mathieson <martin.r.mathieson@googlemail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2018-01-27valgrind-wireshark.sh: fix typo in usageMartin Mathieson1-1/+1
Change-Id: If9210c6f71d4e258a489d254482cf5543b818e11 Reviewed-on: https://code.wireshark.org/review/25491 Reviewed-by: Anders Broman <a.broman58@gmail.com>
2018-01-27Reset file permissionsJaap Keuter5-0/+0
Some oddities with regard to file permissions have crept into the repository. Reset execute rights on various files which do not need them. Change-Id: Ib05658072925d59fc682173673c5638d157a269a Signed-off-by: Jaap Keuter <jaap.keuter@xs4all.nl> Reviewed-on: https://code.wireshark.org/review/25490 Reviewed-by: Guy Harris <guy@alum.mit.edu>
2018-01-27Qt: Translate some about box strings.Gerald Combs1-17/+17
Change-Id: I2f9d1be83a26129e1206f32133b5ba6705f4dd85 Reviewed-on: https://code.wireshark.org/review/25471 Reviewed-by: Gerald Combs <gerald@wireshark.org> Petri-Dish: Gerald Combs <gerald@wireshark.org> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2018-01-27Snort: speed up parsing of options by avoiding g_snprintf()Martin Mathieson1-4/+3
Change-Id: Ibe12ce69b139752e60b66a2af63e31244321bec5 Reviewed-on: https://code.wireshark.org/review/25484 Petri-Dish: Martin Mathieson <martin.r.mathieson@googlemail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2018-01-27http2: underscore replaced by dash in static headers nameAlexander Gryanko1-27/+27
Headers field "name" is used in the hash table for matching field type while processing data. Browsers use dash as delimiter while in the code we use underscore. Change-Id: I6342af9328118b41a8c71e034ef5913a83a84459 Reviewed-on: https://code.wireshark.org/review/25478 Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Michael Mann <mmann78@netscape.net>
2018-01-26Release notes: We ship Qt 5.9.4 on Windows.Gerald Combs1-0/+1
Change-Id: Id4190dee99ade7bf4568010109c1e4dfd0013873 Reviewed-on: https://code.wireshark.org/review/25485 Reviewed-by: Gerald Combs <gerald@wireshark.org>
2018-01-26Qt: Allow filter with deprecated syntax as buttonStig Bjørlykke1-2/+5
Also update widgets at startup. Bug: 14369 Change-Id: I2621b78289620a131bcad21d5ec23b048e4b218c Reviewed-on: https://code.wireshark.org/review/25475 Petri-Dish: Stig Bjørlykke <stig@bjorlykke.org> Tested-by: Petri Dish Buildbot Reviewed-by: Stig Bjørlykke <stig@bjorlykke.org>
2018-01-26Qt: Remove unused label_foldersStig Bjørlykke1-20/+1
Change-Id: I42d7ba0eed9cf2ad50e02a5385c6929e9b31882b Reviewed-on: https://code.wireshark.org/review/25473 Petri-Dish: Stig Bjørlykke <stig@bjorlykke.org> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2018-01-26Qt: Simplify files column in about dialogPeter Wu5-123/+3
HTMLTextDelegate fails to build in Qt 4.8, partially because QStyleOptionViewItem.text is introduced with Qt 5. Another issue with the current implementation is that the "Copy" action includes the HTML tags for italics. To fix these, just remove the italics formatting. Change-Id: I63aed7c3d0da3d9829d410d75fc18daa4049c0ca Fixes: v2.5.0rc0-1822-g73c01d6d05 ("Qt: Rewrite the About dialog to use models") Reviewed-on: https://code.wireshark.org/review/25470 Reviewed-by: Gerald Combs <gerald@wireshark.org> Petri-Dish: Gerald Combs <gerald@wireshark.org> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2018-01-26Added the Proconx CAN-ETH protocol dissectorLazar Sumar8-2/+312
Change-Id: I306341c7cddf8facb4a9ca62254a465a1da22174 Reviewed-on: https://code.wireshark.org/review/25423 Petri-Dish: Michael Mann <mmann78@netscape.net> Tested-by: Petri Dish Buildbot Reviewed-by: Michael Mann <mmann78@netscape.net>
2018-01-25Get rid of the necessity to use a preference to dissect ERSPAN type I packets.Joerg Mayer3-19/+11
Change-Id: I52c4c1fa5601bc66396f397ad23a9e53285a5926 Reviewed-on: https://code.wireshark.org/review/25463 Reviewed-by: Jörg Mayer <jmayer@loplof.de>
2018-01-25Qt 4.8 compatibility fixesPeter Wu15-22/+47
Add missing include headers, define Q_NULLPTR for all Qt objects (to avoid having to include an ugly compat header), use old signal/slot syntax (Q_SIGNALS are protected in Qt 4.8), disable Qt 5 log message handler, etc. Change-Id: I449beb0b74050fdfb0fc5828d8fb74eedb2b45bf Reviewed-on: https://code.wireshark.org/review/25469 Petri-Dish: Peter Wu <peter@lekensteyn.nl> Tested-by: Petri Dish Buildbot Reviewed-by: Gerald Combs <gerald@wireshark.org>
2018-01-25[PER] Fix the bitmask to calculate the number of bits needed to hold theAndersBroman1-1/+1
length. Bug:14366 Change-Id: I79a8051b94feffeb5d667a664fdcff6680dec620 Reviewed-on: https://code.wireshark.org/review/25468 Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2018-01-25ZigBee: Fix change 25456 - Wrong attribute ID correctedKenneth Soerensen1-1/+1
Change-Id: I06251e47feb6593bd940c3a719922abfa3241205 Reviewed-on: https://code.wireshark.org/review/25466 Petri-Dish: Graham Bloice <graham.bloice@trihedral.com> Tested-by: Petri Dish Buildbot Reviewed-by: Graham Bloice <graham.bloice@trihedral.com>
2018-01-25gitlab-ci: fix clang repos.Dario Lombardo1-2/+2
Change-Id: If0dd9074b0c2493c1f66f26271018f5dadc91e3d Reviewed-on: https://code.wireshark.org/review/25467 Reviewed-by: Dario Lombardo <lomato@gmail.com>
2018-01-25Qt: Adjust the about box layout.Gerald Combs1-7/+53
Make sure the compile and build information is centered. Change-Id: I2bfe502df4339c28d8c88553419b7d4d412a2be2 Reviewed-on: https://code.wireshark.org/review/25461 Reviewed-by: Gerald Combs <gerald@wireshark.org> Petri-Dish: Gerald Combs <gerald@wireshark.org> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2018-01-25Don't use static GMutexes.Gerald Combs1-7/+21
They're not supported in older versions of GLib. Change-Id: I8342048bab90fb192b8d7f0253f8b3207be8c5b1 Reviewed-on: https://code.wireshark.org/review/25460 Reviewed-by: Gerald Combs <gerald@wireshark.org> Petri-Dish: Gerald Combs <gerald@wireshark.org> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2018-01-25ZigBee: Add remaining attributes from Smart Energy Prepayment clusterKenneth Soerensen1-12/+86
Change-Id: I7e35dd26d7369cbc39cdb1e078200306d6ada619 Reviewed-on: https://code.wireshark.org/review/25456 Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2018-01-25Minor wireshark-qt.cpp cleanups.Gerald Combs1-21/+1
Remove duplicate and unneeded code. Remove placeholder markers left over from the initial port from GTK+. Change-Id: I4b372057c40cce53d0a00a54e490dec2097a7423 Reviewed-on: https://code.wireshark.org/review/25462 Reviewed-by: Gerald Combs <gerald@wireshark.org> Petri-Dish: Gerald Combs <gerald@wireshark.org> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2018-01-25bootp: DHCP User Class Data iPXE exceptionJaap Keuter1-0/+9
iPXE seems to violate RFC 3004 by ommitting the UC_Len_i field in the User Class Data field. Since this seems to be a 'well known' issue, which is not going to be corrected, detect and circumvent the error detection for this specific use. Bug: 14312 Change-Id: I2a15c336d7f67ee5fd83f955de7126eac146bfb1 Signed-off-by: Jaap Keuter <jaap.keuter@xs4all.nl> Reviewed-on: https://code.wireshark.org/review/25450 Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2018-01-24Qt+WiX: Skip translations.Gerald Combs1-6/+1
We handle translations in QtTranslation.wxs so we need to skip them when generating QtDependentComponents.wxs. Change-Id: I05586c7f8fbbaf72fc30564a157ba1175154d1d3 Reviewed-on: https://code.wireshark.org/review/25459 Reviewed-by: Gerald Combs <gerald@wireshark.org>
2018-01-24OpenFlow(v5&v6) : Fix RX_PWR/TX_BIASLaszlo Papp2-4/+4
Change-Id: Iae6fe2473d80cb14d5a9676b7efce41dc59a30e2 Reviewed-on: https://code.wireshark.org/review/25454 Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2018-01-24CMake: Create directories one at a time.Gerald Combs1-8/+7
Older versions of CMake don't support passing multiple arguments to `-E make_directory`. Change-Id: I62f76753a0ec1a96849de3a1131869f08303dbfc Reviewed-on: https://code.wireshark.org/review/25458 Reviewed-by: Gerald Combs <gerald@wireshark.org>
2018-01-24Qt+WiX: Check windeployqt --list output.Gerald Combs1-33/+27
Windeployqt 5.9.4 can print translation files that don't exist (QTBUG-65974). Fix up whitespace. Change-Id: If0a4ad6e24023707fa7e9cb1132af8a8dc425627 Reviewed-on: https://code.wireshark.org/review/25457 Reviewed-by: Gerald Combs <gerald@wireshark.org>
2018-01-24Openflow: Use offset 2 for tx_min/max as they are uint16_tLaszlo Papp2-8/+8
Change-Id: I2f47e0ede2a31cfdadc69ab125a739b3deaa297e Reviewed-on: https://code.wireshark.org/review/25453 Petri-Dish: Graham Bloice <graham.bloice@trihedral.com> Reviewed-by: Anders Broman <a.broman58@gmail.com> Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
2018-01-24Fix the display of Length and Pad2 in the openflow dissectorLaszlo Papp2-4/+4
Change-Id: I12194d1449812c43c062c50d319d44d6d7437c51 Reviewed-on: https://code.wireshark.org/review/25451 Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com> Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>