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

Fix
- warning: unused parameter 'item'.
- warning: format not a string literal and no format arguments.

svn path=/trunk/; revision=47912

Fix example Lua plugin that passes wrong argument in register_menu function

From Hadriel Kaplan, bug 8201 (https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8201)

svn path=/trunk/; revision=47911

Replace proto_tree_add_text with proto_tree_add_item + expert_info.  Not enough to pacify checkAPIs.pl standards, but its a vast improvement.

svn path=/trunk/; revision=47910

Increase use of proto_tree_add_item (mostly replacing proto_tree_add_text)

svn path=/trunk/; revision=47909

Replace proto_tree_add_text with proto_tree_add_item + expert_info

svn path=/trunk/; revision=47908

Look for desktop-file-install (to be used when installing wireshark.desktop).

svn path=/trunk/; revision=47907

Put a trailing semicolon on the MimeType line (to satisfy desktop-file-validate).

svn path=/trunk/; revision=47906

Move pack_flags to better place.

Reduce sizeof(frame_data) from 136 to 128 (AMD64).

svn path=/trunk/; revision=47905

Fix warning: 'rrc_sub_handle' may be used uninitialized in this function

svn path=/trunk/; revision=47904

Fix a misspelled word; Do some minor tweaking of the wording of a few comments.

svn path=/trunk/; revision=47903

SNMP dissector should handle aes256 (better version than r47690)

svn path=/trunk/; revision=47902

Try to fix compilation on Windows.

svn path=/trunk/; revision=47901

Print human-readable statistics by default. Raw values can be printed
using "-M". Based on a suggestion by Hansang Bae.

svn path=/trunk/; revision=47900

Fix spelling/typos found using a list of commonly misspelled words.

The misspellings were mostly in comments but some were
in text strings visible to the user.

svn path=/trunk/; revision=47899

it's ==> its  &  its ==> it's as needed.

svn path=/trunk/; revision=47898

Revert r44559: go back to using g_malloc()'d memory for SCTP reassembly.  That
way we don't keep two copies of the fragments in memory until the file is
closed.

wmem is probably a better alternative to this.

svn path=/trunk/; revision=47897

it's ==> its  &  its ==> it's as needed.

svn path=/trunk/; revision=47896

its ==> it's

svn path=/trunk/; revision=47895

it's ==> its

svn path=/trunk/; revision=47894

it's ==> its & its == > it's  as needed.

svn path=/trunk/; revision=47893

its ==> it's  &  it's ==> its as needed.

svn path=/trunk/; revision=47892

it's ==> its & its ==> it's  as needed.

svn path=/trunk/; revision=47891

No need for 'if (...initialized) {...}' in proto_reg_handoff...()

svn path=/trunk/; revision=47890

it's ==> its  & its ==> it's as needed.

In a few cases:
 Fix spelling;
 Clean up whitespace and formatting style.

svn path=/trunk/; revision=47889

Dissect more feedback options

svn path=/trunk/; revision=47887

From Alex Badea via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8395

Support AES-GCM ipsec encryption using CTR (since libgcrypt doesn't support
GCM). Using CTR instead gets us decryption, but does not verify authentication.

svn path=/trunk/; revision=47886

Fix typos/errors and make Lua API more consistent.

From Hadriel Kaplan, bug 8393 (https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8393)

svn path=/trunk/; revision=47885

Make compare-abs.sh work on CMake builds

svn path=/trunk/; revision=47884

ABI dumps don't have to depend on config.h directly

They depend on libs which need config.h already.

svn path=/trunk/; revision=47883

LUA API improvements

1. Add col_set_fence() equivalent
2. Update __tostring function to actually return column text

From Hadriel Kaplan, bug 8295 (https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8295)

svn path=/trunk/; revision=47882

Fix Coverity CID 986569: Unused pointer value.

svn path=/trunk/; revision=47881

Add a few comments to CMake TODO items

svn path=/trunk/; revision=47880

Add dumpabi targets to CMake builds

Also update automake generated targets to match CMake generated ones

svn path=/trunk/; revision=47879

From Michal Labedzki via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8396

Add Bluetooth HCRP dissector.

From me: rename a preference to be a bit clearer.

svn path=/trunk/; revision=47878

Bugfix Lua pinfo.cols.protocol not holding value in postdissector

From Hadriel Kaplan, bug 6020 (https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=6020)

svn path=/trunk/; revision=47877

Fix https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8397 :
Prevent an out of bounds access when trying to call RRC sub dissectors and fallback to DATA dissector

svn path=/trunk/; revision=47876