code review of parse_s1_W_stats()

   range check for array index
   don't assign the result of pntohs() to a gint16
   range check for the values stored in phdr.(cap)len

svn path=/trunk/; revision=52618

Don't do canaries in blocks of guint32, it appears to cause alignment issues
when running tests on Solaris. Revert back to byte-at-a-time, but do fewer bytes
to avoid a performance hit.

svn path=/trunk/; revision=52617

Check the length *before* accessing the array, and cap the
length-retrieved-from-packet at the size of the array we have.

Fixes https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9270

svn path=/trunk/; revision=52616

Fix a length check in the OID code, we add an extra byte if is_first, not the
other way around. Also add an assert so it blows up if we get it wrong, rather
than leading to uninitialized memory.

Fixes the last errors I can find in
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9268

svn path=/trunk/; revision=52615

Factor out common parts of ABI checks to UseABICheck.cmake

svn path=/trunk/; revision=52614

Fix Coverity CID 1106801: Missing break in switch.

svn path=/trunk/; revision=52613

Make sure we have a valid capture file pointer in win32_export_file().
Fixes bug 9271.

svn path=/trunk/; revision=52612

Fill in the help links. Make sure they do something when they're clicked.

Try to make the task list width match its contents.

svn path=/trunk/; revision=52611

Remove some commented code.

svn path=/trunk/; revision=52610

Fix the bitcoin dissector to be able to dissect recent version of the protocol.  Bug 8844 (https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8844)

Changes:
  - All messages now have a checksum (and not only version and verack).
  - In the version message: user agent added as a string preceded by a varint length.
  - Port in an address is in little endian and not big endian.
  - In the version message the receiving and emitting address where inverted.

From Eric Masson

svn path=/trunk/; revision=52609

Reverting as requested by Jakub, eventhoug I don't think this prticular entry is useful as it's duplicated in the tree and almost certanly never vissible.

svn path=/trunk/; revision=52608

Let Qt lay out and size elements in FollowStreamDialog. Make it
resizable. Get rid of the group box -- the OS X and Windows HIGs
discourage its use and I'm not sure if it fits the GNOME HIG in this
case either. Make the stream contents monospace. Set the cursor to the
beginning of the text when we follow a stream.

Continue the War On Unnecessary Dialogs. Move "Find"ing from a pop-up
window to the main Follow dialog. Wrap back to the beginning as needed.

Add a "Cancel" button. Make sure it and the escape key work as expected.

svn path=/trunk/; revision=52607

Replace XXX with explanation based on discussion with Jakub on -dev.

svn path=/trunk/; revision=52606

Slightly clarify the wmem release note entry.

svn path=/trunk/; revision=52605

 Adding to col info is probably useless as next dissector will overwrite it.

svn path=/trunk/; revision=52604

Try to pptimize heuristics slightly.

svn path=/trunk/; revision=52603

Add dissection of NCM/MBIM data traffic based on data interface class protocol code

svn path=/trunk/; revision=52602

Look for QtWidgets in the right directory.

svn path=/trunk/; revision=52601

PacketList and ProtoTree need access to MainWindow at initialization
time in order to fill in their respective context menus. R52386 broke
that. Fix it and add comments about handling reparenting. Fixes bug
9265.

svn path=/trunk/; revision=52600

do some range checks when reading vwr files
don't assign the output of pntoh24() to a gint16

unfortunately, vwr detection does not work reliably and many pdf files
are recognized as vwr - this commit should prevent wireshark from
crashing when it tries to load the USB 2.0 spec as pdf ;-)

svn path=/trunk/; revision=52599

Make pkgconfig work - sort of:
- fix a typo
- Add the library dirs and not just the libraries.
  This would help if the paths that are now returned
  were actually correct.

svn path=/trunk/; revision=52598

As reported by Didier Gautheron on -dev (wireshark.org/lists/wireshark-dev/201310/msg00159.html) :
Since r49923, TCP flags are no more displayed in info column when tree is NULL

svn path=/trunk/; revision=52597

TLS server key exchange misses sig and hash algorithm for TLS 1.2.  Bug 9208 (https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9208).

1. Fix ECDH server key exchange for TLS 1.2
2. Fix RSA server key exchange for TLS 1.2

From Hauke Mehrtens

svn path=/trunk/; revision=52596

DTLS server key exchange misses sig and hash algorithm for TLS 1.2.  Bug 9208 (https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9208)

