Make Solaris Studio hide internal shared library symbols by default

svn path=/trunk/; revision=48020

Export some wmem functions using the new scheme.

svn path=/trunk/; revision=48019

Don't include wmem.h in packet_info.h, just use the struct name directly.
Otherwise wmem tweaks require rebuilding the entire tree for no particular
reason.

svn path=/trunk/; revision=48018

Don't pass -v straight to valgrind, it's rarely useful. Just increasing
num-callers is usually enough.

svn path=/trunk/; revision=48017

Convert at least enough proto_tree_add_text to proto_tree_add_item/expert_info to passify checkAPIs.pl.

Removed check_col calls.

svn path=/trunk/; revision=48016

Move the setting of pt to where it was before the changes, so it's
before the point at which it's added to the protocol tree.

Put the ATM channel in the ATM tree rather than the top-level tree.

Fix the name for the CPI field (copy-and-pasteo).

svn path=/trunk/; revision=48015

Rearrang the code to align with sqlite lemon and copy over a few more things.

svn path=/trunk/; revision=48014

From beroset:
remove C++ incompatibilities

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

svn path=/trunk/; revision=48013

Minor tweaks to "ColorSelection" UI:
 - For each choice (marked/ignored fg/bg) always show orig color as "prev color";
 - Improve spacing between widgets.

Also;
 - Add a few explanatory comments;
 - Add editor modelines;
 - Cleanup whitespace/long-lines/formatting.

svn path=/trunk/; revision=48012

Define a new exception for reassembly errors, and throw it in several cases
instead of using DISSECTOR_ASSERT. When a dissector passes bad data to the
reassembly machine, that isn't necessarily the dissector's fault - the data may
come straight from the packet, and the dissector may not have enough information
to know it's bad without telling the reassembly machine in the first place.

Also fix a bug in the reassembly machine. If it were given a fragment and all of
the following conditions were met:
- the other associated fragments were already marked as done (reassembled)
- the fragment went beyond the end of the conceptual reassembled buffer
- the dissector had not set the PARTIAL_REASSEMBLY flag
then the reassembly machine would incorrectly think there was an overlap and
run past the end of the already-reassembled buffer.

Should fix the rest of
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8380

#BACKPORT
This is probably too big and intrusive to backport directly, and parts of it
will need adapting anyways since reassemble.c has changed. But the bug exists
and crashes in 1.6 and 1.8, so we'll have to do something.

svn path=/trunk/; revision=48011

Pass the same offset to tvb_ensure_bytes_exist and fragment_add.

One minor part of https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8380

svn path=/trunk/; revision=48010

General cleanup including:
1. Convert proto_tree_add_text to proto_tree_add_item/expert info
2. Change to "new style" dissector
3. Use standard malformed packet interface

Reviewed by Francesco Fondelli

svn path=/trunk/; revision=48009

Use explicit casts.

svn path=/trunk/; revision=48008

Fix a copy/paste error and factorize dissection of extension header length and next extension header

svn path=/trunk/; revision=48007

From beroset:
remove C++ incompatibilities

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

svn path=/trunk/; revision=48006

Wite space changes.

svn path=/trunk/; revision=48005

From Lakshmi Narayana Madala :
GTP dissector enhanced to support UDP Port extension header.
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8418

svn path=/trunk/; revision=48004

lemon.c:3176:8: error: attempt to free a non-heap object ‘buf’

svn path=/trunk/; revision=48003

Becport more code.

svn path=/trunk/; revision=48002

Missed one place.

svn path=/trunk/; revision=48001

lemon.c:78:14: note: expected ‘int (*)(const void *, const void *)’ but argument is of type ‘int (*)(const char *, const char *)’

svn path=/trunk/; revision=48000

From beroset:
remove C++ incompatibilities from frame_data_sequence.c

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

svn path=/trunk/; revision=47999

More backports from sqlite lemon to try to address warnings.

svn path=/trunk/; revision=47998

lemon.c:4628:5: error: conflicting types for ‘Configcmp’
lemon.c:366:5: note: previous declaration of ‘Configcmp’ was here

svn path=/trunk/; revision=47997

More backports from sqlite lemon to try to address warnings.

svn path=/trunk/; revision=47996

Move expert_group_vals expert_severity_vals constant definitions to expert.h

