Commit Graph

47156 Commits

Author SHA1 Message Date
Jeff Morriss 8d29106ffb 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
2013-06-28 13:04:01 +00:00
Jeff Morriss 0d75d64949 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
2013-06-28 12:52:19 +00:00
Michael Mann c1ddd6046d replace object_identifier_id with actx->external.direct_reference
svn path=/trunk/; revision=50209
2013-06-28 12:52:13 +00:00
Michael Mann 9009944d51 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
2013-06-28 12:43:29 +00:00
Jeff Morriss c29fb44886 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
2013-06-28 12:42:41 +00:00
Jeff Morriss 8c826964bf 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
2013-06-28 12:18:03 +00:00
Guy Harris 9a09ef907a 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
2013-06-28 09:59:19 +00:00
Guy Harris fd5281c763 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
2013-06-28 08:36:35 +00:00
Michael Mann 277aed869e 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
2013-06-28 03:03:36 +00:00
Evan Huus 10918860b6 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
2013-06-28 01:47:18 +00:00
Guy Harris ca9122bfec 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
2013-06-27 23:11:05 +00:00
Jakub Zawadzki 4504ed0bcc use hf_try_val_to_str() in two more places.
svn path=/trunk/; revision=50199
2013-06-27 22:26:34 +00:00
Luis Ontanon 24a5467228 Propagate the main_window_update change of few commits ago into the qt client
:wq



svn path=/trunk/; revision=50198
2013-06-27 19:24:39 +00:00
Jakub Zawadzki ac3f0d7483 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
2013-06-27 19:17:22 +00:00
Guy Harris b7a3865c91 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
2013-06-27 19:14:41 +00:00
Luis Ontanon 99013aefa8 MS: list interfaces!
svn path=/trunk/; revision=50195
2013-06-27 19:00:01 +00:00
Jeff Morriss 16fc2c5998 Remove a no-longer-used variable.
svn path=/trunk/; revision=50194
2013-06-27 18:44:11 +00:00
Jakub Zawadzki c36a384b25 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
2013-06-27 18:40:08 +00:00
Luis Ontanon 7f5d688bda Windows changes for main_window_update ....
svn path=/trunk/; revision=50192
2013-06-27 17:45:07 +00:00
Luis Ontanon c780d97411 forgot from the las commit...
svn path=/trunk/; revision=50191
2013-06-27 17:27:41 +00:00
Jakub Zawadzki 3f5665a6b1 Add functions hf_try_val_to_str[_const]
svn path=/trunk/; revision=50190
2013-06-27 17:26:23 +00:00
Jakub Zawadzki 1895c1c5a0 cleanup, use packet_list_iter_nth_child() in another place.
svn path=/trunk/; revision=50189
2013-06-27 17:12:23 +00:00
Luis Ontanon 0bdc0efc46 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
2013-06-27 17:10:50 +00:00
Guy Harris b450609a55 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
2013-06-27 10:35:06 +00:00
Guy Harris f3f156073b 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
2013-06-27 10:30:33 +00:00
Pascal Quantin 9334a49cb1 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
2013-06-27 10:17:39 +00:00
Evan Huus 31bd48bce3 *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
2013-06-27 03:44:48 +00:00
Luis Ontanon e48b0084e1 MS: Pong from the dispatcher!
svn path=/trunk/; revision=50183
2013-06-27 03:41:48 +00:00
Evan Huus ffe6d9c4d6 Misc little cleanups to wmem tree (comments, whitespace, assertions)
svn path=/trunk/; revision=50182
2013-06-27 03:01:53 +00:00
Evan Huus 32f4010527 Remove debugging lines from wmem_test, I think I've figured out the problem.
svn path=/trunk/; revision=50181
2013-06-27 03:00:33 +00:00
Michael Mann 11cc4642a3 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
2013-06-27 02:34:57 +00:00
Michael Mann 9672f3d6a4 Bugfix implementation of RFC 4379. Reported through bug 8835 (https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8835)
svn path=/trunk/; revision=50179
2013-06-27 01:15:24 +00:00
Guy Harris 04cc1e2fb5 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
2013-06-27 00:45:39 +00:00
Guy Harris eaf5e3fd1f 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
2013-06-26 22:43:51 +00:00
Jakub Zawadzki 97d59088b0 #gtk #packet-list simplify, use gtk_tree_view_column_add_attribute() for text attribute.
svn path=/trunk/; revision=50176
2013-06-26 22:03:57 +00:00
Guy Harris 53bc39d6ed 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
2013-06-26 21:29:38 +00:00
Guy Harris c2e995bd6d Suppress -Wformat-nonliteral when building GLib, to avoid a clang bug
where it issues bogus warnings.

svn path=/trunk/; revision=50174
2013-06-26 19:38:47 +00:00
Jeff Morriss ffc5183fd2 (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
2013-06-26 18:19:43 +00:00
Jakub Zawadzki 0426c05d49 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
2013-06-26 17:15:25 +00:00
Pascal Quantin faecb6b9a8 Fix https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8853 :
Add a missing subtree initialization

svn path=/trunk/; revision=50171
2013-06-26 16:23:54 +00:00
Jeff Morriss 681efb2de3 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
2013-06-26 15:12:50 +00:00
Michael Mann 8da89ad091 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
2013-06-26 14:09:10 +00:00
Jeff Morriss cbe96aeb4d 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
2013-06-26 13:52:52 +00:00
Jeff Morriss 6df12cbcf7 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
2013-06-26 13:26:25 +00:00
Michael Mann 6e1f213fca 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
2013-06-26 12:11:08 +00:00
Jeff Morriss 500d399393 Move tap-*-common.{h,c} into ui/ since that's the home for common UI functions.
svn path=/trunk/; revision=50165
2013-06-26 02:12:12 +00:00
Jeff Morriss bbf6a3766e Move tempfile.{h,c} into wsutil.
svn path=/trunk/; revision=50163
2013-06-26 01:14:35 +00:00
Guy Harris 3ab335948b Nothing uses the HAVE_GTK #define, so get rid of it. (Keep the
Makefile.am conditional, however.)

svn path=/trunk/; revision=50162
2013-06-26 00:48:58 +00:00
Jeff Morriss 9ef2507e34 editcap no longer has to link against nstime.o, it's in libwsutil now.
svn path=/trunk/; revision=50161
2013-06-26 00:45:38 +00:00
Jeff Morriss 003ba14660 This file is in the top-level directory. Don't try to include things from ../
svn path=/trunk/; revision=50160
2013-06-26 00:20:56 +00:00