1. Fix ECDH server key exchange for DTLS 1.2
2. Fix RSA server key exchange for DTLS 1.2

From Hauke Mehrtens

svn path=/trunk/; revision=52595

DRX: start to show state prior to and after current PDU.

svn path=/trunk/; revision=52594

As requested by Zoltán Lajos Kis, split the openflow dissector int one file per protocol version.

svn path=/trunk/; revision=52593

SMB2 SRT can be very inaccurate in the presence of retransmissions:   Retransmitted responses not only add additional (bogus) transactions but also the latency associated with them. This can greatly inflate the maximum and average SRT stats especially in the case of retransmissions triggered by the expiry of the rexmit timer (RTOs). Only calculating SRT for the first response received accomplishes this goal without requiring the TCP pref "Do not call subdissectors for error packets" to be set.

svn path=/trunk/; revision=52592

whitespace fixes; mostly: remove trailing blanks

svn path=/trunk/; revision=52591

Minor whitespace changes

svn path=/trunk/; revision=52590

Don't clobber PKG_CONFIG_PATH.

svn path=/trunk/; revision=52589

Minor whitespace, formatting & etc changes.

svn path=/trunk/; revision=52588

When printing PDML, don't assume that all values with bitmasks are represented
as guint32 internally - they may be signed and/or 64-bit.

svn path=/trunk/; revision=52587

[Automatic manuf, services and enterprise-numbers update for 2013-10-13]

svn path=/trunk/; revision=52584

Subsume README.binarytrees into wmem doxygen. The README was out of date
anyways, since Michael made key operations non-destructive in r44380.

svn path=/trunk/; revision=52583

openSAFETY: Dissection for second crc as well as plausability check.  Bug 9261 (https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9261).

From Roland Knall

svn path=/trunk/; revision=52582

Set CMake to use Python when building the tap register files.

svn path=/trunk/; revision=52581

Add CMake properties to targets so that they are logically organised when using a Visual Studio solution.
Add CMake properties to group the source files in epan into logical blocks when using a Visual Studio solution.

svn path=/trunk/; revision=52580

Fix dissection of Home Provider CID and fix a typo error

svn path=/trunk/; revision=52579

So a while back Jeff added some code to check that the offset+length passed to
proto_tree_add_item was valid *before* we short-circuited based on a NULL tree.
This was good in that it removed a common source of really-long-loop bugs. It
was less good in that it cost us about 8% in speed when doing a tree-less
dissection, but we decided the tradeoff was worth it.

After Anders' recent mail to -dev about performance, I started thinking about
how to optimize this. It occurred to me that the vast majority of the logic
involved in the check was dealing with the length value - fetching the actual
length if it was a counted string, calculating the length if it was -1, adding
the length to the offset in a way that was free from overflows, etc.

All of this is (theoretically) unnecessary - simply checking the offset without
worrying about the length will still catch the very-long-loops, since it is the
offset that increases in each iteration, not the length.

All that to justify:
- implement tvb_ensure_offset_exists which throws an exception if the offset is
  not within the tvb
- use it instead of all the complicated other logic in the pre-short-circuit
  step of proto_tree_add_item and friends

This gives us back about 3/4 of the performance we lost in the original patch.
We're still ~2% slower than without any check, but this is the best I can think
of right now.

svn path=/trunk/; revision=52578

Order our conditionals to take advantage of the fact that we know tvb->length <=
tvb->reported_length. Small performance win on the hot path, and a bit easier to
read.

svn path=/trunk/; revision=52577

Update the User Guide for the upcoming 1.11.0 release

svn path=/trunk/; revision=52576

Tweak/simplify some tvbuff length check functions slightly. In my benchmarks of
dissecting without tree, they are costly because they now happen for every
proto_tree_add_item call even if tree is NULL.

svn path=/trunk/; revision=52575

Remove some things that I added while not understanding the real
problem/solution.
"Long" since solved by copying the solution from Makefile.nmake.

Small update to the status of cmake builds on windows

svn path=/trunk/; revision=52574

Redo r52569 in a way that doesn't break the GUI. Add a mem_pool member to
tree_data and reference it directly when allocating/freeing tree items. This
lets us keep multiple around when we need them, and still lets us use
wmem_free_all for a major speedup. It also, coincidentally, lets us get rid of
the annoying fi_tmp hack that was needed before, since that element gets swept
up in the free_all with everything else.

Keep one pool cached to avoid creating/destroying a pool for each packet,
another minor performance win.

