Age | Commit message (Collapse) | Author | Files | Lines |
|
A (better?) fix for https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8030
See also thread starting at:
http://www.wireshark.org/lists/wireshark-dev/201212/msg00001.html
svn path=/trunk/; revision=46331
|
|
svn path=/trunk/; revision=45977
|
|
Use glib allocator for data_source.
Thread on wireshark-dev: http://www.wireshark.org/lists/wireshark-dev/201210/msg00116.html
svn path=/trunk/; revision=45673
|
|
svn path=/trunk/; revision=45672
|
|
Use pkthdr instead of pseudo_header as argument for dissecting.
svn path=/trunk/; revision=45601
|
|
resolution information between capture files so that we don't leak host
entries from one file to another (e.g. embarassing-host-name.example.com
from file1.pcapng into a name resolution block in file2.pcapng).
host_name_lookup_cleanup and host_name_lookup_init must now be called
after each call to se_free_all. As a result we now end up reading our
various name resolution files much more than we should.
svn path=/trunk/; revision=45511
|
|
svn path=/trunk/; revision=45016
|
|
when calling it with pattern value of 0
dissector_add_uint() is *not only* used by tcp.port/udp.port dissector tables where 0 is not valid port number,
in some dissector tables 0 is valid protocol number, packet type, etc..
Sample dissectors using dissector_add_uint(.pattern = 0):
packet-ansi_637.c:2348: dissector_add_uint("ansi_a.sms", 0, ansi_637_trans_handle);
packet-cip.c:6017: dissector_add_uint("cip.class.iface", 0, cip_class_generic_handle );
packet-lon.c:723: dissector_add_uint("cnip.protocol", 0, lon_handle);
packet-q931.c:3599: dissector_add_uint("lapd.sapi", LAPD_SAPI_Q931 /* 0 */, q931_handle);
svn path=/trunk/; revision=44980
|
|
Legalize formerly improper use of the API when calling it with
pattern value of 0
svn path=/trunk/; revision=44978
|
|
svn path=/trunk/; revision=44874
|
|
svn path=/trunk/; revision=44873
|
|
svn path=/trunk/; revision=44872
|
|
svn path=/trunk/; revision=44871
|
|
Add new parameter 'data' to heur_dissector_t and new_dissector_t, for now it's always NULL
svn path=/trunk/; revision=44860
|
|
registration tables, and use g_free as the value_destroy_func. This saves us
from manually freeing the value when we remove an item, and prevents us from
leaking memory when we accidentally overwrite an existing item.
svn path=/trunk/; revision=44814
|
|
Remove * from gconstpointers, they are already pointer types.
Add modelines to packet.c and clean up indentation a bit.
svn path=/trunk/; revision=44698
|
|
svn path=/trunk/; revision=44697
|
|
svn path=/trunk/; revision=44696
|
|
make Save-As/Displayed/All-Packets save not only the displayed packets but
also any other packets needed (e.g., for reassembly) to fully dissect the
displayed packets.
This works only for the "All packets" case; choosing only the Selected packet,
the Marked packets, or a range of packets would require actually storing which
packets depend on which (too much memory) or going through the packet list many
times (too slow). Also, this behavior is always the case: you can't save the
displayed packets without their dependencies (I don't see why this would be
desirable).
So far this is done for SCTP and things using the reassembly routines (TCP has
been tested).
The Win32 dialog was modified but hasn't been tested yet.
One confusing aspect of the UI is that the Displayed count in the Save-As
dialog does not match the number of displayed packets. (I tried renaming the
button "Displayed + Dependencies" but it looked too big.) The tooltip tries
to explain this and the fact that this works only in the All-Packets case;
suggestions for improvement are welcome.
Implementation details:
Dissectors (or the reassembly code) can list frames which were needed to
build the current frame's tree. If the current frame passes the display
filter then each listed frame is marked as "depended upon" (this takes up the
last free frame_data flag).
When performing a Save-As/Displayed/All-Packets then choose packets which
passed the dfilter _or_ are depended upon.
svn path=/trunk/; revision=41216
|
|
heuristic dissection;
Rename some vars;
Do some minor re-indentation and whitespace changes.
svn path=/trunk/; revision=40601
|
|
svn path=/trunk/; revision=40309
|
|
trough the proto dialouge.
svn path=/trunk/; revision=40215
|
|
crashes due to having no tvbuffs for an epan_dissect_t.
Fixes bug 6663 and its soon-to-be-duplicates.
svn path=/trunk/; revision=40164
|
|
svn path=/trunk/; revision=39764
|
|
has been called.
In the conversation cleanup routine, free the GSlist for any proto_data which
may have been hanging off the (se_allocated) conversation.
svn path=/trunk/; revision=39484
|
|
svn path=/trunk/; revision=38893
|
|
svn path=/trunk/; revision=38881
|
|
Untested fix for bug 6135.
svn path=/trunk/; revision=38410
|
|
in README.devloper. Remove g_gnuc.h since it's no longer needed. Remove
tvbuff_init(), tvbuff_cleanup(), reassemble_init(), and
reassemble_cleanup() since they were only used for older GLib versions
which didn't support GSlices. Assume we always support the "matches"
operator.
svn path=/trunk/; revision=37978
|
|
only abort if WIRESHARK_ABORT_ON_DISSECTOR_BUG is set.
svn path=/trunk/; revision=37510
|
|
svn path=/trunk/; revision=37216
|
|
svn path=/trunk/; revision=36735
|
|
svn path=/trunk/; revision=36412
|
|
keys to have _uint in their names, to match the routines that handle
dissector tables with string keys. (Using _port can confuse people into
thinking they're intended solely for use with TCP/UDP/etc. ports when,
in fact, they work better for things such as Ethernet types, where the
binding of particular values to particular protocols are a lot
stronger.)
svn path=/trunk/; revision=35224
|
|
the data source does not need to be allocated if (!tree).
Rev 30158 took the if (!tree) check out indicating that the check was invalid.
So: (since packet_add_new_data_source() now only calls add_new_data_source()),
remove packet_add_new_data_source().
svn path=/trunk/; revision=34717
|
|
svn path=/trunk/; revision=32723
|
|
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
|
|
svn path=/trunk/; revision=32361
|
|
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=4422
svn path=/trunk/; revision=32360
|
|
svn path=/trunk/; revision=31961
|
|
svn path=/trunk/; revision=30824
|
|
svn path=/trunk/; revision=30527
|
|
svn path=/trunk/; revision=30346
|
|
or isn't visible.
Clean up some indentation.
svn path=/trunk/; revision=30158
|
|
free memory properly on shutdown.
This is an initial step. There's still some work to do.
svn path=/trunk/; revision=29754
|
|
This is an expensive operation because we:
* Disable the TRY_TO_FAKE_THIS_ITEM optimization
* Use GString to store the protocols
We should only do this if the 'hf_frame_protocols' is referenced (unlikely)
svn path=/trunk/; revision=29733
|
|
deprecates add_new_data_source(). This is based on the following observation:
1) The tvb + name (aka. data_source) is only used when the protocol tree is visible
The current implementation of add_new_data_source() doesn't take this into account and simply allocates a data_source regardless. This is what packet_add_new_data_source() tries to rectify.
A couple of dissectors have already been switched over to the new packet_add_new_data_source(). Many are still missing. Help appreciated!
svn path=/trunk/; revision=29427
|
|
svn path=/trunk/; revision=29276
|
|
registrations,
svn path=/trunk/; revision=29275
|
|
initialize everything by hand. Fixes a Valgrind warning.
svn path=/trunk/; revision=29122
|