Commit Graph

13604 Commits

Author SHA1 Message Date
Guy Harris 8b33d8b19a Keep a list of all the x11_conv_data_t structures, so that when we're
freeing our data structures, we can go through all those structures and
free up the hash tables associated with them, rather than just leaking
them.

svn path=/trunk/; revision=13891
2005-03-24 04:44:33 +00:00
Guy Harris b598c79446 GtkStyles are reference-counted, and a style returned by
"gtk_style_copy()" has a reference count of 1; release the reference
when we're finished setting other widgets' styles to it.

svn path=/trunk/; revision=13890
2005-03-24 03:58:38 +00:00
Guy Harris af2aab9ee1 Plug some more small leaks.
svn path=/trunk/; revision=13889
2005-03-24 03:10:31 +00:00
Guy Harris e4177b39e9 Plug a small leak found by the "leaks" tool on OS X.
svn path=/trunk/; revision=13888
2005-03-23 22:49:32 +00:00
Guy Harris 873bd2bc91 The correlation ID is a 32-bit integer, not a string.
svn path=/trunk/; revision=13887
2005-03-23 22:30:37 +00:00
Guy Harris 7176376813 The DNS name data starts at the beginning of the suboption data, not the
beginning of the suboption.

svn path=/trunk/; revision=13886
2005-03-23 21:42:40 +00:00
Anders Broman 4b29cdad3c Some more IE stuff and added some ref:s
svn path=/trunk/; revision=13885
2005-03-23 21:38:05 +00:00
Guy Harris 06009efc45 When checking whether suboption data fits within the option, check
whether the suboption data offset plus the suboption data length is >,
not >=, the end of the option data - the end of the option data is the
first byte *after* the option data.

For CableLabs string suboptions, use the right offset when fetching the
data.

svn path=/trunk/; revision=13884
2005-03-23 21:22:18 +00:00
Guy Harris 7fd976834e From Anton Ivanov: add support for Frame Relay over ATM.
svn path=/trunk/; revision=13883
2005-03-23 20:25:29 +00:00
Gerald Combs 46b36c7fd5 Add "cleanup_dissection".
svn path=/trunk/; revision=13882
2005-03-23 16:23:30 +00:00
Guy Harris d53df176c1 Add a "cleanup_dissection()" routine, intended to free up data
structures allocated by a dissection.  Currently, it's the same as
"init_dissection()", but they should be split with "init_dissection()"
allocating the initial data structures and "cleanup_dissection()"
freeing them and *not* reallocating the initial data structures.

Use "cleanup_dissection()" in "cf_close()" to make it easier to find leaks.

svn path=/trunk/; revision=13881
2005-03-23 12:58:59 +00:00
Anders Broman 68e9137eb1 Let packet-gsm_a.c dissect L3 messages.
svn path=/trunk/; revision=13880
2005-03-23 12:57:09 +00:00
Guy Harris 532dc3b69a Audit for TLV loops that don't check the length to make sure it's large
enough, and fix them - and handle the already-fixed case similarly (note
that the length is bogus, and break out of the loop).

Put object header items into the protocol tree in the order in which
they appear in the packet.

For unknown subobjects, make the "Data (N bytes)" item cover only the
data, not the header (which is already covered).

Fix the offset in some calls.

svn path=/trunk/; revision=13879
2005-03-23 09:47:11 +00:00
Ulf Lamping 42301df7bf buildbot reported bug fixed: prevent an endless loop
But what's the right way to do instead? Just inserted a return, this should do no harm.

Could someone with more knowledge of this dissector review this change?

svn path=/trunk/; revision=13878
2005-03-23 09:08:36 +00:00
Ulf Lamping 98b3ae923f hf_oampdu_info_state_parser seems to be an uint8 not bool, as it can contain 4 values
svn path=/trunk/; revision=13877
2005-03-23 08:38:47 +00:00
Guy Harris 189c2652f7 "tvb_get_stringz()" and "abs_time_to_str()" don't return null pointers,
so don't check whether they've done so with "DISSECTOR_ASSERT()". 
("proto_item_set_string()" and friends already do so.)

