aboutsummaryrefslogtreecommitdiffstats
AgeCommit message (Collapse)AuthorFilesLines
2014-09-22Consistently use tab indentation.Guy Harris1-139/+141
(If somebody wants to convert the entire file to 4-space indentation, go ahead.) Change-Id: I1e3829289ac67db79eea2eb16e6a4ba40c449a8d Reviewed-on: https://code.wireshark.org/review/4250 Reviewed-by: Guy Harris <guy@alum.mit.edu>
2014-09-22Fix dist.Gerald Combs1-0/+6
Change-Id: I68963f000dc927398d7d41193674e55050e7c634 Reviewed-on: https://code.wireshark.org/review/4248 Reviewed-by: Gerald Combs <gerald@wireshark.org>
2014-09-22Qt: Use wsiconcap while we're capturing.Gerald Combs8-4/+47
Set the main window icon to the "wsiconcap" version while we're capturing similar to the GTK+ version. Verified on Windows. Not sure if this will do anything on other platforms. Change-Id: I9b082601a2c47e5f52cc38ac8d9b4d9f5fb9a4d9 Reviewed-on: https://code.wireshark.org/review/4230 Petri-Dish: Gerald Combs <gerald@wireshark.org> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Gerald Combs <gerald@wireshark.org>
2014-09-22fix-encoding-args.pl: Don't use a feature available only starting with perl 5.14Bill Meier1-6/+3
Specifically: Window msys git uses an earlier version of Perl (5.8) and thus invoking fix-encoding-args.pl in the pre-commit hook script gives a Perl syntax error. See https://code.wireshark.org/review/4172 for details (Also see following for Perl info: http://www.perl.com/pub/2011/05/new-features-of-perl-514-non-destructive-substitution.html) Change-Id: I95a222eaa31942cc6e873ffbde57c5fa5bd2a625 Reviewed-on: https://code.wireshark.org/review/4245 Reviewed-by: Bill Meier <wmeier@newsguy.com>
2014-09-22Bluetooth: Handle LE ACL sessionMichal Labedzki1-5/+56
Low Energy use LE Meta event for creating connection so add session when it occur. Change-Id: I6fad42651a3dc6a45cfb1cf679660de5955e76b0 Reviewed-on: https://code.wireshark.org/review/4168 Petri-Dish: Michal Labedzki <michal.labedzki@tieto.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Michal Labedzki <michal.labedzki@tieto.com>
2014-09-22extcap: Always create extcap/ dir in run/Michal Labedzki1-0/+1
Change-Id: I5d0a2591b9086a138cefb0019f710ffb34a34c52 Reviewed-on: https://code.wireshark.org/review/3876 Petri-Dish: Michal Labedzki <michal.labedzki@tieto.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Michal Labedzki <michal.labedzki@tieto.com>
2014-09-22Try to fix some buildbot warningsMichal Labedzki14-129/+112
Most interesting are: warning: cannot optimize loop, the loop counter may overflow [-Wunsafe-loop-optimizations] warning: ISO C forbids zero-size array [-Wpedantic] warning: ISO C90 doesn't support unnamed structs/unions [-Wpedantic] warning: cast discards '__attribute__((const))' qualifier from pointer target type [-Wcast-qual warning: initializer element is not computable at load time [enabled by default] Change-Id: I5573c6bdca856a304877d9bef643f8c0fa93cdaf Reviewed-on: https://code.wireshark.org/review/3174 Petri-Dish: Michal Labedzki <michal.labedzki@tieto.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Michal Labedzki <michal.labedzki@tieto.com>
2014-09-22Clean up reading code.Guy Harris1-81/+78
The only place where a short read should be treated as an EOF is if the read of the block header reads 0 bytes. All other short reads, including reads of the block header returning at least 1 byte but not enough for a complete block header, and any reads of the stuff *following* the block header even if they return 0 bytes, should be treated as "short read" errors. If the option length is bigger than the option buffer size, treat that as a bad file (I'm not sure that can happen, so maybe it should be treated as an internal error instead). Use file_skip() rather than file_seek() when skipping forward N bytes. If it fails, treat that as an error under all circumstances. When reading the first section header block in the open routine, have pcap_read_block() return -2 if it doesn't look like an SHB (too short, wrong block type, bad block length, unknown byte-order magic number), as that means the file isn't a pcap-ng file and the open should return 0. Return -1, not 0, for all errors in various block-reading routines. file_seek() returning 0 is *not* an error. file_seek() returning -1 (or any other negative number *is* an error; its return value is signed, so don't assign it to an unsigned variable. This might fix the test errors for the Lua file format handler tests. Change-Id: Ifa7d9834c38bf238461c9cc9625a2aa761cb6ff2 Reviewed-on: https://code.wireshark.org/review/4238 Reviewed-by: Guy Harris <guy@alum.mit.edu>
2014-09-22 packet-infiniband: added support for RoCE-v2 (RRocE)Wasim Abu Moch1-15/+66
Added support for the new Routable RDMA over Converged Ethernet. (NEW PATCH) Change-Id: I857a27c4f29cc78fcea8efa113e3c7e968eb041b Reviewed-on: https://code.wireshark.org/review/4156 Reviewed-by: Evan Huus <eapache@gmail.com>
2014-09-22dcerpc: correctly initialize certain struct fieldsEvan Huus1-5/+1
Bug: 10482 Change-Id: I4d39d6a79d4ab89685a17a0c622c6181e4fada3d Reviewed-on: https://code.wireshark.org/review/4233 Reviewed-by: Evan Huus <eapache@gmail.com>
2014-09-22Fix up formatting (bring function names over to column 0).Jeff Morriss1-104/+167
Change-Id: I054f3ec13fd5907c8f1e0546292777a5596fc029 Reviewed-on: https://code.wireshark.org/review/4232 Reviewed-by: Jeff Morriss <jeff.morriss.ws@gmail.com>
2014-09-21Fix dissection of 7 bits ASCII/GSM strings when the bit offset is not byte ↵Pascal Quantin1-8/+10
aligned Bug: 10491 Change-Id: Ib55d83b7739050ba5afd84e8184af3c4608d5776 Reviewed-on: https://code.wireshark.org/review/4228 Tested-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: Pascal Quantin <pascal.quantin@gmail.com>
2014-09-21Qt: Multi-state capture start icon.Gerald Combs12-49/+326
Add ".on" versions of the x-capture-start icon along with corresponding code in the StockIcon constructor to look for ".on" variants and add them with the QIcon::On state. Make the plain versions of x-capture-start blue to match the general application icon. The goal is to make the toolbar and app icon fins blue when Wireshark is "at rest" and green when it's capturing. Change-Id: I31f4f9d910fc99c41d7c63bd9a722db1611760c7 Reviewed-on: https://code.wireshark.org/review/4225 Reviewed-by: Gerald Combs <gerald@wireshark.org>
2014-09-21[Automatic manuf, services and enterprise-numbers update for 2014-09-21]Gerald Combs4-5/+282
Change-Id: I1057fe3657c400a6a0b10a67a7e2451d1266bf44 Reviewed-on: https://code.wireshark.org/review/4222 Reviewed-by: Gerald Combs <gerald@wireshark.org>
2014-09-21CMake: Add conditional builds.Gerald Combs2-10/+40
In my version of XCode ranlib complains bitterly and returns an error if a module has no symbols. Make the AirPcap, RTP player, and remote capture code conditional. We could also try passing "-no_warning_for_no_symbols" to ranlib but we probably shouldn't try to comple the code in the first place. Change-Id: Ide4f3d649eac354d54f0dde27dbc8e9cb270c65f Reviewed-on: https://code.wireshark.org/review/4220 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: Evan Huus <eapache@gmail.com>
2014-09-20Fix wslua_filehandler_open() to work as wiretap open routines should.Guy Harris1-2/+12
They should return 1 on success, -1 on error, and 0 if there was no error but the file isn't one of the type for the routine. They must not return any other value. If they return -1, they must set *err. Change-Id: I0f1e1675b4cc8f0214ad67a23de0e4ecb09cabea Reviewed-on: https://code.wireshark.org/review/4221 Reviewed-by: Guy Harris <guy@alum.mit.edu>
2014-09-20Restore the inclusion of request.cnf.c and response.cnf.c in the generated ↵Michael Mann5-1278/+782
DCE/RPC MAPI dissector. Also remove the use of proto_tree_add_text for something filterable. Change-Id: I7c252fb24f5ab51e4d1913fe3b0e520a4150baec Reviewed-on: https://code.wireshark.org/review/4214 Reviewed-by: Michael Mann <mmann78@netscape.net>
2014-09-20Report (and free) err_info for WTAP_ERR_DECOMPRESS.Guy Harris1-0/+1
Change-Id: I61ecc58be4b0688c43df0b0f006fb899bdc1721c Reviewed-on: https://code.wireshark.org/review/4216 Reviewed-by: Guy Harris <guy@alum.mit.edu>
2014-09-20Qt: Update the x-capture-file-close icon.Gerald Combs4-49/+45
Change-Id: I9ea4fe42ccdbc85c8c3f22a426b6e08d595a7460 Reviewed-on: https://code.wireshark.org/review/4215 Reviewed-by: Gerald Combs <gerald@wireshark.org>
2014-09-20PIDL: fix trailing tabspaceAlexis La Goutte1-9/+9
Change-Id: If3b623d82d6bbbf9943a35c8885620881eedb091 Reviewed-on: https://code.wireshark.org/review/4204 Reviewed-by: Michael Mann <mmann78@netscape.net>
2014-09-20Qt: Add stock icons.Gerald Combs118-96/+4900
The new icons are patterned after the existing capture icons. They could probably benefit from some adjustments here and there. Start moving toward icon names and a directory layout compliant with freedesktop.org's Icon Naming Specification and Icon Theme Specification. We aren't fully compliant and might never be but anyone with exposure to FDO icon themes should at least know where everything is. Make Capture Start (x-capture-start) the first icon in the toolbar. Define the Colorize Packets, Auto Scroll, and zoom icons even though the Qt UI doesn't use them yet. Leave the Capture Filter, Display Filter, Coloring Rules, Preferences, and Help icons off for now. The GTK+ toolbar is overly cluttered and I'm not sure they're necessary. Try not to break ui/gtk/toolbar_icons.h. Remove welcome.qrc. I initially added it in case we needed to overlay the welcome screen with translucent .pngs but that never happened. To do: - Move the old GTK+ icons to their own directory. - Find a better name for the "toolbar" directory. "stock"? - Make the toolbar configurable. Change-Id: Ie07592113d307b8db786aedace672312a870fe38 Reviewed-on: https://code.wireshark.org/review/4182 Reviewed-by: Gerald Combs <gerald@wireshark.org>
2014-09-20Add .travis.yml for Travis CI (for Github user...)Alexis La Goutte1-0/+20
4 jobs * autotools / gcc * autotools / clang * cmake / gcc * cmake / clang Change-Id: Ifc5b079415232b77680c87b2dee441566e08c835 Reviewed-on: https://code.wireshark.org/review/3944 Reviewed-by: Evan Huus <eapache@gmail.com> Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
2014-09-20Make load_cap_file() just return a success/fail indication.Guy Harris1-7/+5
We don't care *why* it fails; it's printed a message to indicate that. Just have it indicate whether it succeeded or not. Change-Id: I98160999baa5f25ea03ef803b7dbf77dff1518de Reviewed-on: https://code.wireshark.org/review/4213 Reviewed-by: Guy Harris <guy@alum.mit.edu>
2014-09-20Free err_info on WTAP_ERR_DECOMPRESS.Guy Harris1-0/+1
Change-Id: I57fbb17baf43b0f1365fef72778ac0e65935456a Reviewed-on: https://code.wireshark.org/review/4210 Reviewed-by: Guy Harris <guy@alum.mit.edu>
2014-09-20Remove file_error() calls after pcap_read_block() calls.Guy Harris1-2/+0
pcap_read_block() takes err and err_info arguments, and sets them on error; no need to call file_error() if pcap_read_block() fails. Change-Id: I33b96d31395bf7d66abdecbebd5cf775e8662004 Reviewed-on: https://code.wireshark.org/review/4209 Reviewed-by: Guy Harris <guy@alum.mit.edu>
2014-09-20Add file_error() calls after file_read().Guy Harris1-0/+3
If file_read() doesn't return the exact number of bytes you asked for, and you really need all those bytes to be there, you have to call file_error() to find out what the problem is. Change-Id: I4cc87bc1b6cc5a49bbcbf93b56106f57af290d20 Reviewed-on: https://code.wireshark.org/review/4205 Reviewed-by: Guy Harris <guy@alum.mit.edu>
2014-09-20Fix sizing of qtshark based on the saved (recent) geometry.Jeff Morriss1-0/+1
Do this by emitting the StaticRecentFilesRead signal after reading the static portion of the recent file. Unfortunately this results in the main window appearing in one size and then resizing to the saved size but I haven't figured out how to fix that yet. At the very least this prevents qtshark's default geometry from being saved to my recent file and affecting wireshark-gtk's geometry. Change-Id: I47b66078a9d00cb321badc744f1337ff39ca42a7 Reviewed-on: https://code.wireshark.org/review/4177 Reviewed-by: Evan Huus <eapache@gmail.com>
2014-09-20Fix Dead Store (Dead assignement/Dead increment) warning found by ClangAlexis La Goutte1-1/+1
Change-Id: I5bfbfc03795cd8ba3cd102d2249672ce3e48bc5f Reviewed-on: https://code.wireshark.org/review/4179 Reviewed-by: Evan Huus <eapache@gmail.com>
2014-09-20Bluetooth: HCI: Add support for user/vendor commandsMichal Labedzki3-13/+102
Some Vendors can provide own subset of HCI commands/events, so provide them possibility to write own dissectors and use them by "Decode As". Change-Id: I87ff60ae12ac63ddd4b12b26951aa9f5143eabc6 Reviewed-on: https://code.wireshark.org/review/4166 Petri-Dish: Michal Labedzki <michal.labedzki@tieto.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Michael Mann <mmann78@netscape.net> Reviewed-by: Evan Huus <eapache@gmail.com>
2014-09-20Added Wx Application-Id.Josip Medved1-0/+1
Change-Id: I46152382be0a374019d51f18e7116aad8241e95a Reviewed-on: https://code.wireshark.org/review/4162 Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com> Reviewed-by: Evan Huus <eapache@gmail.com>
2014-09-20Add -I option in editcap to extend Duplicate packet removalkardam2-3/+43
Description: Ignore the specified bytes number at the beginning of the frame during MD5 hash calculation. Useful to remove duplicated packets taken on several routers or SW(differents mac addresses for example). e.g. -I 26 in case of Ether/IP/ will ignore ether(14) and IP header(20 - 4(src ip) - 4(dst ip)). The default value is 0. This option is only relevant when used with -d|-D|-w Bug: 8511 Change-Id: I009a09d32778a182b2d88f372651f658a4938882 Reviewed-on: https://code.wireshark.org/review/4104 Tested-by: Evan Huus <eapache@gmail.com> Reviewed-by: Evan Huus <eapache@gmail.com>
2014-09-20ssl: allocate regexes with glibEvan Huus1-2/+3
they're stored in a static variable so we only ever need compile them once and they can just hang around forever Change-Id: Icf43745ad80f4984443a67af21c979625363fc6f Ping-Bug: 10474 Reviewed-on: https://code.wireshark.org/review/4139 Reviewed-by: Evan Huus <eapache@gmail.com>
2014-09-20Set err_info if the attempts to read record headers fail.Guy Harris1-5/+5
If it fails due to, for example, the file being gzipped and having a bad gzip CRC, the error returned is WTAP_ERR_DECOMPRESS and, for that error, err_info is expected to be set to a string giving details of the problem, so we need to pass back to our caller the string in question. Bug: 10484 Change-Id: I3aa2a92d04fcc08946ff073a40efa708079bbb3e Reviewed-on: https://code.wireshark.org/review/4201 Reviewed-by: Guy Harris <guy@alum.mit.edu>
2014-09-20proto_tree_add_text -> proto_tree_add_subtree[_format] for DCE/RPC dissectors.Michael Mann34-7082/+6844
Change-Id: I84755d059ef70ca98b0e7626b6425360daf0529d Reviewed-on: https://code.wireshark.org/review/4199 Petri-Dish: Michael Mann <mmann78@netscape.net> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Michael Mann <mmann78@netscape.net>
2014-09-206LoWPAN: fix encapsulating address decoding when a Mesh header is presentPascal Quantin1-12/+20
Bug: 10462 Change-Id: I37d532edc41193123b1a7bb380d48cb76c118bf4 Reviewed-on: https://code.wireshark.org/review/4161 Petri-Dish: Pascal Quantin <pascal.quantin@gmail.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com> Reviewed-by: Michael Mann <mmann78@netscape.net>
2014-09-20The Shomiti document is available on the Wayback Machine.Guy Harris1-7/+4
I don't know why it was unavailable at some point, but it appears to be available again. Update the comment to reflect that. Change-Id: I7dc1fcb554e73d3b0fc2bd2fbdf6235e791a4253 Reviewed-on: https://code.wireshark.org/review/4200 Reviewed-by: Guy Harris <guy@alum.mit.edu>
2014-09-20Logcat: Fix crashes when try to use logcat_text open routine on binary fileMichal Labedzki1-9/+9
Change-Id: Ied0778af9d5ff0e49c6efd4ea9411ae1a72cb8e5 Reviewed-on: https://code.wireshark.org/review/4190 Petri-Dish: Michal Labedzki <michal.labedzki@tieto.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Michael Mann <mmann78@netscape.net> Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
2014-09-19Don't assert that values fit in 32 bits if the NDR64 flag is set.Michael Mann5-27/+65
For now, this just pacifies fuzz-testing. If real world examples have this, there needs to be a drastic overhaul to support fields that could be either 32 or 64-bit values. Bug:9329 Change-Id: I3e28808ca0291868a5f84258b0ee1e2a922703c2 Reviewed-on: https://code.wireshark.org/review/4189 Reviewed-by: Michael Mann <mmann78@netscape.net>
2014-09-19Bluetooth: ACL/AVDTP: Fix ACL session handlingMichal Labedzki2-11/+40
In case like reusing "connection handle" number in next connection (2 -> 3 -> 2 again) AVDTP dissector does not properly dissected packets (it should be Signal, but dissect as Media). Add expert error to all packets that are not in ACL connection. This will show invalid packages (send after dissconnect), however incomplete (mostly beginning) connection will be marked too, but I think it have more pros than cons. Change-Id: I29cf706600cb12c8b93d0188dd2fba18dd5eba88 Reviewed-on: https://code.wireshark.org/review/4167 Petri-Dish: Michal Labedzki <michal.labedzki@tieto.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Michael Mann <mmann78@netscape.net>
2014-09-19Update the documentation to use "Wireshark.pro".Gerald Combs2-18/+12
Change-Id: I024a882030e489cbd273a4245b0cd3be656f060f Reviewed-on: https://code.wireshark.org/review/4191 Reviewed-by: Gerald Combs <gerald@wireshark.org>
2014-09-19Update generated DCE/RPC dissectors.Michael Mann45-8101/+8780
Now all generated dissectors can successfully compile. Change-Id: I74728926915d5a9f1694cc49c753b80f213df767 Reviewed-on: https://code.wireshark.org/review/2598 Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com> Reviewed-by: Michael Mann <mmann78@netscape.net>
2014-09-19packet-http2.c: Do encoding-arg changes (all benign)Bill Meier1-23/+23
For: - FT_BYTES: Always use just ENC_NA - integral/floating (other than FT_[U]INT8): Do ENC_NA --> ENC_BIG_ENDIAN Change-Id: I0885f7d110014cb8a7eba1c1892ed8d0852d076a Reviewed-on: https://code.wireshark.org/review/4187 Reviewed-by: Bill Meier <wmeier@newsguy.com>
2014-09-19packet-http2.c: Fix a very minor bugBill Meier1-1/+1
- incorrect fetch/display of hf_http2_push_promise_r field Change-Id: Iddf2db4807713017519b5abcbff3ba824b3624c6 Reviewed-on: https://code.wireshark.org/review/4186 Reviewed-by: Bill Meier <wmeier@newsguy.com>
2014-09-19Add editor modelines; Adjust indentation as needed.Bill Meier8-161/+261
Change-Id: Ie8d4056ab706bbc85c8247682b1a64ea98bc89f2 Reviewed-on: https://code.wireshark.org/review/4185 Reviewed-by: Bill Meier <wmeier@newsguy.com>
2014-09-19Do encoding-arg changes (all benign)Bill Meier56-204/+203
For: - FT_BYTES: Always use just ENC_NA - integral/floating (other than FT_[U]INT8): Do ENC_NA --> ENC_BIG_ENDIAN Also: - FT_UINT... --> FT_UINT8 in a few cases (to match proto_tree_add_item...) - Change one case of incorrect '||' to '|' Change-Id: I427e0e61618ff8faf55691c8a695930f67d455b0 Reviewed-on: https://code.wireshark.org/review/4184 Reviewed-by: Bill Meier <wmeier@newsguy.com>
2014-09-19fix-encoding-args.pl: Add a few additional fixups;Bill Meier1-17/+29
Specifically: A number of cases of the use of the encoding arg "ENC_ASCII | ENC_NA" for FT_BYTES FIELDTYPEs seem to have crept into Wireshark source. Also: Add a ToDo comment as to code rework needed to be able to better *validate* encoding args. Change-Id: I9d4b10b869fe2aef3f20bcd2d7ad6531375aba87 Reviewed-on: https://code.wireshark.org/review/4183 Reviewed-by: Bill Meier <wmeier@newsguy.com>
2014-09-18Qt: add support for some more command line optionsPascal Quantin1-5/+183
Change-Id: I63b2c51ba857dc5282897c32594102aad2ef9e2f Reviewed-on: https://code.wireshark.org/review/4127 Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
2014-09-18Bluetooth: Fix some Clang Analyzer warningsMichal Labedzki2-3/+5
Logic error Dereference of null pointer epan/dissectors/packet-bthci_acl.c 379 Logic error Dereference of null pointer epan/dissectors/packet-bthci_acl.c 390 Logic error Dereference of null pointer epan/dissectors/packet-btsdp.c 3673 Logic error Dereference of null pointer epan/dissectors/packet-btsdp.c 1439 Logic error Dereference of null pointer epan/dissectors/packet-btsdp.c 1662 Thanks for Alexis La Goutte for pointing that. Change-Id: Ia646b33de9e368d32983efddfaa41edf7896d7fd Reviewed-on: https://code.wireshark.org/review/4165 Petri-Dish: Michal Labedzki <michal.labedzki@tieto.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Michael Mann <mmann78@netscape.net>
2014-09-18fix-encoding-args.pl: Add a few fixups for FT_UINT_STRINGBill Meier1-0/+3
Change-Id: Ic22e154859877c48a85b808d0029e10e07876b73 Reviewed-on: https://code.wireshark.org/review/4174 Reviewed-by: Bill Meier <wmeier@newsguy.com>
2014-09-18Fix encoding-arg mostly for FT_UINT_STRING; Also: Do misc as needed.Bill Meier16-70/+149
Misc: (as needed) - Add editor modelines; - Adjust whitespace. Change-Id: I0e5d877a2a6273842ad8e771ac0c0acbfcb83200 Reviewed-on: https://code.wireshark.org/review/4173 Reviewed-by: Bill Meier <wmeier@newsguy.com>