Actually use the new environment variable created in r49784--apparently I forgot to check in this file (sigh).

svn path=/trunk/; revision=50257

[Automatic manuf, services and enterprise-numbers update for 2013-06-30]

svn path=/trunk/; revision=50254

proto_custom_set: don't fetch values twice.

svn path=/trunk/; revision=50252

ip_proto.h now requires expert.h.

svn path=/trunk/; revision=50249

Add expert info to dissect_ip_tcp_options().  The proto_tree_add_text seems a bit duplicative, but the byte highlighting over the whole option is nice.

svn path=/trunk/; revision=50248

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

svn path=/trunk/; revision=50247

Fix warning from fix-encoding-args

epan/dissectors/packet-scsi.c:  FT_BYTES:         proto_tree_add_item(tree, hf_scsi_report_opcodes_cdb_usage_data,                            tvb_v, offset_v+4, length, [[ENC_BIG_ENDIAN]-->[ENC_NA]]);

svn path=/trunk/; revision=50246

Fix indent (Tabs => Spaces) and add modelines info for follow_* files

svn path=/trunk/; revision=50245

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

svn path=/trunk/; revision=50244

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

svn path=/trunk/; revision=50243

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

svn path=/trunk/; revision=50242

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

svn path=/trunk/; revision=50241

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

svn path=/trunk/; revision=50240

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

svn path=/trunk/; revision=50239

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

svn path=/trunk/; revision=50238

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

svn path=/trunk/; revision=50237

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

svn path=/trunk/; revision=50236

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

svn path=/trunk/; revision=50235

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

svn path=/trunk/; revision=50234

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

svn path=/trunk/; revision=50233

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

svn path=/trunk/; revision=50232

Update .gitignore to add some Windows specific build stuff to ignore list.

svn path=/trunk/; revision=50231

Separate IP option header fields for each dissector that uses dissect_ip_tcp_options().  Bug 8823 (https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8823)

svn path=/trunk/; revision=50230

From Fabio Tarabelloni via
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8718

Zigbee dissectors for the power profile cluster and the appliance control
cluster

svn path=/trunk/; revision=50229

Improvements for the KDSP dissector.  Bug 8864 (https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8864)

From Yuri Schaeffer

It addresses the following issues:

- Payload was included for all CAPPACKET messages. Even when not flagged by bitmap (bug).
- Frame Checksum (FCS) was not read from bitmap all following data is off by 4. (bug)
- Headers indicated in bitmap could use own subtree
- Payload is malformed because it is assumed the span 'the rest of the packet'. In reality more commands can follow. (bug)

svn path=/trunk/; revision=50228

One further iteration, get things right, put placeholders for things to do.

svn path=/trunk/; revision=50227

Callers to dissect_ndr_* functions expect pdata to be initialized on return from
the function, so make sure it is even in error conditions and other anomaly
cases. Fixes the valgrind errors with the capture from
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8828

svn path=/trunk/; revision=50226

Rename octet_to_hex() to low_nibble_of_octet_to_hex() to indicate what
it actually does (it takes the low-order nibble of an octet and returns
a single character).

Cast away some warnings.

svn path=/trunk/; revision=50225

Squelch a warning.

Rename the "is_int" argument to fill_label_number() to make it clearer
what it indicates, i.e. whether the number is signed or unsigned.

svn path=/trunk/; revision=50224

Use octet_to_hex() in another place.

svn path=/trunk/; revision=50223

Rename AC_WIRESHARK_GCC_CFLAGS_CHECK to
AC_WIRESHARK_COMPILER_FLAGS_CHECK, because it doesn't just affect CFLAGS
and it doesn't just affect the flags for GCC.

svn path=/trunk/; revision=50222

Change formatting in the hopes of making the code a bit easier to figure
out, and update a comment.

svn path=/trunk/; revision=50221

Merge fill_label_[u]int to fill_label_number, hfinfo_[u]int_value_format to hfinfo_number_value_format

svn path=/trunk/; revision=50220

When checking whether a given option should be added to the compiler
flags, if the option should be added to the flags for both C and C++,
test both the C and C++ compilers and, if the answers are different,
print a warning; the user might have (intentionally or unintentionally)
selected mismatched compilers, e.g. clang and g++ on OS X.

