Set the progress bar animation behavior to match other parts of the
program.

svn path=/trunk/; revision=52993

Try to make the animation a little more natural.

svn path=/trunk/; revision=52992

Move the geometry shenanigans a bit further ahead in the startup
sequence. Fix some prefs logic.

svn path=/trunk/; revision=52991

Fix maximized geometry behavior on non-OS X systems.

svn path=/trunk/; revision=52990

Fix a comparison.

svn path=/trunk/; revision=52989

Add initial support for geometry settings in the main window.

In the current Qt startup sequence the main window + splash screen is
displayed before the main preferences are loaded which means we can't
immediately apply the correct geometry. For now, use an animation to
morph from the default to the saved geometry after the prefs are loaded.

Get rid of main_do_quit() (Qt) and main_window_exit() (Qt+GTK). It looks
like they were unused.

Add wsApp->isInitialized() and use it to see if we should exit() when
the main window is closed. Otherwise we won't shut the application down
and clean up properly.

svn path=/trunk/; revision=52988

BAT has a Bouquet ID, no Service ID

svn path=/trunk/; revision=52987

add one more private data specifier

svn path=/trunk/; revision=52986

Remove an extraneous semicolon.

svn path=/trunk/; revision=52985

Add some documentation to hopefully help dissector writers avoid the problem
in https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9323 :

The string added to proto_tree_add_string*() must be NULL terminated.

svn path=/trunk/; revision=52984

Initialise current_security to NULL, to avoid possibly using uninitialised

svn path=/trunk/; revision=52983

Get rid of value_strings that appear to be a throwback to when ROHC appeared in this dissector

svn path=/trunk/; revision=52982

Don't call RRC dissector if should be ciphered.  Take care to allow SecurityModeComplete to be shown though (i.e. the next UL frame after security is configured)

svn path=/trunk/; revision=52981

Allow string-based dissector tables to pass data between dissectors.

svn path=/trunk/; revision=52980

Replace some proto_tree_add_string_format with proto_tree_add_item +
proto_tree_set_text - the string was not the important part, the formatting was.
We were passing the string directly from tvb_get_ptr, but this meant that if the
packet didn't contain a null-terminator we would run off the end. Since the
string comes straight from the packet, just let _add_item handle the length
calculations etc efficiently, and set the display later.

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

I'm a bit confused honestly why most of these are being set hidden after being
added and formatted, but at least there are no memory errors anymore.

svn path=/trunk/; revision=52979

Update the ASN.1 generated dissectors that were missed as part of the patches in bug 9344 (https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9344)

svn path=/trunk/; revision=52978

When adding an entry to a dissector string table, take a copy of the pattern
string (and pass g_free to g_hash_table_new_full to free it).

This means callers don't have to worry about the scope of the memory they pass
in, and fixes https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9296

svn path=/trunk/; revision=52977

Set SVN properties.

svn path=/trunk/; revision=52976

Fix proto.c:4498: warning: 'compare_filter_name' defined but not used

svn path=/trunk/; revision=52975

Optimize proto_get_id_by_filter_name() by using the existing hastable for proto_filter_names.

svn path=/trunk/; revision=52972

Mark an unused parameter as such

svn path=/trunk/; revision=52971

if pointer==0, the section starts immediately
don't add an empty fragment to the list

this should fix the asserts reported in
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9325
and
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8352

svn path=/trunk/; revision=52969

separate function for dissecting the adaptation field

svn path=/trunk/; revision=52967

Revert r52905 and r52908: they're causing plenty of fuzz failures.

https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9323 is tracking the fix.

svn path=/trunk/; revision=52966

Remove no-longer-used parameter.

svn path=/trunk/; revision=52965

Pass T.38 data to subdissectors directly instead of using pinfo->private_data.

svn path=/trunk/; revision=52964

Remove use of pinfo->private_data

svn path=/trunk/; revision=52963

Add casts to results of g_hash_table_lookup() calls

svn path=/trunk/; revision=52955

Show security config settings in each PDCP PDU.  Don't call IP dissector for ciphered userplane frames. SRB messages are more complicated - need to keep track of SecurityModeRequest and SecurityModeResponse as they are not ciphered

svn path=/trunk/; revision=52953

Add some items to the list of things still to be done.

svn path=/trunk/; revision=52952

Adjust when dissect_diameter_avp loop is called to avoid access violations.

from Lukasz Markowski

svn path=/trunk/; revision=52951

Whitespace fixes: Replace some tabs by spaces & etc

svn path=/trunk/; revision=52950

From Marko Hrastovec via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9347 :

Asterix dissector had two #if 0 ... #endif blocks because it obviously did not
compile on some systems.

Blocks were created because of mistyped variable names and variable not used
warning/error.

The errors were corrected.

svn path=/trunk/; revision=52949

From didier gautheron: remove redundant or use faster col_xxx functions