The various changes in approach seem to balance out pretty much exactly, this
still gives ~11% over pre-52569.

svn path=/trunk/; revision=52573

Revert r52569, it breaks the GUI since that sometimes has multiple trees active
at once.

svn path=/trunk/; revision=52572

If the offset is out of bounds, have tvb_length_remaining() and tvb_reported_length_remaining() return 0 instead of -1.  This should provide a better long-term fix for bugs/problems such as seen in https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9263.  In the long-run, if we like this behavior, we can change the return value to a guint as well.  Let's let it simmer for awhile first ...

svn path=/trunk/; revision=52571

Don't assume that tvb_length_remaining() or tvb_reported_length_remaining() always return a value >= 0.  Part of fix for https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9263

#BACKPORT(1.10,1.8)

svn path=/trunk/; revision=52570

Use wmem_free_all on the tree pool rather than freeing each node, label and
field_info separately. We still have to walk the tree in order to free certain
fvalues, but that's not a big deal. Another ~11% speed-up running "tshark -nVr"
on a large capture.

svn path=/trunk/; revision=52569

Use a wmem block allocator to allocate field_infos, proto_nodes and item_labels.
Even without making use of free_all (which should be possible) this still
results in ~8% speedup running "tshark -nVr" on a large file in my tests.

svn path=/trunk/; revision=52568

Update User Guide for 1.11 preparation

svn path=/trunk/; revision=52567

From Todd Newton via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9262 :
Fix dissection of PTP Management messages

svn path=/trunk/; revision=52566

We can't use proto_tree_add_uint_* for FT_TIME fields, use proto_tree_add_time_*
instead. Fixes another one of the issues that shows up in
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9263

svn path=/trunk/; revision=52565

Based on help and suggestions from Johannes Berg, fix Radiotap Header decode the
right way.

Fixes https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9256 without
reintroducing https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9212

svn path=/trunk/; revision=52564

From RobiOneKenobi via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9229

Add a new strutil function format_text_chr that replaces unprintable characters
with a single passed-in char (eg a '.' or a '-') instead of a C-style escape.
This is useful for displaying binary data that frequently but not always
contains text; otherwise the number of C escape codes makes it unreadable.

svn path=/trunk/; revision=52563

Have pixbuf_to_widget() use a guint8 * to match
gdk_pixbuf_new_from_inline().

svn path=/trunk/; revision=52562

Another Solaris Studio C++ fix.

svn path=/trunk/; revision=52561

Free the actual struct, not the typedef thereof, because the typedef is actually
a typedef of a *pointer* to the struct, not the struct itself, which are
different sizes.

This doesn't show up under valgrind because the length isn't checked in that
case, everything gets subsumed in valgrind's malloc/free hooks.

Fixes https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9264

svn path=/trunk/; revision=52560

Solaris Studio C++ won't let us pass the address of main() to
init_progfile_dir(). Try using QCoreApplication::applicationFilePath()
instead.

svn path=/trunk/; revision=52559

Update to the latest Windows library tags. Add an "install_qt" target.

svn path=/trunk/; revision=52558

QT_DIR → QT5_BASE_DIR. This matches the CMake environment.

svn path=/trunk/; revision=52557

Look for Qt in WIRESHARK_LIB_DIR. Add a conditional check for qmake.

svn path=/trunk/; revision=52556

With Gerald's patch we should no longer require GTK includes for QT-Only builds
(untested).

svn path=/trunk/; revision=52555

One more HAVE_LIBPCAP.

svn path=/trunk/; revision=52554

cf_start_tail() is (now) just a wrapper around cf_open().  Get rid of it and just call cf_open().

svn path=/trunk/; revision=52553

Try to fix Qt compilation when libpcap is disabled.

svn path=/trunk/; revision=52552

Fix some formatting.

svn path=/trunk/; revision=52551

Replace a question in a comment with a better explanation (that answers the question).

svn path=/trunk/; revision=52550

Fix https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8943 :

List Windows Server 2012 (and a couple of Home Server versions) as supported OS.

While at it, remove the list of the various "NT-based" Windows versions from
a couple of other places (this effectively removes statements that we still
support NT4.0 and 2000).

svn path=/trunk/; revision=52549

Build with Qt by default.

svn path=/trunk/; revision=52548

More ui/util.h includes.

svn path=/trunk/; revision=52545

Include ui/util.h.

svn path=/trunk/; revision=52544

There's no need to include prefs.h twice.

svn path=/trunk/; revision=52543