svn path=/trunk/; revision=50219

MS: new child
MS: ping child
MS: set/get child parameter

svn path=/trunk/; revision=50218

Merge _hfinfo_int_value_format and _hfinfo_uint_value_format to one function hfinfo_number_vals_format

In hfinfo_number_vals_format check how hf should be formatted.

svn path=/trunk/; revision=50217

#micro-optimization, remove g_snprintf()'s from _hfinfo_[u]int_value_format with some new fresh replacements.

svn path=/trunk/; revision=50216

Do the "is $CXX actually a C++ compiler" test as early as possible, so
that it doesn't fail due to the C++ compiler not supporting -W options
that the C compiler does.

(We should fix that, too, by having separate checks for whether the C
and C++ compilers support particular options.)

svn path=/trunk/; revision=50215

Use newly assigned DLT for PDU export functionality

svn path=/trunk/; revision=50212

From Dirk via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8850 :

The attached patch fixes the integer type of the WCCP identity mask value.
This is a bitmask which should be printed as hex, it doesn't make sense to
print it as an IPv4 address. See
http://tools.ietf.org/id/draft-wilson-wrec-wccp-v2-01.txt section 5.7.7 and
the attached capture file as an example.

The current draft http://tools.ietf.org/html/draft-mclaggan-wccp-v2rev1-00#section-6.15
doesn't mention "mask" in the names of the field any more, but the description
still describes them as mask values.

svn path=/trunk/; revision=50211

From Lorand Jakab via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8852 :

The I and R flags in Map-Notify LISP control packets are shown at an incorrect
position.  The attached patch fixes the bug.

svn path=/trunk/; revision=50210

replace object_identifier_id with actx->external.direct_reference

svn path=/trunk/; revision=50209

Remove all global variables in P1 dissector.  Bug 8515 (https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8515)

Added a "subtree context" structure to asn1_ctx_t. This should allow other ASN.1 dissector global variables to be replaced when only used for transferring data between fields in a subtree.

svn path=/trunk/; revision=50208

From Niels de Vos via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8861 :

Recent versions of GlusterFS have extended the RPC protocol with new
procedures. The RPC-program-version has not been updated (yet?).

The attached adds support dissecting the FREMOVEXATTR, FALLOCATE and
DISCARD procedures.

svn path=/trunk/; revision=50207

Standard C++ header files don't have .h in their name: #include <iostream>
rather than #include <iostream.h>.

Also reference cout as std::cout in the test program.

svn path=/trunk/; revision=50206

AC_LANG_PROGRAM's second argument is put inside "int main(){"/"}" by the
macro, and mustn't include that itself.

svn path=/trunk/; revision=50205

AC_PROG_CXX doesn't actually check whether what it finds is a real live
C++ compiler (it might not be one on, for example, OS X, due to "cc"
being a C compiler, "CC" referring to "cc" due to the case-insensitivity
of the default OS X file system, and "CC" being one of the names checked
for in AC_PROG_CXX), so if we really need a C++ compiler, test it with a
program that a C compiler won't compile.

svn path=/trunk/; revision=50204

Make colors used by colorize_filter_te_as_* as preferences.  Bug 5413 (https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5413)

svn path=/trunk/; revision=50203

From Fabio Tarabelloni via
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8718

More zigbee dissection, adding the following clusters:
- appliance identification
- meter identification
- appliance statistics
- appliance events and alert

svn path=/trunk/; revision=50202

Suppress "Of *COURSE* you're developing a Mac-only application - you're
on a Mac, right?  So of *COURSE* you want to use our shiny new frameworks
rather than those ugly old open-source multi-platform libraries, right?"
warnings.

svn path=/trunk/; revision=50200

use hf_try_val_to_str() in two more places.

svn path=/trunk/; revision=50199

Propagate the main_window_update change of few commits ago into the qt client
:wq

svn path=/trunk/; revision=50198

cleanup [u]int fill proto with value_string function

Create generic int/uint fill functions from hfinfo_[u]int_value_format.

XXX: to be honest I don't get it why if dev picked up BASE_DEC_HEX and has value string we're truncating it to BASE_DEC...

