Create a new frame_data_sequence data type; it represents a dense
sequence of frame_data structures, indexed by the frame number.  Extract
the relevant bits of the capture_file data structure and move them to
the frame_data_sequence, and move the relevant code from cfile.c and
tweak it to handle frame_data_sequence structures.

Have a possibly-null pointer to a frame_data_sequence structure in the
capture_file structure; if it's null, we aren't keeping a sequence of
frame_data structures (we don't keep that sequence when we're doing
one-pass processing in TShark).

Nothing in libwireshark should care about a capture_file structure; get
rid of some unnecessary includes of cfile.h.

svn path=/trunk/; revision=36881

Another hack to handle the one-pass case.

svn path=/trunk/; revision=36880

Short-term hack to fix bug 5855 - don't try to process all the
frame_data structures for all the packets if we don't actually *have*
any frame_data structures for any packets, e.g. in TShark in one-pass
mode.

Also:

Use #if 0/#endif instead of commenting out.

Consistently use 4-space indentation.

svn path=/trunk/; revision=36879

If HAVE_LIBGNUTLS isn't defined, don't build functions that are used
only if HAVE_LIBGNUTLS is defined.

svn path=/trunk/; revision=36878

If HAVE_LIBGNUTLS isn't defined, don't build functions that are used
only if HAVE_LIBGNUTLS is defined.

svn path=/trunk/; revision=36877

From Adam Langley via bug 4349:

This patch adds support for getting the pre-master secret of a TLS
connection from a log file. Currently Wireshark can decrypt and TLS
connection only if it has the server's private key.

I commonly have a use case where I control the TLS client, but not the
server.  In order to decrypt in this case, I've added support to NSS
(used by Chrome and Firefox) to log the keys to a file on disk:

https://bugzilla.mozilla.org/show_bug.cgi?id=536474

Given this file, Wireshark can then decrypt the resulting TLS connections.

The format is such that Wireshark opens and linearly scans the file each
time it sees a ClientKeyExchange. If the key log grows too large, this
is pretty inefficient. However, it's simple and the number of
interesting TLS connections when debugging is usually very small.

svn path=/trunk/; revision=36876

From Ivan Sy via bug 3343:

- Support for DTLS and SSL RSA keys list using User Accessible Table
- Support for IPv6 SSL as posted by bug#3343 comment#1
- 'any' and 'anyipv4' for IPv4 wildcard
- 'anyipv6' for IPv6 wildcard
- UAT fields validation.

From me:

- Update paramaters to match UAT API changes.
- Change the UAT filename.
- Fix buffer overflow for IPv6 addresses.
- Allow the use of hostnames along with numeric addresses.
- Don't convert strings to addresses twice.
- Don't use the same variable name for different data types.
- Make "any" mean "any IPv4 or any IPv6".
- Bend the concept of obsolete preferences slightly so that we can convert
  and old-style key list to a UAT.
- Clean up whitespace.
- Don't point to a User's Guide section for now; it may make more sense to
  keep using the wiki page.

SSL dissector changes have been tested. DTLS dissector changes have not.

svn path=/trunk/; revision=36875

From Yaniv Kaul:
Fix 'set but not used' errors that GCC 4.6 emits.

svn path=/trunk/; revision=36874

get_host_ipaddr() was returning numeric addresses in host byte order
and hostnames in network byte order. Have it return everything in
network byte order.

svn path=/trunk/; revision=36873

Fix Dead Store (Dead assignement/Dead increment) Warning found by Clang

svn path=/trunk/; revision=36872

Fix Dead Store (Dead assignement/Dead increment) Warning found by Clang

svn path=/trunk/; revision=36871

Fix Dead Store (Dead assignement/Dead increment) Warning found by Clang

svn path=/trunk/; revision=36870

Fix Dead Store (Dead assignement/Dead increment) Warning found by Clang

svn path=/trunk/; revision=36869

Fix Dead Store (Dead assignement/Dead increment) Warning found by Clang

svn path=/trunk/; revision=36868

- Synchronize gtk/Makefile.am checkapi with gtk/Makefile.nmake
- fix: Found deprecated-gtk APIs in packet_win.c: gtk_entry_new_with_max_length