Remove GTK+ and global cfile dependencies from file_dlg_win32.c.

Move the declaration of set_last_open_dir() to ui/util.h. It still
has separate GTK+ and Qt implemenations. We might want to move it to
ui/util.c at some point. Remove a lot of unnecessary GTK+ includes.
Remove most of the references to the global cfile while we're at it.

svn path=/trunk/; revision=52542

The env variable is QT5_BASE_DIR, as documented in README.cmake
Make documentation and implementatoin consistent

svn path=/trunk/; revision=52541

The path for the Qt5 installation now needs to be set as a
windows environment variable.

svn path=/trunk/; revision=52540

qtshark and wireshark(gtk2) now actually find their libs.

svn path=/trunk/; revision=52539

Now the GUI programs (wireshark/gtk2 and qtshark) compile and link

svn path=/trunk/; revision=52538

Fix another fuzz failure

svn path=/trunk/; revision=52537

We don't need to check for lauxlib.h.

svn path=/trunk/; revision=52536

Make airpcap detection work

svn path=/trunk/; revision=52535

Check for sa_len in struct sockaddr.

svn path=/trunk/; revision=52534

We don't check for lauxlib.h or lualib.h, and we don't need to - we just
check whether we have Lua or not.  Get rid of the cmakedefines for those
two headers.  (The autoconf script should perhaps do the check
differently.)

svn path=/trunk/; revision=52533

Fix check for inflatePrime in zlib (ZLIB_LIBRARIES isn't set at that
point, you want ZLIB_LIBRARY).

svn path=/trunk/; revision=52532

Destroy dissector tables on shutdown and use epan scoped memory of handles.
Cleans up another 600KB of "still accessible" memory.

svn path=/trunk/; revision=52531

"install-generated-files" depends on "doc". Set dependencies
accordingly.

svn path=/trunk/; revision=52530

qtshark.exe doesn't depend on tshark_OBJECTS.

svn path=/trunk/; revision=52529

Free all the heuristict dissector lists and their entries on shutdown, another
few KB of "still reachable" data down.

svn path=/trunk/; revision=52528

Fix compilation when we don't HAVE_LIBGCRYPT

svn path=/trunk/; revision=52527

Don't assume people have subversion--I just copied my home directory to a new
machine and it would have been nice if this script and installed subversion
for me.

Install perl-podlators too (for pod2man).  Need to check if that's the package
name to use on SuSE too...

svn path=/trunk/; revision=52526

Don't return an error if cleaning the Qt directory fails.

svn path=/trunk/; revision=52525

Note that dladdr() isn't being found on OS X, even though it's
available.

svn path=/trunk/; revision=52524

When TCP options are truncated, we only add the truncated length to the tree,
not the whole length. Fixes an invalid access with the capture from
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9263

svn path=/trunk/; revision=52523

Move the QT_MACEXTRAS_LIB cmakedefine in cmakeconfig.h.in to the
equivalent location in config.h.in, to simplify comparing the two files.

Add some tests from the autoconf script to CMake:

check for setresgid() and setresuid();

check for struct stat having an st_flags member.

Get rid of INTTYPES_H_DEFINES_FORMATS; we don't appear to check for it,
and we don't use it (we're using GLib's formatting functions, which have
their own way of handling 64-bit integers).

Get rid of STDC_HEADERS; it's in autoconf because autoconf was
originally developed back when you couldn't rely in ANSI C and has never
been removed, and something we're using for other purposes checks for
it, not because we explicitly test for it, but we don't test it in the
code, and CMake doesn't even check for it.

svn path=/trunk/; revision=52522

Use the right package name.

svn path=/trunk/; revision=52521

Try to discover Qt similar to the way we discover Python.

svn path=/trunk/; revision=52520

We use g_ascii_strtoll(), not strtoll(), so we no longer need to check
for strtoll() (if we ever did check for it).

svn path=/trunk/; revision=52519

Try to locate QtMacExtras.

svn path=/trunk/; revision=52518

Fix other HAVE_LUA entry.

svn path=/trunk/; revision=52517

Fix a few things - untested until current build finishes.

svn path=/trunk/; revision=52516

Don't check for pcap_version(); we don't use it.

Sort the #cmakedefines in cmakeconfig.h.in to more closely match the
order in config.h.in, to make it easier to make sure CMake and autofoo
are doing the same checks.  Remove some #cmakedefines that don't
correspond to check that are being done either by CMake or autofoo.

svn path=/trunk/; revision=52515