svn path=/trunk/; revision=50197

Actually, that's not a clang bug; my test case had a typo.  It's still a
case where clang is complaining about code that's OK, it's just not OK
in a way that lets clang determine that it is.

svn path=/trunk/; revision=50196

MS: list interfaces!

svn path=/trunk/; revision=50195

Remove a no-longer-used variable.

svn path=/trunk/; revision=50194

cleanup int/uint fill proto functions.

- hfinfo_uint_value_format(), hfinfo_int_value_format() formats value instead of returning proper format
- remove hfinfo_int_format(), hfinfo_uint_format

svn path=/trunk/; revision=50193

Windows changes for main_window_update ....

svn path=/trunk/; revision=50192

forgot from the las commit...

svn path=/trunk/; revision=50191

Add functions hf_try_val_to_str[_const]

svn path=/trunk/; revision=50190

cleanup, use packet_list_iter_nth_child() in another place.

svn path=/trunk/; revision=50189

get main_window_update out of the way... pass an update_cb to the capture_sync stuff
... as per the XXX comment removed from tshark.c this was a mess to keep the linker
happy... I couldn't!

I did this without even understanding whether calling main_window_update was realy
necessary in most cases. I guess nothing or more specific update cbs would be best.

svn path=/trunk/; revision=50188

For now, don't turn on -Wcast-align by default.  There are at least some
bugs it points out that probably mean the code won't work on machines
that require alignment (e.g., SPARC machines), but we'll turn it on once
we fix them.  (clang is fussier than GCC about this.)

svn path=/trunk/; revision=50187

Don't assume the system include files are under /usr/include; trick the
compiler into saying where they are, and use that.

Patch a bug in PortAudio pa_stable_v19_20111121 that clang found.

svn path=/trunk/; revision=50186

Fix https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8857 :
Do not assume that we must have at least 1 character present before \r\n

svn path=/trunk/; revision=50185

*sigh*

From the comment above wmem_tree_insert32_array():
 * If you use ...32_array() calls you MUST make sure that every single node
 * you add to a specific tree always has a key of exactly the same number of
 * keylen words or things will most likely crash. Or at least that every single
 * item that sits behind the same top level node always have exactly the same
 * number of words.

So clearly generating thousands of keys with random lengths while testing is
going to cause problems. Generate a set of random lengths, then use those
lengths consistently (but still generating random keys of those lengths).

Should hopefully fix the intermittent build-bot failures.

(unfortunately this does not manifest nicely, and I cannot see an easy way to
assert it so that we catch other people trying to use different-length key
subtrees)

svn path=/trunk/; revision=50184

MS: Pong from the dispatcher!

svn path=/trunk/; revision=50183

Misc little cleanups to wmem tree (comments, whitespace, assertions)

svn path=/trunk/; revision=50182

Remove debugging lines from wmem_test, I think I've figured out the problem.

svn path=/trunk/; revision=50181

From Audric Schiltknecht, bug 8521 (https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8521)

According to ETSI TS 102 771 (GSE implementation guidelines), "mandatory extension headers" - when GSE's protocol type field is (strictly) less than 256 (0x100) - are of 'pre-defined length (and format) that must be known by all GSE receivers'.

svn path=/trunk/; revision=50180

Bugfix implementation of RFC 4379.  Reported through bug 8835 (https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8835)

svn path=/trunk/; revision=50179

Don't assume that compilers not named "clang" will, by default, report
an error, or not issue warnings, by default if you give them an unknown
-f flag.  Instead, test that flag with all compilers, and use -Werror to
force it to error out.

As with C/C++ flags, so with C++-only flags.

svn path=/trunk/; revision=50178

Don't assume that compilers that need an extra option to report errors,
rather than just warnings, for unknown warning options are named
"clang"; they might not be.

svn path=/trunk/; revision=50177

#gtk #packet-list simplify, use gtk_tree_view_column_add_attribute() for text attribute.

svn path=/trunk/; revision=50176

Another clang issue to work around:

libgcrypt expects gnu89-style semantics for "inline"; GCC defaults to
gnu89, but clang defaults to gnu99.

