aboutsummaryrefslogtreecommitdiffstats
AgeCommit message (Collapse)AuthorFilesLines
2014-03-31Continue to remove $Id$ from top of fileAlexis La Goutte17-33/+0
(Using sed : sed -i '/^ \$Id\$/,+1 d') (No star only 2 spaces before) Change-Id: Id7b254031769a9dca2941304e4d3a0f4bdbc3f54 Reviewed-on: https://code.wireshark.org/review/883 Reviewed-by: Anders Broman <a.broman58@gmail.com>
2014-03-31Continue to remove $Id$ from top of fileAlexis La Goutte47-47/+0
(Using sed : sed -i '/^<!-- \$Id\$ -->/,+0 d') (XML Comments) Change-Id: Ifd03f70ae84dfbbbe04d91f10fd30857fd197014 Reviewed-on: https://code.wireshark.org/review/882 Reviewed-by: Anders Broman <a.broman58@gmail.com>
2014-03-31Doc tweaks and macro parameterizationEvan Huus2-62/+73
Change-Id: I9898bedf05a785683e79866a149336cbbf402d27 Reviewed-on: https://code.wireshark.org/review/892 Reviewed-by: Evan Huus <eapache@gmail.com>
2014-03-31Continue to remove $Id$ from top of fileAlexis La Goutte6-18/+0
Remove all $Id:$ (by hand) Change-Id: Ia8790ee852fe338469e855713199220d345a59b7 Reviewed-on: https://code.wireshark.org/review/880 Reviewed-by: Anders Broman <a.broman58@gmail.com>
2014-03-31Continue to remove $Id$ from top of fileAlexis La Goutte35-69/+1
(Using sed : sed -i '/^\* \$Id\$/,+1 d') (no space before star) Change-Id: I318968db2b8512ba1303b5fc5c624c66441658f0 Reviewed-on: https://code.wireshark.org/review/879 Reviewed-by: Anders Broman <a.broman58@gmail.com>
2014-03-31Continue to remove $Id$ from top of fileAlexis La Goutte39-116/+38
(Using sed : sed -i '/^ \t$Id\$/,+0 d') (tab before $Id$) Also modify generator (NDR.pm) Change-Id: I348a1d129d1d1320bd80b428038ea5ed291d6ca8 Reviewed-on: https://code.wireshark.org/review/878 Reviewed-by: Anders Broman <a.broman58@gmail.com>
2014-03-31Continue to remove $Id$ from top of fileAlexis La Goutte35-74/+0
(Using sed : sed -i '/^\$Id\$/,+1 d') (No space or star before $Id$) Change-Id: I0801bd7cf234d32487008a8b6dcee64875b07688 Reviewed-on: https://code.wireshark.org/review/876 Reviewed-by: Evan Huus <eapache@gmail.com>
2014-03-31Continue to remove $Id$ from top of fileAlexis La Goutte7-14/+0
(Using sed : sed -i '/^ \* \$Id\$/,+1 d') (Double space between star and $Id$) Change-Id: If9b8f345e3b6493de0b573600e60005c8b0b33c3 Reviewed-on: https://code.wireshark.org/review/877 Reviewed-by: Evan Huus <eapache@gmail.com>
2014-03-31Use tvb_get_string_enc() with UTF_8 when checking headers as it's faster.AndersBroman1-1/+1
Change-Id: I88cd6f71c1fcbf67948dd0ebdc6e5641d6ba111c Reviewed-on: https://code.wireshark.org/review/891 Reviewed-by: Anders Broman <a.broman58@gmail.com>
2014-03-31According to RFC 3261AndersBroman1-50/+51
7 SIP Messages SIP is a text-based protocol and uses the UTF-8 charset So use ENC_UTF_8|ENC_NA. Change-Id: I0101eca3dd7d8ff9ebf98fd733548131b862919e Reviewed-on: https://code.wireshark.org/review/890 Reviewed-by: Anders Broman <a.broman58@gmail.com>
2014-03-31Inlining wmem_strbuf_grow seems to give better performance.AndersBroman1-1/+1
Change-Id: I80f3d42fb01461af97984b463a6e8b8217eed552 Reviewed-on: https://code.wireshark.org/review/888 Reviewed-by: Evan Huus <eapache@gmail.com> Reviewed-by: Anders Broman <a.broman58@gmail.com>
2014-03-31Add --enable-extra-clang-checks for autotools and CMakeListAlexis La Goutte3-8/+41
With -Wunreachable-code flags (and disable for the moment -Wdocumentation) Change-Id: I126c962b32e650a63b78092e95896736ae7335c9 Reviewed-on: https://code.wireshark.org/review/678 Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
2014-03-31Fix Argument with 'nonnull' attribute passed null found by ClangAlexis La Goutte1-0/+1
Change-Id: Ie59c379a4f00c807ebc50c95eea2017f8cac0c69 Reviewed-on: https://code.wireshark.org/review/796 Reviewed-by: Evan Huus <eapache@gmail.com>
2014-03-31packet-osc dissector bug fix of path validity funcHanspeter Portner1-2/+3
- 'is_valid_path' function has been corrected - an OSC path is valid if: - it consists of printable characters only - does not contain ' ' and '#' - characters '*' ',' '?' '[' ']' '{' '}' are valid, but have special meaning at the receiving end (used for pattern matching) Change-Id: I4ff4308d0955da2ef377d606b7778819b97754a0 Reviewed-on: https://code.wireshark.org/review/868 Reviewed-by: Evan Huus <eapache@gmail.com> Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
2014-03-30Fix fuzz-test.sh '-b' option so it works again; Do some other ↵Bill Meier4-36/+44
tools/*test*.sh changes - Add -b option to randpkt-test.sh and test-captures.sh; - Create/ue a common function to do '-x' tests on files/dirs; - Rename exit_error function to ws_exit_error Change-Id: I032c9d784bec1fb6b0717aaad08a061e4d935476 Reviewed-on: https://code.wireshark.org/review/872 Reviewed-by: Bill Meier <wmeier@newsguy.com> Tested-by: Bill Meier <wmeier@newsguy.com>
2014-03-30Fix Bug 9938 'tshark "-Y -2 -w" saves non-dependent frames'Hadriel Kaplan1-3/+10
When the '-Y' display filter option is given with a '-2', and a '-w' to write out the packets, tshark grabs *all* dependent frames in the catprue file, even those that weren't dependents of a matching packet. Note that this also uses the '-2' two-pass option, since only two-pass mode writes out dependent frames to begin with. Change-Id: I17726447bec434ba2566e98fb78893d1331e3056 Reviewed-on: https://code.wireshark.org/review/866 Reviewed-by: Hadriel Kaplan <hadrielk@yahoo.com> Reviewed-by: Evan Huus <eapache@gmail.com>
2014-03-30[Automatic manuf, services and enterprise-numbers update for 2014-03-30]Gerald Combs3-8/+185
Change-Id: Iff714193adc4035d96f69d4944b43da7edb3cfa8 Reviewed-on: https://code.wireshark.org/review/869 Reviewed-by: Evan Huus <eapache@gmail.com>
2014-03-29Hide unused variable.Guy Harris1-1/+4
Leave it there, but commented out, just in case it *should* be used. "#if 0" out the code that sets it. Change-Id: I8802fc416030106d9d8421b0d7b8612597794bab Reviewed-on: https://code.wireshark.org/review/867 Reviewed-by: Guy Harris <guy@alum.mit.edu>
2014-03-29Rename "size" variable to "length", to match other string routines.Guy Harris1-4/+4
Change-Id: I385345cfafc7e7b4d3e66713fb0fe570b39f438d Reviewed-on: https://code.wireshark.org/review/865 Reviewed-by: Guy Harris <guy@alum.mit.edu>
2014-03-29Add a pcorey48tohll() macro, and use #defines for some offsets.Guy Harris1-26/+25
The "48 bits, in a weird byte order" is a variant of 64-bit "Corey-endian", with the upper 16 bits of the result - which are in the *middle* of the bytes of the number - ignored. Define a pcorey48tohll() macro and use that, rather than the loop. There are a bunch of #defines for offsets in the headers; use them rather than magic constants. Change-Id: Idfdc8a741278d71a5db47c067914c97615c3e02d Reviewed-on: https://code.wireshark.org/review/864 Reviewed-by: Guy Harris <guy@alum.mit.edu>
2014-03-29Use sized strbufs when extracting tvb stringsEvan Huus1-33/+19
We rarely know exactly how long a string will be, but we frequently have a good lower bound (that's better than the default strbuf size of 16). Starting at that size probably reduces the amount of allocation/copying needed. Also make use of the new _finalize() method to save memory and avoid constness problems. Change-Id: I3f043bd12c1ccfce5990168fb6531ecd287bec5b Reviewed-on: https://code.wireshark.org/review/856 Reviewed-by: Guy Harris <guy@alum.mit.edu> Reviewed-by: Evan Huus <eapache@gmail.com>
2014-03-29Splay tree implementation for wmemEvan Huus6-1/+711
This is a tree implementation intended to replace the current red-black tree in wmem_tree (which was inherited from emem), assuming there are no regressions. Splay trees bubble recently accessed keys to the top, and as such have a number of very nice properties: https://en.wikipedia.org/wiki/Splay_tree This implementation is a variant known as "independent semi-splaying", which has better practical performance. It should do about as well as the red-black tree for random insertions and accesses, but somewhat better for patterned accesses (such as accessing each key in order, or accessing certain keys very frequently). There are a few other changes relative to the red-black tree implementation that are worth mentioning: - Instead of requiring complex keys to be split into guint32 chunks and doing this weird trick with sub-trees, I let the keys be arbitrary pointers and allowed the user to specify an arbitrary comparison function. If the function is NULL then the pointers are compared directly for the simple integer-key case. - Splay trees do not need to store a red-black colour flag for each node. It is also much easier to do without the parent pointer in each node. And due to the simpler system for complex keys, I was able to remove the "is_subtree" boolean. As such, splay nodes are 12 bytes smaller on 32-bit platforms, and 16 bytes smaller on a 64-bit platform. All done in about half the lines of code. Change-Id: I89fb57e07d2bb7e3197190c7c2597b0c5adcc03b Reviewed-on: https://code.wireshark.org/review/758 Reviewed-by: Evan Huus <eapache@gmail.com>
2014-03-29Upgrade Windows builds to Lua 5.2.1Pascal Quantin6-11/+12
Change-Id: I5d2058673b69beb8393aae0571703cd9dbb79c4e Reviewed-on: https://code.wireshark.org/review/863 Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com> Tested-by: Pascal Quantin <pascal.quantin@gmail.com>
2014-03-29Tighten the NO_MORE_DATA_CHECK macrosEvan Huus2-2/+2
We should return if we've already somehow overflowed the apparent length, not just if we've hit it dead on. Fixes bug #9932 and probably others. Change-Id: I2719c3bd38f03e18ea557df962ee4a1beb64c2e8 Reviewed-on: https://code.wireshark.org/review/862 Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com> Reviewed-by: Evan Huus <eapache@gmail.com>
2014-03-29Fix some compiler issues on windows with Lua 5.2Hadriel Kaplan4-6/+13
Change-Id: I5c692bbc5fd1a9f0d361413e44e70282c3299901 Reviewed-on: https://code.wireshark.org/review/860 Reviewed-by: Hadriel Kaplan <hadrielk@yahoo.com> Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com> Tested-by: Pascal Quantin <pascal.quantin@gmail.com>
2014-03-29Implement finalize() method for wmem strbuf.Evan Huus3-0/+30
Change-Id: Ib7a2b0d348b3624f41253e2d0995a4a38a9fe45d Reviewed-on: https://code.wireshark.org/review/859 Reviewed-by: Evan Huus <eapache@gmail.com>
2014-03-28Please checkAPI.plPascal Quantin1-3/+3
Change-Id: I78e9d3781df81fd4edda7e9b20749cb096a05499 Reviewed-on: https://code.wireshark.org/review/861 Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
2014-03-28Update README.wslua with latest info.Hadriel Kaplan1-56/+135
Change-Id: I20ea6c374f791054f16f0aaba33967b869348ff5 Reviewed-on: https://code.wireshark.org/review/857 Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
2014-03-28Add handling of OSC over TCP; fix a bug in the UDP heuristic.Bill Meier1-26/+92
Bug Fixed: UDP heuristic wasn't properly setting the dissector for the UDP conversation. From https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9837#c3: I did a little survey about alternative OSC transmission, but UDP. As OSC is packet-based, it is tramitted raw via packet-oriented protocols (e.g. UDP). For reliable stream-based protocols (TCP, USB), the raw OSC packet is prefixed with the packets Int32 size as a packetization scheme. For unreliable stream-based protocols (RS232 and other serial lines), the raw OSC packet is SLIP and/or double SLIP encoded as packetization scheme. There was discussion in the past to make SLIP encoding the default for all stream-based protocols, but apparently it has never been adopted for any OSC via TCP implementation I've found in the web. As OSC is used in networked Audio, most implementations run with the Nagle algorithm disable, and send the prefixed length and the raw OSC packet separately. Change-Id: Ife690cc5ea0575c65124a7b441431e1cc6ba5091 Reviewed-on: https://code.wireshark.org/review/858 Reviewed-by: Bill Meier <wmeier@newsguy.com> Tested-by: Bill Meier <wmeier@newsguy.com>
2014-03-28tvb_get_ascii_string is really expensive. On a large capture file which IAndersBroman2-49/+49
profiled in october Fetch cost has gone from 15,6M to 24,2M, changing tvb_get_string() to tvb_get_string_enc() with ENC_UTF_8 where it seems safe helps a bit and should be done any way. Change-Id: I4d3e640bfde3304a991c09e2a30ad7dd132fc5ac Reviewed-on: https://code.wireshark.org/review/855 Reviewed-by: Anders Broman <a.broman58@gmail.com>
2014-03-28Fix crash when calling "MAP Summary" dialog when no file is openPascal Quantin1-0/+4
Fixes bug 9934 Change-Id: I2915d0715431cf983511fe7f5f036be99bbd2467 Reviewed-on: https://code.wireshark.org/review/853 Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com> Tested-by: Pascal Quantin <pascal.quantin@gmail.com>
2014-03-28Deleted one line to much in previous chgange.AndersBroman1-0/+1
Change-Id: I19d1153558130356286bac55f6cb321d30a767b1 Reviewed-on: https://code.wireshark.org/review/852 Reviewed-by: Anders Broman <a.broman58@gmail.com>
2014-03-28Allow chained calls with Lua TreeItem functions, and fix a couple of minor ↵Hadriel Kaplan5-23/+121
errors. A common Lua idiom is to use chained calls, i.e. tree:foo():bar():choo(). This actually works for tree:add() because it returns the new child tree item which is then the one being applied to the next chained call. But it doesn't work beyond that for things like set_generated() and so on. So this commit fixes that. This also fixes the Lua tree:add() function for the FT_BOOL type to let it be a Lua boolean value. And it reverts a previous change to Struct.tohex() to allow coercion of the argument. Change-Id: I10f819d363163914ba320c87d4bedebe5b50cacf Reviewed-on: https://code.wireshark.org/review/851 Reviewed-by: Anders Broman <a.broman58@gmail.com>
2014-03-27dissect sRGB chunkMartin Kaiser1-0/+23
Change-Id: I5a9eefb7f5108de0e46b70453a4485b1cbc3983d Reviewed-on: https://code.wireshark.org/review/850 Reviewed-by: Martin Kaiser <wireshark@kaiser.cx>
2014-03-27use an expert info if a png chunk is too large for usMartin Kaiser1-8/+20
Change-Id: I6a06bf297ef68ca351deff2e08eec7cd12ba8fe0 Reviewed-on: https://code.wireshark.org/review/849 Reviewed-by: Martin Kaiser <wireshark@kaiser.cx>
2014-03-27Fix bug9931 'Encapsulated ethernet packets sometimes show invalid FCS'Hadriel Kaplan9-10/+18
This fixes part-1 of bug9931: the uninitialized use of a wtap_pkthdr struct. The second part of the bug deals with dissectors calling the Ethernet dissector for ecnapsulated Ethernet packets but using the wrong dissector handle to do so. That's unrelated to the issue this commit addresses, so I'm splitting them up. Change-Id: I87be7b736f82dd74d8c261062f88143372b5344c Reviewed-on: https://code.wireshark.org/review/848 Reviewed-by: Hadriel Kaplan <hadrielk@yahoo.com> Reviewed-by: Anders Broman <a.broman58@gmail.com>
2014-03-27Add dissection of 3G Interface Type package and add some comments aboutAndersBroman3-8/+58
the usage of variables in the h248_package_t struct. Change-Id: Ic5419ab5c20051e56963fe8ea1728d78f95538f0 Reviewed-on: https://code.wireshark.org/review/846 Reviewed-by: Anders Broman <a.broman58@gmail.com>
2014-03-27Regenerate the dissector with the latest template.AndersBroman1-2/+2
Change-Id: Iefdd7122537108c37e71650ad2674418109d2f1b Reviewed-on: https://code.wireshark.org/review/845 Reviewed-by: Anders Broman <a.broman58@gmail.com>
2014-03-27NAS EPS: display also detach cause in case of network initiated procedurePascal Quantin1-7/+11
Change-Id: I36fef9eb8a81b03f29c812f94524f4cdf056448b Reviewed-on: https://code.wireshark.org/review/844 Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com> Tested-by: Pascal Quantin <pascal.quantin@gmail.com>
2014-03-27MBIM: fix identification of DSS streams and add a way to register an ↵Pascal Quantin2-10/+19
external dissector for DSS Change-Id: Ic7f5708b60ee5846d95725dbbd14d8958d43372e Reviewed-on: https://code.wireshark.org/review/843 Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com> Tested-by: Pascal Quantin <pascal.quantin@gmail.com>
2014-03-27From Martin Mathiesson: Display Detach Type in info col.AndersBroman1-2/+8
Change-Id: I33692ac93f60cbabec0804f6ee7e75a9d5b9497c Reviewed-on: https://code.wireshark.org/review/842 Reviewed-by: Anders Broman <a.broman58@gmail.com>
2014-03-27Add paths to Lua package.path so require works for user scriptsHadriel Kaplan2-3/+64
This adds the global and personal plugins directories to the package.path setting in Lua, so doing 'require' will work properly. Change-Id: Iec33bc60cd7d41aa122da456db91d4ccc3085f82 Reviewed-on: https://code.wireshark.org/review/841 Reviewed-by: Hadriel Kaplan <hadrielk@yahoo.com> Reviewed-by: Anders Broman <a.broman58@gmail.com>
2014-03-26two more chunk namesMartin Kaiser1-0/+4
Change-Id: I5a34a59c88a4119be2fac4acdd352d474ffc62cd Reviewed-on: https://code.wireshark.org/review/840 Reviewed-by: Martin Kaiser <wireshark@kaiser.cx> Tested-by: Martin Kaiser <wireshark@kaiser.cx>
2014-03-26add some more chunk namesMartin Kaiser1-0/+8
Change-Id: I6d294a901af88f993ca6a44ababad194fb44a693 Reviewed-on: https://code.wireshark.org/review/839 Reviewed-by: Martin Kaiser <wireshark@kaiser.cx> Tested-by: Martin Kaiser <wireshark@kaiser.cx>
2014-03-26clean up the png dissectorMartin Kaiser1-64/+69
use value_string instead of our own data type read the type string in one go Change-Id: I115c99c4636540702c1fd301f09a92a0dd466fcd Reviewed-on: https://code.wireshark.org/review/838 Reviewed-by: Martin Kaiser <wireshark@kaiser.cx> Tested-by: Martin Kaiser <wireshark@kaiser.cx>
2014-03-26use the correct png chunk lenMartin Kaiser1-1/+1
the parameter for proto_tree_add_text() is the length, not the end offset Change-Id: Ie24d5982b7ff13363061fb087438dd714cae748a Reviewed-on: https://code.wireshark.org/review/837 Reviewed-by: Martin Kaiser <wireshark@kaiser.cx>
2014-03-26More fiddling with heuristic.AndersBroman1-2/+2
Change-Id: I50b78cc38c6f529695ecd54ced0ed94e8a683534 Reviewed-on: https://code.wireshark.org/review/835 Reviewed-by: Anders Broman <a.broman58@gmail.com>
2014-03-26Fix off by one errorAndersBroman2-5/+5
Change-Id: I24c9e988786a95489f918bed2c043e6eb23e4caa Reviewed-on: https://code.wireshark.org/review/834 Reviewed-by: Anders Broman <a.broman58@gmail.com>
2014-03-26Postpone replacing GTK+ stock icon usageBalint Reczey1-10/+16
This prevents regressions observable on GTK+ 3.10 which caused icons to disappear in some cases. Change-Id: Ic971258807510dc038e30c6c64fb547f0529a3c2 Reviewed-on: https://code.wireshark.org/review/792 Reviewed-by: Balint Reczey <balint@balintreczey.hu>
2014-03-26LTE RRC: add file forgotten in gb981173Pascal Quantin1-1/+8
Change-Id: I1727e56e678c32402f6331463c06c62072c954d8 Reviewed-on: https://code.wireshark.org/review/833 Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
generated by cgit v1.2.3 (git 2.25.1) at 2024-05-20 00:59:03 +0000