Fix indentation.

svn path=/trunk/; revision=13876
2005-03-23 03:51:38 +00:00
Guy Harris 5a6cf59062 Thou shalt not g_assert() in a dissector.
svn path=/trunk/; revision=13875
2005-03-23 02:48:47 +00:00
Guy Harris d4d1206ad5 Remind people not to use "g_ntoh[ls]()" or "g_hton[ls]()" to convert
big-endian to little-endian - that doesn't work on big-endian machines.

svn path=/trunk/; revision=13874
2005-03-23 02:39:21 +00:00
Guy Harris 596a4f865b It doesn't matter whether, when building with WinPcap, we have
pcap_lib_version() in the WinPcap on the system on which we're building.

svn path=/trunk/; revision=13873
2005-03-23 01:55:11 +00:00
Guy Harris 46e1907f13 If we have "pcap_freecode()", use it to free the instructions for a BPF
filter after installing the filter.

Set HAVE_PCAP_LIB_VERSION if we're building with WinPcap 3.1; it's not
present in earlier versions, but is present in current 3.1 betas.

Check HAVE_PCAP_LIB_VERSION when building capture-wpcap.c.

svn path=/trunk/; revision=13872
2005-03-23 01:25:01 +00:00
Ulf Lamping 79daa1e80e use DISSECTOR_ASSERT instead of g_assert
svn path=/trunk/; revision=13871
2005-03-23 00:09:12 +00:00
Ulf Lamping 58e7915cc0 add a DISSECTOR_ASSERT_NOT_REACHED() macro
svn path=/trunk/; revision=13870
2005-03-22 23:59:54 +00:00
Olivier Biot 71d79f31c0 As suggested by Ulf Lamping, prevent Ethereal from bailing out if there's a protocol error, as we now have a DISSECTOR_ASSERT() macro.
svn path=/trunk/; revision=13869
2005-03-22 22:27:14 +00:00
Ulf Lamping b955abc3ed bugfix for buildbot randpkt: don't call proto_tree_add_string with a NULL pointer
TO THE DISSECTOR DEVELOPERS, PLEASE FIX THIS: I could easily found several places where a g_assert is called, depending on the input packet data. NEVER DO THIS, as this crashes Ethereal simply by invalid packet data input!!!

svn path=/trunk/; revision=13868
2005-03-22 21:47:11 +00:00
Ulf Lamping cf7dee822b replace another g_assert by DISSECTOR_ASSERT, so Ethereal won't crash if a dissector triggers that assert
svn path=/trunk/; revision=13867
2005-03-22 21:39:37 +00:00
Anders Broman b042374230 More IE dissection
svn path=/trunk/; revision=13866
2005-03-22 21:33:53 +00:00
Jörg Mayer fcda7c68a0 Remove trailing ';'
svn path=/trunk/; revision=13865
2005-03-22 16:02:09 +00:00
Anders Broman d1e004b8d2 Initial checkin of a camel dissector.
svn path=/trunk/; revision=13864
2005-03-22 15:41:40 +00:00
Anders Broman a17d3c60ba Dissect part of the payload.
svn path=/trunk/; revision=13862
2005-03-22 15:36:33 +00:00
Anders Broman c728f022df Export q850_cause_code_vals
svn path=/trunk/; revision=13861
2005-03-22 15:34:51 +00:00
Anders Broman 9645994141 Export q850_cause_code_vals as its's used by other dissector(s)
svn path=/trunk/; revision=13860
2005-03-22 15:33:50 +00:00
Anders Broman 982f9318bb ERROR seems to be a reserved word change it to SCTP_ERROR to be able to build.
svn path=/trunk/; revision=13859
2005-03-22 15:19:53 +00:00
Michael Tüxen ab64f95039 From Irene Ruengeler: Bugfixes and support of chunk statistics per end point.
Some cleanups of the code.