svn path=/trunk/; revision=50175

Suppress -Wformat-nonliteral when building GLib, to avoid a clang bug
where it issues bogus warnings.

svn path=/trunk/; revision=50174

(Attempt to) fix https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8813 :

Enable INET6 checking on Solaris versions other than 5.8.

(Yes, theoretically this should be checking for Solaris 5.7 or later.  I'm
going to go out on a limb and assume that no one is actually still using 5.7
or, if they are, they aren't trying to compile a modern Wireshark on it...)

svn path=/trunk/; revision=50173

packet-editor: Allow focus for byteview, it should fix problems with field hexeditor not getting key-press-event.

Generally it should be also safe for builds without packet-editor, just in case ifdef it.

svn path=/trunk/; revision=50172

Fix https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8853 :
Add a missing subtree initialization

svn path=/trunk/; revision=50171

Don't link wireshark against the Qt libraries and don't like wireshark-qt
against the Gtk+ libraries.

(I didn't notice this before because the resulting programs aren't, on my
system, actually linked against the libraries they don't use--presumably
because of the "-Wl,--as-needed" flag.)

svn path=/trunk/; revision=50170

Adjust when name_len offset is used.  Part of bug 8776 (https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8776)

From Dario Lombardo

svn path=/trunk/; revision=50169

echld is not an optional package, it's an optional Wireshark feature: use
--enable-echld rather than --with-echld to control whether it is built.

svn path=/trunk/; revision=50168

If the user enables gtk3, (silently) disable gtk2 rather than forcing the user
to also specify --without-gtk2 or complaining that they have specified both.

svn path=/trunk/; revision=50167

openSAFETY: CRC Maintenance, new CRC, new expert interface implemented.  Bug 8847 (https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8847)

From Roland Knall

svn path=/trunk/; revision=50166

Move tap-*-common.{h,c} into ui/ since that's the home for common UI functions.

svn path=/trunk/; revision=50165

Move tempfile.{h,c} into wsutil.

svn path=/trunk/; revision=50163

Nothing uses the HAVE_GTK #define, so get rid of it.  (Keep the
Makefile.am conditional, however.)

svn path=/trunk/; revision=50162

editcap no longer has to link against nstime.o, it's in libwsutil now.

svn path=/trunk/; revision=50161

This file is in the top-level directory. Don't try to include things from ../

svn path=/trunk/; revision=50160

As pointed out by Guy: timestats uses packet_info so it belongs in epan
not wsutil.

svn path=/trunk/; revision=50159

Put wsutil/*.h into the wireshark-dev package (in a blind attempt to fix the debian packaging step).  Why do we allow/encourage a wirshark-dev package?  IMO Wireshark (C language) development should only be done from the source (a tarball or better yet SVN).

svn path=/trunk/; revision=50158

There's no need to compile timestats.c any more--it's now in libwsutil.

svn path=/trunk/; revision=50157

Arrays seems to be where the failure is, so remove excess verbiage around other
tests, and add more details to that section.

Now we wait for the buildbots to fail again...

svn path=/trunk/; revision=50156

Link this plugin with libwsutil too (in order to pick up time symbols).

svn path=/trunk/; revision=50155

Move a couple of time-related modules into wsutil.

A bunch of files didn't really need to include these header files so remove
the include line rather than changing it.

svn path=/trunk/; revision=50154

Fix build with packet editor enabled.  (Fix a typo while we're at it.)

svn path=/trunk/; revision=50153

Last changes to autofoo, rename the files as they live now in their own dir...

svn path=/trunk/; revision=50152

From Reinhard Speyerer via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8851:
ansi_637_tele dissector displays MSB as MBS for Call-Back Number

svn path=/trunk/; revision=50151

Look for moc-qt4 before moc and uic-qt4 before uic, because (at least on my
machine, with both qt4 and qt5 development tools installed), moc and uic are the
qt5 versions, but the rest of the configuration process grabs qt4 headers and
link paths, leading to a lot of weird errors.

svn path=/trunk/; revision=50150

Of course I forgot to test the normal/default case (where no --with arguments
are given)...

Actually default --with-gtk2 to "yes".

svn path=/trunk/; revision=50149