- when the text parameter is constant col_add_str() and col_set_str() are equivalent but col_set_str() is faster.
- same for  replace col_append_fstr and col_append_str
- remove col_clear() when it's redundant:
    + before a col_set/col_add if the dissector can't throw an exception.
- replace col_append() after a col_clear() with faster col_add... or col_set

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

svn path=/trunk/; revision=52948

Add a cast to try to fix OSX buildbot

svn path=/trunk/; revision=52947

Pass security command details (integrity and ciphering algorithms) to PDCP (which doesn't yet do much with them)

svn path=/trunk/; revision=52946

Make the packet analysis for SCTP independent from GTK and QT.

svn path=/trunk/; revision=52945

simplify file_error()

svn path=/trunk/; revision=52944

allow err_info==NULL in file_error()

this fixes https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9309
reported by Peter Wu

svn path=/trunk/; revision=52943

From Peter Wu

write editcap debug infos to stderr
print some of those messages only in verbose mode

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

svn path=/trunk/; revision=52942

Revert part of 52896 and (for now) all of 52935. As Jeff pointed out,
the PortableApps version relies on U3_-prefixed environment variables.

svn path=/trunk/; revision=52941

Make it so out-of-tree builds (autofoo and cmake) have access to the various
data files (diameter/*, COPYING, manuf) when running *shark from the
build directory.

Do this by passing in the top-level source directory as a compile-time
definition (unfortunately this has to be in the top-level Makefile too because
some programs link directly with epan/filesystem.c).

The plugins dir is no longer below the datafile directory but rather the
progfile directory (if we have one).  Handle the special case of AUTHORS-SHORT
(a data file but a generated one) by checking the file name before building
the path.

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

svn path=/trunk/; revision=52940

Support building and installing both Qt and GTK+.

When forcing CFLAGS and CXXFLAGS, add -g and -O2; newer versions of
autoconf do so for GCC-compatible compilers (clang is GCC-compatible in
that sense), but only do so when those variables aren't forced.

svn path=/trunk/; revision=52939

Have GTPv2 pass its "instance ID" to "private extension" subdissectors rather than use pinfo->private_data.  There are no "private extension" subdissectors in the Wireshark source, so I assume it's all done by third-party plugins.  They must now update their dissectors to get the "instance ID" from the dissector data parameter.

svn path=/trunk/; revision=52938

Fix scroll bar ranges.

svn path=/trunk/; revision=52937

Add c-ares. Untested, but it *should* work.

svn path=/trunk/; revision=52936

Remove some (but not all) U3 remnants.

svn path=/trunk/; revision=52935

TABs -> spaces
add editor modelines

svn path=/trunk/; revision=52934

Add Tinkerforge to the list of new protocols

svn path=/trunk/; revision=52933

FIX compiler warning:
  packet-gtp.c: In function 'dissect_gtp_common':
  packet-gtp.c:7838: warning: unused variable 'pd_save'

svn path=/trunk/; revision=52932

From Ishraq Ibne Ashraf: Tinkerforge protocol dissector

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

From me:

1. Move certain global vars to local storage in dissect_tfp_common()

2. Declare all remaining global vars as static;

3. Fix some bugs:
    - base58_encode() needed to be called before call to col_add_fstr()
    - display of UID string in tree was being truncated to 4 characters

4. Cleanup whitespace: use consistent indentation (tabs); remove trailing whitespace;

5. Add editor modelines

svn path=/trunk/; revision=52931

Add an extra parameter to gtp_match_response() so pinfo->private_data doesn't need to be used.

svn path=/trunk/; revision=52930

Add some verbosity.

svn path=/trunk/; revision=52929

Fix the compilation.
I guess this is what is meant. Only compile tested...

svn path=/trunk/; revision=52928

Add an if(tree) and tweek col_add_fstr().

svn path=/trunk/; revision=52927

From Peter Wu via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9341 :

SVN rev 50525 tried to fix out-of-source-tree builds, but did so by
writing the .rcc.cpp files to the source tree. This was noticed when
running `make distcheck`. This patch reverts that behavior, and creates
the image/ directory to fix out-of-tree builds.

In other words, fix things so "make distcheck" with Qt works in out-of-tree
builds too.

svn path=/trunk/; revision=52926

make Pointer filterable

svn path=/trunk/; revision=52925

packed_analysis_data_t -> packet_analysis_data_t

svn path=/trunk/; revision=52924

correct typos

svn path=/trunk/; revision=52923

remove unnecessary return; lines

svn path=/trunk/; revision=52922

add tags file to .gitignore

svn path=/trunk/; revision=52921

initialize tree variables

svn path=/trunk/; revision=52920

Pass diam_sub_dis_t directly to the subdissectors instead of using pinfo->private_data.

svn path=/trunk/; revision=52919

Remove accidentally-added private dissectors from Makefile

svn path=/trunk/; revision=52918

Comment changes only

svn path=/trunk/; revision=52917

Pass tcap_private into the dissectors that need it rather than use private_data.  Not sure if value_ptr is the "proper" place to pass tcap_private "internally" within the TCAP dissector, but it gets away from (potential) private_data (ab)use.

svn path=/trunk/; revision=52916

Revert SVN #52914;
OSX-10.6-x64 buildbot still givs an error
(At least we now know which line of code gives the error).

svn path=/trunk/; revision=52915

From Shekhar Chandra: support for version 3.0 of netscaler packet wire format.
  https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9280

(Revised patch: let's see if this version compiles w/o error
                on all the buildbots)

svn path=/trunk/; revision=52914

More size and position tweaks.

svn path=/trunk/; revision=52913

Use _add_item not _add_uint as pointed out by checkAPI

svn path=/trunk/; revision=52912

Fix up some formatting.

svn path=/trunk/; revision=52911

Remove "Cisco MDS-specific" fields from packet_info.  2 of the 3 fields were used strictly for columns that are considered "deprecated" and I think the third could be put in that category as well.

I assume the column enumerations haven't already been removed because of legacy "indexing issues", but if I'm wrong, we should definitely remove the columns altogether.  Could also see renaming columns to DEPRECATED_[X].

svn path=/trunk/; revision=52910

Dissect Kafka "fetch" requests and responses.

svn path=/trunk/; revision=52909

As pointed by Evan: don't leak memory when the string pointer is NULL.

svn path=/trunk/; revision=52908

From Michal Labedzki via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9333 :
NFC: Add PN532 HCI dissector

svn path=/trunk/; revision=52907

Try to improve the appearance of the sequence diagram.

Use integer coordinates for the arrows and text. Add smooth_font_size()
to qt_ui_utils which will hopefully adjust the size on Windows to
something more readable.

svn path=/trunk/; revision=52906

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

Actually handle non-NULL-terminated FT_STRINGs properly.

svn path=/trunk/; revision=52905

Initialise a couple of variables to avoid bogus may-be-used-uninitialised warnings with my version of gcc

svn path=/trunk/; revision=52904

From Michal Labedzki via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9333 :
USB: Decode HID interface subclasses and protocols

svn path=/trunk/; revision=52903

From Michal Labedzki via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9333 :
USB: CDC: Decode ACM, CM and Union Functional Descriptors

svn path=/trunk/; revision=52902

From Michal Labedzki via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9333 :
USB: Decode CDC subclasses and protocols

svn path=/trunk/; revision=52901

Mark unused parameters.

svn path=/trunk/; revision=52900

Remove iplen and iphdrlen from struct _packet_info.
Dissectors should just use (reported) tvb length and taps have other ways to get the data.

svn path=/trunk/; revision=52899

Fix relative path now that u3 folder has been removed

svn path=/trunk/; revision=52898

Do not reset pinfo->curr_layer_num in packet-frame.c
Before this change, curr_layer_num could be off by 1 depending on whether we
generated items for Frame protocol or not, leading to a failure to get the
UDP heuristic dissector identified during first pass

svn path=/trunk/; revision=52897

Remove U3 code and packaging.

svn path=/trunk/; revision=52896

Add Gettext to PACKAGELIST. Homebrew puts it in a nonstandard location.

svn path=/trunk/; revision=52895

Have X.25 dissector pass boolean q_bit_set value through dissector data rather than pinfo->private_data.

Other "related" dissectors weren't manipulating pinfo->private_data, so it doesn't make sense for them to bother saving/restore it (now that q_bit_set isn't being used)

svn path=/trunk/; revision=52894

Allow "circuit dissectors" to pass data between dissectors.  Not sure if this is immediately useful (ie remove need for packet_info (private_)data), but opens the possibility.

svn path=/trunk/; revision=52893

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

svn path=/trunk/; revision=52890

From Zoltán Lajos Kis:

dissect flow_removed and port_status messages

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

svn path=/trunk/; revision=52889

From Zoltán Lajos Kis:

dissect experimenter, get/set config, packet in and packet out messages

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

svn path=/trunk/; revision=52888

Various updates - go into a little depth now.

svn path=/trunk/; revision=52887

Treat IE 173 (symbol proprietary) like we treat the vendor IE

svn path=/trunk/; revision=52886

Have ANSI TCAP dissector pass data through subdissector parameter instead of using struct _packet_info.

svn path=/trunk/; revision=52885

More SVN properties cleanup.

svn path=/trunk/; revision=52884

Remove use of pinfo->private_data and just pass the data around through function parameters

Remove g_pinfo from packet-ansi_637.c and add packet_info* as a function parameter while we're at it.

svn path=/trunk/; revision=52883

Fix up some more svn properties.

svn path=/trunk/; revision=52882

Don't set svn:keywords on .pod files.  Remove it from those that have it.
Reorganize svnadd a little bit.

svn path=/trunk/; revision=52881

From robionekenobi via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9328 :
Suppress some VS2013 warnings when compiling gtk ui

svn path=/trunk/; revision=52880