svn path=/trunk/; revision=47995

Remove asn1/dsp from autotools Makefile system since it lacks distclean target

svn path=/trunk/; revision=47994

Backports from sqlite lemon.

svn path=/trunk/; revision=47993

Export libwireshark symbols using WS_DLL_PUBLIC define

Also remove old WS_VAR_IMPORT define and related Makefile magic
everywhere in the project.

svn path=/trunk/; revision=47992

Fix a cut-n-pasteo and add a const cast to quiet a warning.

svn path=/trunk/; revision=47991

From Steve Magnani via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8338

Centralize logic related to per-interface conversations, and expose it for use
by class-specific dissectors.

Class-specific descriptor dissectors also need to know the interface in whose
context they are called to work.

This is a prerequisite for a USB Video Class dissector, which needs to decode
many class-specific descriptors.

svn path=/trunk/; revision=47990

Backport const casts from sqlite lemon.

svn path=/trunk/; revision=47989

From beroset:
remove C++ incompatibilities from packet-xml.c

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

svn path=/trunk/; revision=47988

try to remove C++ incompatibilities from lemon

svn path=/trunk/; revision=47987

From Ed Beroset via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8416 :
Remove C++ incompatibilities from packet-asn1.c

svn path=/trunk/; revision=47984

Revert r47839, it is not needed with r47979.

svn path=/trunk/; revision=47980

Add a much better workaround for bug #8382 and some expert info.
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8382

svn path=/trunk/; revision=47979

From Ed Beroset via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8416 :
Remove C++ incompatibilities from WiMAX plugins

svn path=/trunk/; revision=47978

From Ed Beroset via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8416 :
Remove C++ incompatibilities from packet-smb.C

svn path=/trunk/; revision=47977

Fix [-W shadow] issue in SVN #47958 in a slightly different manner than the fix  SVN #47960.

svn path=/trunk/; revision=47976

Whitespace changes: Convert spaces to tabs.  Add modelines.

svn path=/trunk/; revision=47975

Addendum to r47962, caught by Jakub Zawadzki via
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8379

Respect the offset given us by TCP when fetching PDU len.

svn path=/trunk/; revision=47974

Fix Coverity CID 280396: Logically dead code.

svn path=/trunk/; revision=47973

Fix Coverity CID 280371: Logically dead code.

svn path=/trunk/; revision=47972

Fix  warning: C++ style comments are not allowed in ISO C90 [enabled by default]

svn path=/trunk/; revision=47971

Fix Coverity CID 280369: Logically dead code.

svn path=/trunk/; revision=47970

Convert spaces to tabs in capture_step_snapshot().
(Maybe this will make the XP buildbot happy so test.sh succeeds again?)

svn path=/trunk/; revision=47969

From beroset:
remove C++ incompatibilities from packet-pw-atm.c

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

svn path=/trunk/; revision=47968

Fix Coverity CID 280367: Logically dead code.

svn path=/trunk/; revision=47967

Fix Coverity CID 280365: Logically dead code.
#BACKPORT (1.8, 1.6)

svn path=/trunk/; revision=47966

Fix Coverity CID 280362: Logically dead code.
#BACKPORT (1.8, 1.6)

svn path=/trunk/; revision=47965

Fix Coverity CID 280391: Logically dead code.

svn path=/trunk/; revision=47964

Fix Coverity CID 988967: Logically dead code.

svn path=/trunk/; revision=47963

From Sebastiano Di Paola via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8379

New dissector for the honeypot-feeds protocol.

From me: Misc. tweaks to expert info layout and remove a few unneeded initializers.

svn path=/trunk/; revision=47962

Add some 'const's to satisfy latest GCC.

svn path=/trunk/; revision=47961

Rename a variable s/color_edit_dlg/color_edit_window/ to fix a -Wshadow error

svn path=/trunk/; revision=47960

Re-add statement accidentally removed in r47950 that was resulting in a
variable being used uninitialized.

svn path=/trunk/; revision=47959

color_edit_dlg update:
- Significantly simplify & rework code;
- Use GtkColorChooser instead of GtkColorSelection if GTK 3.4 or newer.
- Minor UI cleanup (default buttons & etc)
- Fix crash which occurred in the (existing) GTK3 related code.
- Fix some other relatively minor bugs (including some minor memory leaks);
- Remove some old, commented-out code;
- Use a more consistent naming scheme for certain variables.

