aboutsummaryrefslogtreecommitdiffstats
path: root/epan/tvbparse.c
AgeCommit message (Collapse)AuthorFilesLines
2019-07-26HTTPS (almost) everywhere.Guy Harris1-1/+1
Change all wireshark.org URLs to use https. Fix some broken links while we're at it. Change-Id: I161bf8eeca43b8027605acea666032da86f5ea1c Reviewed-on: https://code.wireshark.org/review/34089 Reviewed-by: Guy Harris <guy@alum.mit.edu>
2019-01-03epan: remove redundant casts.Dario Lombardo1-2/+2
Found by clang-tidy. Change-Id: Iaf6cf84c33b03ddfcd39a333b49f4987002afa56 Reviewed-on: https://code.wireshark.org/review/31338 Petri-Dish: Dario Lombardo <lomato@gmail.com> Petri-Dish: Guy Harris <guy@alum.mit.edu> Tested-by: Petri Dish Buildbot Reviewed-by: Guy Harris <guy@alum.mit.edu>
2018-09-19Try to discourage the use of APIs via counting.Gerald Combs1-46/+45
Add the ability to specify maximum function counts for each group to checkAPIs. Add maximum counts for the "termoutput" and "abort" groups where needed. Show summaries in various checkAPI targets. Switch uses of ws_g_warning back to plain g_warning. Change-Id: I5cbddc8c671729e424eed8551f69116d16491976 Reviewed-on: https://code.wireshark.org/review/29721 Petri-Dish: Peter Wu <peter@lekensteyn.nl> Tested-by: Petri Dish Buildbot Reviewed-by: Peter Wu <peter@lekensteyn.nl>
2018-02-08epan: use SPDX indentifiers.Dario Lombardo1-13/+1
Skipping dissectors dir for now. Change-Id: I717b66bfbc7cc81b83f8c2cbc011fcad643796aa Reviewed-on: https://code.wireshark.org/review/25694 Petri-Dish: Dario Lombardo <lomato@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2018-01-09Fix tvbparse recursion limit check.Stig Bjørlykke1-5/+24
When doing recursion check we must also count down when done. Bug: 14253 Change-Id: Icacc86e8b25e106e151117dbcc2f132b1bbe898e Reviewed-on: https://code.wireshark.org/review/25226 Petri-Dish: Stig Bjørlykke <stig@bjorlykke.org> Tested-by: Petri Dish Buildbot Reviewed-by: Gerald Combs <gerald@wireshark.org>
2018-01-08Limit tvbparse recursion.Gerald Combs1-2/+21
Add a recursion check to tvbparse so that we don't overflow our stack. Bug: 14253 Change-Id: I0f667c3720311318267a1184b33e33253f8ff729 Reviewed-on: https://code.wireshark.org/review/25202 Reviewed-by: Gerald Combs <gerald@wireshark.org> Petri-Dish: Gerald Combs <gerald@wireshark.org> Tested-by: Petri Dish Buildbot Reviewed-by: Michael Mann <mmann78@netscape.net>
2017-10-26Add a ws_in6_addr typedef for struct e_in6_addr.Guy Harris1-1/+1
That allows a parallel typedef of ws_in4_addr for guint32. Change-Id: I03b230247065e0e3840eb87635315a8e523ef562 Reviewed-on: https://code.wireshark.org/review/24073 Reviewed-by: Guy Harris <guy@alum.mit.edu>
2016-08-28tvbparse: fix memleakPeter Wu1-0/+10
Use same wmem_epan_scope() as "w" (tvbparse_wanted_t). Change-Id: I73fdb1fb3b55a91b7bb0fc36e435024c6f0b3d73 Ping-Bug: 12790 Reviewed-on: https://code.wireshark.org/review/17361 Petri-Dish: Peter Wu <peter@lekensteyn.nl> Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Peter Wu <peter@lekensteyn.nl>
2016-07-26Fix some of the checkAPIs.pl warnings for g_warning.Michael Mann1-45/+46
1. Create ws_g_warning for legitimate uses of g_warning 2. Use proto_tree_add_debug_text 3. Comment some out Change-Id: Ida044bf40286b955fdd529c4f9907c8e09b3d7c5 Reviewed-on: https://code.wireshark.org/review/16678 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>
2016-06-14tvbparse: change g_hash_table to wmem_map.Dario Lombardo1-4/+4
Change-Id: I245595051617f6d6333314240825a526adf19339 Reviewed-on: https://code.wireshark.org/review/15244 Reviewed-by: Michael Mann <mmann78@netscape.net>
2016-06-14tvbparse: change g_malloc to wmem_alloc.Dario Lombardo1-13/+13
Change-Id: I6a6363400d9fd1d76b9de0c5deea83dfc7cea1e2 Reviewed-on: https://code.wireshark.org/review/15245 Petri-Dish: Michael Mann <mmann78@netscape.net> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Michael Mann <mmann78@netscape.net>
2016-06-13tvbparse: change g_strdup_printf to wmem_strdup_printf.Dario Lombardo1-2/+2
Change-Id: I57ff604d23967eb1f6481c7019e3eab0389dc3a5 Reviewed-on: https://code.wireshark.org/review/15257 Petri-Dish: Dario Lombardo <lomato@gmail.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Guy Harris <guy@alum.mit.edu> Reviewed-by: Dario Lombardo <lomato@gmail.com>
2016-02-15Use cast to char for explicit conversion [-Woverflow]João Valverde1-1/+1
Change-Id: If33cf41f46f2be9c66fc4a626af6a2c010fba7d3 Reviewed-on: https://code.wireshark.org/review/13931 Reviewed-by: Michael Mann <mmann78@netscape.net> 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>
2015-11-27Officially remove tvb_get_string and tvb_get_stringz.Michael Mann1-1/+1
Almost all replacements were done awhile ago, just put the final nail in the coffin. Change-Id: I0a708d886da5a500c2a1e2c9ee2736794bdb9411 Reviewed-on: https://code.wireshark.org/review/12206 Reviewed-by: Michael Mann <mmann78@netscape.net> Petri-Dish: Michael Mann <mmann78@netscape.net> Reviewed-by: Anders Broman <a.broman58@gmail.com>
2015-09-15Eliminate proto_tree_add_text from odds and ends.Michael Mann1-1/+1
A few calls in the epan directory and comments in the ui directory Change-Id: Ia8f8830ac6909ab94d3a03283bfd173456bc9718 Reviewed-on: https://code.wireshark.org/review/10492 Petri-Dish: Michael Mann <mmann78@netscape.net> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Michael Mann <mmann78@netscape.net>
2015-02-24epan: fix build error with gcc5Daniel Mack1-1/+1
gcc5 bails out with the below error. CC libwireshark_la-tvbparse.lo CC libwireshark_la-tvbuff_base64.lo tvbparse.c: In function 'cond_not_char': tvbparse.c:254:18: error: logical not is only applied to the left hand side of comparison [-Werror=logical-not-parentheses] if (! offset < tt->end_offset ) { ^ cc1: all warnings being treated as errors Makefile:1843: recipe for target 'libwireshark_la-tvbparse.lo' failed Change-Id: I4e8654e97079da6b39be580fc766eecefb4c7ea6 Reviewed-on: https://code.wireshark.org/review/7339 Reviewed-by: Guy Harris <guy@alum.mit.edu>
2015-01-17Replace all remaining instances of ep_new with wmem_new and wmem_packet_scope.Michael Mann1-22/+21
Change-Id: I50b2cd0e233554bc58082e96e8d879b40edfc0c1 Reviewed-on: https://code.wireshark.org/review/6595 Reviewed-by: Michael Mann <mmann78@netscape.net>
2014-10-10Add editor modelines; Adjust whitespace as needed.Bill Meier1-0/+12
Change-Id: I3dc57f4c2ca57585103e3b71503ac4c332903e50 Reviewed-on: https://code.wireshark.org/review/4594 Reviewed-by: Bill Meier <wmeier@newsguy.com>
2014-09-03tvbparse: do not increment the token length when the new sub token length is 0Pascal Quantin1-2/+4
It fixes the unwanted addition of ignore characters when using tvbparse_some as seen with the capture from bug 10417 Change-Id: If68b587e1821fb34aa1fb56666986e1fa6385c33 Reviewed-on: https://code.wireshark.org/review/3905 Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
2014-06-18Fix compilation warningYan Burman1-2/+2
Change-Id: Ia7922415d8b22ba0c5a04ad6940c7e10bc205d57 Reviewed-on: https://code.wireshark.org/review/2371 Reviewed-by: Evan Huus <eapache@gmail.com>
2014-06-18Optimize tvbparse character set checking.Jakub Zawadzki1-61/+29
This patch makes tvbparse_[not_]chars() to generate array of characters which is accepted, later this array is checked in cond_chars_common(). This results in nice speedup of XML dissector (~33% for my file). Change-Id: I62a5585f8bccaaea1a0c49fc70c7552531493442 Reviewed-on: https://code.wireshark.org/review/2356 Reviewed-by: Anders Broman <a.broman58@gmail.com>
2014-06-03Change ep_alloc() to wmem_new() change depcrecated API.AndersBroman1-5/+5
Change-Id: I1eb3d3f3ccdfa265099446e8d609273867304a73 Reviewed-on: https://code.wireshark.org/review/1936 Reviewed-by: Anders Broman <a.broman58@gmail.com>
2014-03-04Remove all $Id$ from top of fileAlexis La Goutte1-2/+0
(Using sed : sed -i '/^ \* \$Id\$/,+1 d') Fix manually some typo (in export_object_dicom.c and crc16-plain.c) Change-Id: I4c1ae68d1c4afeace8cb195b53c715cf9e1227a8 Reviewed-on: https://code.wireshark.org/review/497 Reviewed-by: Anders Broman <a.broman58@gmail.com>
2014-01-13tvparse_until() is used, so don't make it static.Guy Harris1-1/+1
svn path=/trunk/; revision=54738
2014-01-13#if 0 out unused routines claimed to be "untested" and #if 0'ed out inGuy Harris1-1/+1
epan/tvparse.h, and static routines used by that code. svn path=/trunk/; revision=54727
2013-12-24Replace all(?) remaining ep_stack occurences with wmem stacks.Evan Huus1-10/+11
svn path=/trunk/; revision=54435
2013-09-22emem -> wmem conversion:Pascal Quantin1-1/+1
- ep_tvb_get_bits() -> wmem_packet_tvb_get_bits() - tvb_g_memdup()/ep_tvb_memdup() -> tvb_memdup() - tvb_fake_unicode()/tvb_get_ephemeral_faked_unicode() -> tvb_get_faked_unicode() - tvb_get_g_string()/tvb_get_ephemeral_string()/tvb_get_seasonal_string() -> tvb_get_string() - tvb_get_g_unicode_string()/tvb_get_ephemeral_unicode_string() -> tvb_get_unicode_string() - tvb_get_ephemeral_string_enc() -> tvb_get_string_enc() - update docs accordingly svn path=/trunk/; revision=52172
2013-03-02From Ed Beroset via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8416 :Pascal Quantin1-53/+53
Remove C++ incompatibilities from tvbparse and tvbuff svn path=/trunk/; revision=48025
2012-09-20We always HAVE_CONFIG_H so don't bother checking whether we have it or not.Jeff Morriss1-2/+0
svn path=/trunk/; revision=45016
2012-06-28Update FSF address - part II.Jakub Zawadzki1-1/+1
svn path=/trunk/; revision=43538
2012-03-26Wrap a couple of very long lines.Chris Maynard1-2/+12
svn path=/trunk/; revision=41770
2010-12-31From Jakub Zawadzki via bug 4753: New dissector: JSON, w/minor changes by me:Chris Maynard1-3/+3
1) Add links to RFC 4627 and the json.org web site. 2) Comment out hf_json_member_key to keep tools/checkhf.pl happy. 3) Avoid duplicate (application/json) from being displayed in Info column. svn path=/trunk/; revision=35317
2010-04-03(Trivial): indentation & whitespace cleanup.Bill Meier1-404/+409
svn path=/trunk/; revision=32366
2010-04-03 From Yaniv Kaul: constify parametersBill Meier1-33/+33
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=4422 From me: Fix a number of instances where the function prototype or the function definition wasn't changed so there was a mismatch thus causing Windows (but not gcc) compilation errors. svn path=/trunk/; revision=32365
2010-04-02Revert SVN #32360 until Windows compilation errors corrected.Bill Meier1-29/+29
svn path=/trunk/; revision=32361
2010-04-02From Yaniv Kaul: constify parametersBill Meier1-29/+29
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=4422 svn path=/trunk/; revision=32360
2010-01-28Fix various gcc -Wshadow warnings.Bill Meier1-4/+4
svn path=/trunk/; revision=31720
2009-04-08More size_t fixes.Gerald Combs1-2/+2
svn path=/trunk/; revision=28000
2008-12-20Fix some typos and spelling (mostly in text strings)Bill Meier1-1/+1
svn path=/trunk/; revision=27068
2008-08-05luis.ontanon@gmail.com => luis@ontanon.orgLuis Ontanon1-1/+1
svn path=/trunk/; revision=25937
2007-03-28Remove almost all of the casts I committed recently and in place ofStephen Fisher1-1/+1
them, add -Wno-pointer-sign to CFLAGS when gcc will accept it. svn path=/trunk/; revision=21253
2007-03-26Fix a bunch more warnings. Add -Werror when using --with-warnings-as-errorsStephen Fisher1-1/+1
under gcc to tools/lemon, plugins/mate and epan/ svn path=/trunk/; revision=21204
2006-05-21name changeRonnie Sahlberg1-2/+2
svn path=/trunk/; revision=18197
2006-02-09Add heuristic dissection of XMLLuis Ontanon1-2/+34
svn path=/trunk/; revision=17228
2005-12-28Change tvbparse's engine to use callbacks instead of a big switch.Luis Ontanon1-554/+1083
add some primitives. svn path=/trunk/; revision=16916
2005-11-22Trivial warning fixesJörg Mayer1-1/+1
svn path=/trunk/; revision=16561
2005-10-01one step further... Luis Ontanon1-3/+7
TPG now uses the ignore feature of tvbparse named sub_rules can have cardinality epan/tvbparse.c: do not crash on zero cardinality svn path=/trunk/; revision=16065
2005-09-29I needed to cut'n'paste something more...Luis Ontanon1-1/+6
I did not even try to compile it before check it in, sorry! svn path=/trunk/; revision=16057
2005-09-29add a debugging function that creates a tree exploding the parsed elementsLuis Ontanon1-0/+32
svn path=/trunk/; revision=16054
2005-09-28tvbparse.[ch]:Luis Ontanon1-26/+67
- add a tvbparse_handle() (for recursion) - change tvbparse_until() to allow more control when parsing - make the wanted control an union so that different types of data can be used packet-xml.c: - change the parser definition to match changes to tvbparse_until() svn path=/trunk/; revision=16045