svn path=/trunk/; revision=36867

Add new function packet_hex_editor_print.

svn path=/trunk/; revision=36866

Packet editing for single field_info:
- Support FT_STRING and FT_STRINGZ types.
- hex editor (mainly for unsupported types)

svn path=/trunk/; revision=36865

Cast away warnings about "struct sockaddr" -> "struct sockaddr_XXX"
casts that are actually safe.

svn path=/trunk/; revision=36864

Get rid of a cast that was causing alignment warnings and that was bogus
anyway (the second argument to g_slist_find_custom is a gconstpointer,
not a gpointer * or a gconstpointer *).

svn path=/trunk/; revision=36863

Get rid of some code that's no longer relevant with the new tree
structure for frame_data items.

svn path=/trunk/; revision=36862

Cast away warnings about "struct sockaddr" -> "struct sockaddr_XXX"
casts that are actually safe.

svn path=/trunk/; revision=36861

Remove const qualifier to name, abbrev and blurb to pacify the buildbots.

svn path=/trunk/; revision=36860

Plug a memory leak.

svn path=/trunk/; revision=36859

Take out extra pair of parenthesis

svn path=/trunk/; revision=36858

Make checkapi happy

svn path=/trunk/; revision=36857

Get rid of some unnecessary casts that just provoke uninteresting
warnings.

More fun with GArrays: cast away some warnings that don't report real
alignment problems and that wouldn't even happen if the "data" member of
a GArray were a "void *".

svn path=/trunk/; revision=36856

More fun with GArrays: cast away some warnings that don't report real
alignment problems and that wouldn't even happen if the "data" member of
a GArray were a "void *".

svn path=/trunk/; revision=36855

More fun with GArrays: cast away some warnings that don't report real
alignment problems and that wouldn't even happen if the "data" member of
a GArray were a "void *".

Clean up indentation.

svn path=/trunk/; revision=36854

Get rid of some extra parentheses that cause clang to warn that you
might have meant to do an assignment rather than a comparison.

svn path=/trunk/; revision=36853

Fix compilation warning:
packet_win.c:230: warning: 'u_val' may be used uninitialized in this function

svn path=/trunk/; revision=36852

from Jakub Zawadzki:
Don't thaw the packet list during file loading to speed up file loading.

svn path=/trunk/; revision=36851

- Experimental support to edit single field_info.

Note: For now only integers (without bitmask), I'm afraid other fields require more work.

svn path=/trunk/; revision=36850

Store the frame_data structures in a tree, rather than a linked list.
This lets us get rid of the per-frame_data-structure prev and next
pointers, saving memory (at least according to Activity Monitor's report
of the virtual address space size on my Snow Leopard machine, it's a
noticeable saving), and lets us look up frame_data structures by frame
number in O(log2(number of frames)) time rather than O(number of frames)
time.  It seems to take more CPU time when reading in the file, but
seems to go from "finished reading in all the packets" to "displaying
the packets" faster and seems to free up the frame_data structures
faster when closing the file.

It *is* doing more copying, currently, as we now don't allocate the
frame_data structure until after the packet has passed the read filter,
so that might account for the additional CPU time.

(Oh, and, for what it's worth, on an LP64 platform, a frame_data
structure is exactly 128 bytes long.  However, there's more stuff to
remove, so the power-of-2 size is not guaranteed to remain, and it's not
a power-of-2 size on an ILP32 platform.)

It also means we don't need GLib 2.10 or later for the two-pass mode in
TShark.

It also means some code in the TCP dissector that was checking
pinfo->fd->next to see if it's NULL, in order to see if this is the last
packet in the file, no longer works, but that wasn't guaranteed to work
anyway:

we might be doing a one-pass read through the capture in TShark;

we might be dissecting the frame while we're reading in the
packets for the first time in Wireshark;

we might be doing a live capture in Wireshark;

in which case packets might be prematurely considered "the last packet".
#if 0 the no-longer-working tests, pending figuring out a better way of
doing it.

svn path=/trunk/; revision=36849

Cast away some clang errors about cast alignments that appear safe.