svn path=/trunk/; revision=47958

Make it so that alternate-prefix RPM builds actually work.

Use the prefix from 'configure' in the RPM (so: to build an RPM which installs
in /opt do "./configure --prefix=/opt && make rpm-package").

(Maybe this approach should also be used for the other options in the .spec
file.)

Only clean up if building the RPM was successful.

svn path=/trunk/; revision=47957

Replace proto_tree_add_text with proto_tree_add_item/expert_info

svn path=/trunk/; revision=47956

Fix dissection of feedback with large CID

svn path=/trunk/; revision=47955

Revert "Fix MSVC build errors related to symbol visibility"

This reverts commit r47952.
The problem was different from what I originally thought, thus
the commit was obsolete.

svn path=/trunk/; revision=47954

Add -DWS_BUILD_DLL to wiretap Windows builds

It fixes symbol exporting from DLLs.

svn path=/trunk/; revision=47953

Fix MSVC build errors related to symbol visibility

svn path=/trunk/; revision=47952

Add SVN keywords

svn path=/trunk/; revision=47951

Use proto_tree_add_item (or similar) instead of proto_item_add_text.

svn path=/trunk/; revision=47950

Convert proto_item_add_text to proto_item_add_item.
General whitespace cleanup, add modelines.

svn path=/trunk/; revision=47949

Export libwiretap symbols using WS_DLL_PUBLIC define

TODO: hide flex-generated functions

svn path=/trunk/; revision=47948

Update a comment.

svn path=/trunk/; revision=47947

Dissect japan ISUP Charging Info IE.

svn path=/trunk/; revision=47946

Avoid a packet rescan if we don't have an AirPcap adapter.

svn path=/trunk/; revision=47945

Be a little quieter (and do a little more cleanup) when building RPMs.

svn path=/trunk/; revision=47944

Add a #define for the default capture buffer size and use it.

svn path=/trunk/; revision=47942

Add ws_symbol_export.h WIRESHARK_COMMON_INCLUDES to fix failing distcheck

svn path=/trunk/; revision=47941

From Anders: revert r47934: it doesn't work everywhere.
See email discussion on -dev.

svn path=/trunk/; revision=47940

From Anders: more places to increase the default capture buffer size.

svn path=/trunk/; revision=47939

Export libwsutil symbols using WS_DLL_PUBLIC define

This change replaces *.def and *.sym file usage following the
guideline at http://gcc.gnu.org/wiki/Visibility

svn path=/trunk/; revision=47938

Default capture buffer size is now 2MB.

svn path=/trunk/; revision=47937

From Hadriel Kaplan via https://wireshark.org/lists/wireshark-dev/201302/msg00216.html :
Fix dissection of method name in case of RAck header

svn path=/trunk/; revision=47936

Update the text of dumpcap -h, default buffer size should be 2 MB.

svn path=/trunk/; revision=47935