svn path=/trunk/; revision=13858
2005-03-22 10:25:07 +00:00
Luis Ontanon 29e0601544 Hint in README.tapping about the stats_tree.
svn path=/trunk/; revision=13857
2005-03-21 22:23:17 +00:00
Luis Ontanon 7c87fca629 Something is better than nothing, Sooner is better than later.
svn path=/trunk/; revision=13856
2005-03-21 21:48:10 +00:00
Anders Broman 0108e3ada9 Updates according to R1.0.3 (2005-02-26) and decode some more IE:s
svn path=/trunk/; revision=13855
2005-03-21 21:15:42 +00:00
Ronnie Sahlberg bafebc7b80 from Hannes Gredler
update for q933
(and changes to move variable declarations to the start of a block since normal c compilers can not handle declarations in the middle of a block)



svn path=/trunk/; revision=13854
2005-03-21 20:52:08 +00:00
Ulf Lamping d2afae8211 two bugfixes (reported by buildbot randpkt):
don't use the return value of gmtime, if it's NULL

prevent from an endless loop, if payload_length is 0

svn path=/trunk/; revision=13853
2005-03-21 20:01:35 +00:00
Luis Ontanon 22084517ad Fix the names that were not consistent after the rename.
svn path=/trunk/; revision=13852
2005-03-21 17:51:07 +00:00
Luis Ontanon 3af24ce10a Add a cleanup call-back to the stats_tree.
svn path=/trunk/; revision=13851
2005-03-21 15:40:33 +00:00
Ronnie Sahlberg 9e7a393c1f make ethereal decrypt gss-krb (secure ldap and ms signandseal for dce)
also when linked with mit kerberos




svn path=/trunk/; revision=13850
2005-03-21 10:52:57 +00:00
Ronnie Sahlberg 31f7bd0e60 remove some heimdal dependencies we dont really need
and prepare for gssapi mit support


svn path=/trunk/; revision=13849
2005-03-21 10:07:55 +00:00
Ulf Lamping 58864f0b5a add capture_fr and stats_tree_branch_max_namelen
svn path=/trunk/; revision=13848
2005-03-21 09:56:55 +00:00
Ulf Lamping 0f790bc7cf fix stats_tree defs
svn path=/trunk/; revision=13847
2005-03-21 09:46:39 +00:00
Guy Harris d79eeaaa02 Declare "stats_tree_create_pivot_node()" and
"create_pivot_node_with_parent_name()".

svn path=/trunk/; revision=13846
2005-03-21 04:06:45 +00:00
Guy Harris 839dd9002c Add a capture_fr() routine when capturing on Frame Relay.
svn path=/trunk/; revision=13845
2005-03-21 03:55:50 +00:00
Guy Harris 7431ca3f58 The table containing payload types and payload dissectors are now sparse
tables that are searched rather than dense tables that are indexed, so
there's no reason to tread payload types 130 and 131 specially.

svn path=/trunk/; revision=13844
2005-03-21 02:00:14 +00:00
Luis Ontanon 8f404dccf2 This one is not in the makefiles (it's an example of stats_tree not realy a feature), make sure it works nontheless
svn path=/trunk/; revision=13843
2005-03-21 01:32:26 +00:00
Guy Harris 460ff2d162 From Lars Ruoff:
when formatting numerical values for CSV lists, save the locale
	before formatting, set the locale to the "C" locale, and then
	restore the locale, so that numbers are always formatted with
	"." as the decimal point;

	add to the RTP streams window a Copy button that copies all data
	from the CList to the clipboard as CSV.

svn path=/trunk/; revision=13842
2005-03-21 01:24:19 +00:00
Luis Ontanon 29499bbb66 This one was missing from the last commit
svn path=/trunk/; revision=13841
2005-03-21 01:22:23 +00:00