svn path=/trunk/; revision=36848

Fix clang error ("increases requird alignment from 1 to 8") by removing
unnecessary VALS() cast

svn path=/trunk/; revision=36847

Make the packet count an unsigned value, as frame numbers are unsigned.

Make the loops that scan through all the packets do so by frame number,
to abstract away the "next" and "previous" pointers in the frame_data
structure.  Add a routine to cfile.c to map frame numbers to frame_data
structures, and put in some special case handling so scanning forward or
backward through the packets is O(N) rather than O(N^2).

svn path=/trunk/; revision=36846

From Loránd Jakab via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5854 :

The Locator/ID Separation Protocol [1] is being standardized within the IETF,
and it is nearing RFC status (pending security review). I have been maintaining
a dissector patch for about a year, see [2]. Feedback received indicates that,
among others, it is widely used by the developers of a large router vendor,
without issues.

In January I submitted the dissector for data plane packets as bug #5602, which
was committed as r35615. The patch attached to this bug adds support for
dissection of control plane packets.

[1] http://tools.ietf.org/html/draft-ietf-lisp
[2] http://lisp.ccaba.upc.edu/wireshark/

svn path=/trunk/; revision=36845

From Joakim Wiberg via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5801 :

The attached patch replaces most of the proto_tree_add_text() with
proto_tree_add_item(). That resulted in several new display filters.

It also uses the correct defines (ENC_NA, ENC_BIG_ENDIAN and ENC_LITTLE_ENDIAN)
instead of TRUE/FALSE in all proto_tree_add_item() calls.

Also it prints all out all connected transport data, since this is used by
"IENetP Test Tool" as a part of the XML export.

svn path=/trunk/; revision=36844