str_util.c: Although the glib documentation doesn't explicitly say so,
it looks like the thousands grouping (') modifier is supported so use it
in format_size.

capinfos.c: Set our locale.

svn path=/trunk/; revision=47934

Combine all the flags fields into one field, and get rid of the
commented-out/#if-ed-out "Data1" field, as it's the same field.

Give the flags in that field names including "flags".

Shuffle the fields around to put "Flags" (a/k/a Data1) where Data1 was,
and put the bits in that field after it.

Update the URL for the spec at IBM.

Reformat the entries in hf_netb[] to use the same style.

svn path=/trunk/; revision=47933

Fix Coverity CID 719419: Dereference after null check.

svn path=/trunk/; revision=47932

Fix Coverity CID 280377: Dead default in switch.

svn path=/trunk/; revision=47931

Fix Coverity CID 280366: Dead default in switch.

svn path=/trunk/; revision=47930

The LDAP dissector uses register_frame_end_routine(), so it needs to
include packet-frame.h.

svn path=/trunk/; revision=47928

Fix https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8378 :
A detailed diagnostic information contains either a SMS-SUBMIT-REPORT or a SMS-DELIVERY-REPORT. Set P2P direction accordingly.

svn path=/trunk/; revision=47927

Include <epan/show_exception.h>, as show_exception() is called.

svn path=/trunk/; revision=47926

Get rid of unnecessary includes of "packet-frame.h".

svn path=/trunk/; revision=47925

Move show_exception() and show_reported_bounds_error() to
epan/show_exception.c, as it's used outside
epan/dissectors/packet-frame.c.  Update their callers to include
<epan/show_exception.h> to get their declaration.

Add a CATCH_NONFATAL_ERRORS macro that catches all exceptions that, if
there's more stuff in the packet to dissect after the dissector call
that threw the exception, doesn't mean you shouldn't go ahead and
dissect that stuff.  Use it in all those cases, including ones where
BoundsError was inappropriately being caught (you want those passed up
to the top level, so that the packet is reported as having been cut
short in the capture process).

Add a CATCH_BOUNDS_ERRORS macro that catches all exceptions that
correspond to running past the end of the data for a tvbuff; use it
rather than explicitly catching those exceptions individually, and
rather than just catching all exceptions (the only place that
DissectorError should be caught, for example, is at the top level, so
dissector bugs show up in the protocol tree).

Don't catch and then immediately rethrow exceptions without doing
anything else; just let the exceptions go up to the final catcher.

Use show_exception() to report non-fatal errors, rather than doing it
yourself.

If a dissector is called from Lua, catch all non-fatal errors and use
show_exception() to report them rather than catching only
ReportedBoundsError and adding a proto_malformed item.

Don't catch exceptions when constructing a trailer tvbuff in
packet-ieee8023.c - just construct it after the payload has been
dissected, and let whatever exceptions that throws be handled at the top
level.

Avoid some TRY/CATCH/ENDTRY cases by using checks such as
tvb_bytes_exist() before even looking in the tvbuff.

svn path=/trunk/; revision=47924

For subvectors, make a top-level item to cover the entire subvector, and
put the length, identifier, and value as visible sub-items underneath
it.  If the length is bad, add an expert info indication under the item
for the length, rather than using proto_malformed.  Add checks for the
lengths of each type of subvector.

Add some more subvector types from IEEE 802.5-1998, and use the names
from that spec.

svn path=/trunk/; revision=47923

When decompressing, put an entry into the protocol tree that covers all
the compressed data, put the entries for particular parts of that data
under that entry, and, when we have a problem decompressing, add an
expert info item to that entry, rather than just adding text or using
proto_malformed.  Do so for *all* decompression problems; in particular,
check for decompressed_entry() returning NULL, which it can do.

tvb_new_child_real_data() won't throw an exception, so don't bother
checking for that.

If, after the first pass, we can't find the uncompressed data for a
frame, report it as a dissector bug.

Check whether we have room for a byte in the uncompressed data buffer
*before* copying it.

Other cleanups.

svn path=/trunk/; revision=47922

From J. Schoenwaelder via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8410 :
Fix dissection of flowDurationMicroseconds in Netflow dissector

svn path=/trunk/; revision=47921

Increase the default buffer size to 2MB.

svn path=/trunk/; revision=47920

Dissect Japan ISUP Charge rate transfer (flexible charging) .

svn path=/trunk/; revision=47919

Use ENC_NA for single byte fields, as that appears to be the generally agreed upon convention.  Add modelines.

#BACKPORT(1.8) + r47915

svn path=/trunk/; revision=47918

Use consistent indentation; Add editor modelines.

svn path=/trunk/; revision=47917

Add a %prefix option for users who want to (easily) configure Wireshark to
install in a non-standard location.

Assume the desktop-integration stuff goes in /usr (regardless of our prefix).

This (with r47914) fixes RPM generation when someone uses a prefix other than
/usr .

Also: run desktop-file-validate on the wireshark.desktop file (just in case it
wasn't installed with desktop-file-install).

svn path=/trunk/; revision=47916

Protocol is ENC_BIG_ENDIAN, adjust accordingly.

Bug 8407 (https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8407)

svn path=/trunk/; revision=47915

Don't install the desktop-integration files in our (Wireshark's) data
directory (which is based on our installation prefix): there's no guarantee
that the desktop also uses the same prefix.

Add a note about how we might be able to find the desktop's data directory
should anyone actually run GNOME with a prefix other than /usr .

Also use 'desktop-file-install' to install the desktop file (if configure
found that command).

svn path=/trunk/; revision=47914