Packet editing:
- Show next editing bit (for hex it'd require patching packet_hex_apply_reverse_tag)
- Remove memset()s
- Try to fix compilation for win64.

svn path=/trunk/; revision=36843

Allocate space for epan_dissect_t inside struct.

svn path=/trunk/; revision=36842

Initial support for editing packets inside wireshark.

svn path=/trunk/; revision=36841

Fix CID 813: Code after ERROR is dead code (never reached)

svn path=/trunk/; revision=36840

new_packet_list_find_row_from_data() is always used to select a packet,
so get rid of the select_flag argument, and rename it
new_packet_list_select_row_from_data().

It's also always passed a frame_data *, so make its argument a
frame_data *.

Its return value is used only to detect whether the packet was found in
the display or not, so make it a gboolean.  Check it in *all* cases
where it's called, and change the dialog message a bit (the most likely
cause is that the user cancelled a redissection of the packets, so not
all packets in the capture file are in the display.

Also, in the find case, pass it the new packet we found.

svn path=/trunk/; revision=36839

Make the interface combo box an entry, so you can type in it again.

svn path=/trunk/; revision=36838

gtk_combo_box_new_text() returns a GtkComboBox, and you connect a
function directly to its "changed" signal; GtkComboBox is a subclass of
GtkBin, but you don't connect to its child.

svn path=/trunk/; revision=36837

[Automatic manuf, services and enterprise-numbers update for 2011-04-24]

svn path=/trunk/; revision=36834

Back out previous change - I guess the Solaris buildbot has a pre-2.5.30
flex.

svn path=/trunk/; revision=36833

Try putting the include of config.h at the top of the generated scanner,
so that the config.h definitions are available before we include
anything else; that way, for example, anything defined to enable
large-file support will be defined before we include any system header
files that might depend on it.

svn path=/trunk/; revision=36832

Rename g_gnuc_malloc.h to g_gnuc.h

svn path=/trunk/; revision=36831

Check if changing to another profile in change_configuration_profile()

svn path=/trunk/; revision=36830

Use orginal nsstime_t structures (without copying).

svn path=/trunk/; revision=36829

Don't pass -fexcess-precision=fast to clang like we do for gcc.

svn path=/trunk/; revision=36828

Avoid passing a NULL string pointer to format routines: some libc's (e.g.,
Solaris') will seg-fault on that.

svn path=/trunk/; revision=36827

Fix various warnings found by clang 3.0 (trunk 129935) complier:

- Extraneous parentheses

- Use _U_ for unused function parameters instead of assigning the variable
  to itself

svn path=/trunk/; revision=36826

As suggested by Jakub Zawadzki:

Rename g_gnuc_malloc.h to g_gnuc.h (since it contains non-malloc related
GNUC stuff).

Use G_GNUC_WARN_UNUSED_RESULT from glib instead of using warn_unused_result
directly.

svn path=/trunk/; revision=36825

Fix warning about using wrong enum that is probably causing an actual
bug: whenever call_state is assigned to VOIP_CALL_SETUP in other parts
of Wireshark code (such as gtk/voip_calls.c), then call_active_state
is set to VOIP_ACTIVE.  Do that here too.  It was set to VOIP_NO_STATE
(from the wrong enum), which would have likely resulted in the wrong
value being assigned.

svn path=/trunk/; revision=36824

Fix a "equality comparison with extraneous parentheses" warning found
by clang 3.0 (trunk 129935) compiler.

svn path=/trunk/; revision=36823

Fix a "equality comparison with extraneous parentheses" warning found
by clang 3.0 (trunk 129935) compiler.

svn path=/trunk/; revision=36822

Use _U_ for unused function parameters instead of assigning the variable
to itself.  Found by clang 3.0 (trunk 129935) compiler.

svn path=/trunk/; revision=36821

Fix a few "equality comparison with extraneous parentheses" warnings
found by clang 3.0 (trunk 129935) compiler.

svn path=/trunk/; revision=36820

Use _U_ for unused function parameters instead of assigning the
variable to itself.  Found by clang 3.0 (trunk) compiler.

svn path=/trunk/; revision=36819

Change enum type in struct to an int to allow different enum types to
be used for the field display type in the struct.

svn path=/trunk/; revision=36818

Fix whitespace a bit and add field names from doc/README.developer to
comments such as FIELDNAME, FIELDABBREV, etc.

svn path=/trunk/; revision=36817

Assume the tags are mallocated, so they're properly aligned.  Cast away
the warnings.

svn path=/trunk/; revision=36816

Remove enum cast that doesn't do anything and causes a warning about
converting between enum types in clang 3.0 (trunk).

svn path=/trunk/; revision=36815

Found by clang 3.0 (built from trunk): various implicit conversion from
enumeration type 'xxx' to different enumeration type 'xxx'

Fix: Create different structs based on the enum type contained within.

svn path=/trunk/; revision=36814

Use _U_ for unused function parameters instead of assigning a variable
to itself.  Found by clang compiler.

svn path=/trunk/; revision=36813

If we're using GNUC, use attribute warn_unused_result to verify that the
return value of proto_item_add_subtree() is used.

(The WARN_IF_UNUSED macro doesn't belong here...  But where should it go?)

svn path=/trunk/; revision=36812

Use the return value of proto_item_add_subtree(): put the filter masks in their own subtree (as it appears was intended).

svn path=/trunk/; revision=36811

Don't create subtrees that are not used (and don't create subtrees which might
have--if the original code was different--contained only one sub-item).
Remove some no-longer-used ett variables.

svn path=/trunk/; revision=36810

Don't assume that a proto_item is equivalent to a proto_tree.
Use the return value from proto_item_add_subtree().

svn path=/trunk/; revision=36809

Don't create a subtree that is not used.

svn path=/trunk/; revision=36808

Use the return value of proto_item_add_subtree()

svn path=/trunk/; revision=36807

Don't assume that a proto_item is equivalent to a proto_tree.
Use the return value from proto_item_add_subtree().

svn path=/trunk/; revision=36806

Don't create a subtree that is not used.

svn path=/trunk/; revision=36805

Use the return value of proto_item_add_subtree()

svn path=/trunk/; revision=36804

Enhance BT-UTP dissector:
* Add type (syn, data, reset...) in column info
* Split dissect_utp_header(...) function to
* dissect_utp_header_v0(...)
* dissect_utp_header_v1(...)
* dissect_utp_extensions(...)

svn path=/trunk/; revision=36803

Fix (latest) Dead Store (Dead nested assignment) Warning found by Clang

svn path=/trunk/; revision=36802

From wireshark.org/lists/wireshark-dev/201104/msg00231.html
Always use the returned tree value of proto_tree_add_subtree (and not the item!)

svn path=/trunk/; revision=36801

Cleanup emem_verify_pointer()

svn path=/trunk/; revision=36800

Add some casts to squelch complaints about alignment; our little slab
allocator should return properly-aligned items.

svn path=/trunk/; revision=36799

Remove invalid svn:executable property

svn path=/trunk/; revision=36798

Add return instructions for system without support for G_GNUC_NORETURN (e.g. some older gcc compilers).
Reported by Anders Broman on -dev list.

svn path=/trunk/; revision=36797

We deal with pointer alignment ourselves, so we ensure the casts are
safe.  Tell the compiler so.

svn path=/trunk/; revision=36796

Either the buf argument to emem_scrub_memory() is properly aligned or it
isn't.  If it is, we don't need to worry about alignment, so the XXX
comment doesn't belong there; if it isn't, then we should do what the
comment says.  For now, assume the comment before the XXX comment is
correct, and just cast away the alignment warning.

svn path=/trunk/; revision=36795

ep_alloc() returns a properly-aligned value for all uses; that's why it
returns void * rather than guint8 *.  Don't pointer-convert that
information away; otherwise, compilers might whine about safe pointer
casts.

svn path=/trunk/; revision=36794

Stuff that comes back from ep_alloc() is properly aligned for all uses;
that's why it returns "void *", not "guint8 *".  Don't pointer-convert
that information away; otherwise, compilers might whine at you about
unsafe casts.

svn path=/trunk/; revision=36793

Cast away bogus warnings.  (I begin to see why C++ has templates here,
complicated though they might be for both users and implementors....)

svn path=/trunk/; revision=36792

*Properly* get rid of bogus cast.

svn path=/trunk/; revision=36791

"guint8 *" means "pointer to array of bytes that may or may not be
properly aligned"; "void *", at least when it's the return value of
ep_malloc(), means "pointer to properly-aligned chunk of memory".  Use
"void *" in get_sdp_data_element() and its callers in that case, to
squelch compiler warnings.

svn path=/trunk/; revision=36790

Cast away alignment warnings about pointers where we never actually
directly dereference pointers to the field members.

Get rid of an unnecessary cast that provokes warnings.

svn path=/trunk/; revision=36789

Cast away warnings about "struct sockaddr" -> "struct sockaddr_XXX"
casts that are actually safe.

svn path=/trunk/; revision=36788

Make all the fields in the various structures just arrays of bytes, so
we can cast not-necessarily-aligned pointers to pointers to those
structures without risk of compiler warnings *or* the underlying problem
the compiler's trying to warn us about (no, you can't always dereference
an unaligned pointer - SPARC traps, and at least some ARM processors may
do something other than what you want, for example).

This also caught some cases where we were not even properly
byte-swapping on big-endian platforms.

This also lets us not muck around with splitting 64-bit times into two
32-bit fields - we have pletohll(), after all.

svn path=/trunk/; revision=36787

Remove unneeded #include <errno>,<math>,<ctype> & "isprint";
Do minoir whitespace & indentation cleanup.

svn path=/trunk/; revision=36786

Add #include <ctype.h> before #include "isprint.h";
Remove an unneeded line of code;
Do some indentation & whitespace cleanup.

svn path=/trunk/; revision=36785

Avoid passing a NULL string pointer to format routines: some libc's (e.g.,
Solaris') will seg-fault on that.

svn path=/trunk/; revision=36784

#include <errno.h> & #include <math.h> not req'd;

svn path=/trunk/; revision=36783

Remove unneeded #include <ctype.h> and "isprint.h";
Do minor whitespace cleanup.

svn path=/trunk/; revision=36782

Remove some unneeded #includes;
Correct some typos;
Use consistent indentation & do some whitespace cleanup.

svn path=/trunk/; revision=36781

Remove some uneeded #includes;
Minor indentation & whitespace cleanup.

svn path=/trunk/; revision=36780