From David Arnold:
Replace use of INCLUDES with AM_CPPFLAGS in all Makefiles to placate recent autotools.

svn path=/trunk/; revision=48261

Use explicit casts.

svn path=/trunk/; revision=48260

From beroset:

remove C++ incompatibilities from UAT_VS_DEF macro and all uses

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

svn path=/trunk/; revision=48259

SPC-3 and later has a 16-bit allocation length for the INQUIRY allocation
length.

While SPC-2 only has one byte for allocation length

Change this to treat allocation length as a 16 bit quantity starting one byte prior to the current single byte that wireshark dissects.
This makes it correct for SPC-3 and later  and still works for SPC-2 since that first byte is reserverd, == must be 0, in SPC-2 and prior.

svn path=/trunk/; revision=48258

Fix Coverity CID 989732: Structurally dead code.
Also, change a few tabs -> spaces.  Add modelines.

svn path=/trunk/; revision=48257

Fix Coverity CID 990224 and 990225: Unused pointer value.

svn path=/trunk/; revision=48256

Improve heuristics for gmtrailer.

svn path=/trunk/; revision=48255

Dissect GRE Key and Service Selection.

svn path=/trunk/; revision=48254

Dissect Contractor Number digits.

svn path=/trunk/; revision=48253

Minor improvements to block allocator debugging.

svn path=/trunk/; revision=48252

Document the wmem override environment variable in the man pages, and improve
README.wmem in a couple of places.

svn path=/trunk/; revision=48251

Prettify the WinPcap version display.

svn path=/trunk/; revision=48250

Followup to r48218: remove documentation of the WIRESHARK_DEBUG_USE_SLICES environment variable.

svn path=/trunk/; revision=48249

Add Peter Hatina;
Do a few other minor edits.

svn path=/trunk/; revision=48247

Update GTK3 port status.

svn path=/trunk/; revision=48245

GTK3 "expand" handling changes.

As suggested by Peter Hatini in the patch attached to Bug #8315.

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

svn path=/trunk/; revision=48244

Remove leftover "//".

svn path=/trunk/; revision=48242

From the proposed patch by Peter Hatina in Bug #8315.

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

svn path=/trunk/; revision=48241

Set the "Capture Options" window as the parent ('transient_for') of the "Interface Management" window.

svn path=/trunk/; revision=48240

Use explicit casts.

svn path=/trunk/; revision=48239

From: Erik Hugne

tipc: update link header according to spec
The bcastsequence gap have been removed, and the sequence field is now 13 bits.

svn path=/trunk/; revision=48238

"Capture ! Options ! Manage Interfaces": Cleanup handling of Window epansion.

Improves 'expand' handling for both Gtk2 & Gtk3;

Adapted (with additions by me) from a proposed patch by Peter Hatina (Bug #8315).

Also:

1. Don't use gtk_widget_set_size() in several cases when a single use of
   gtk_window_set_default_size() for the "Manage Interfaces" window is sufficient.

2. Add an XXX comment as to a specific call to 'gtk_window_destroy_with_parent'
   actually being a no-op on Windows.

svn path=/trunk/; revision=48237

Allocate zeroed memory instead of manually zeroing some of the fields. We were
missing a few anyways.

Fixes the rest of https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8456

Also, add modelines.

svn path=/trunk/; revision=48236

Use col_append_sep_str instead of col_set_str. Fixes another piece of

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

Fix suggested by Jakub in comment #5.

svn path=/trunk/; revision=48235

Off by one error in name length.

svn path=/trunk/; revision=48234

r30032 again, actually make emem_verify_pointer() works with glib chunks.

after allocating glib chunk, amount_free should be 0,
and free_offset should be set to chunk size.

svn path=/trunk/; revision=48233

Use explicit casts.

svn path=/trunk/; revision=48232

Actually remove slab.h from the makefile.

svn path=/trunk/; revision=48231

Update usage info for valgrind script. We almost need a proper man page for
this now.

svn path=/trunk/; revision=48230

Glib docs recommend using the slice API if you know you won't need to realloc.

svn path=/trunk/; revision=48228

Remove slab.h and replace its last remaining usage with glib slices.

Take the opportunity to deduplicate some code that was common to both an
if block and its else block.

svn path=/trunk/; revision=48227

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

svn path=/trunk/; revision=48225

Add two missing NULL checks, fixes most of

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

svn path=/trunk/; revision=48224

Make the memory limit (in bytes or packets) used for storing packets
configurable via command line options and print specific drop numbers
at the end. This allows to tune the buffering when using multiple
threads.
This also fixes a logic arrow in enforcing the limits.
The patch for the enhanced counters is from Anders Broman.

svn path=/trunk/; revision=48223

Only cast the variable after checking it.

svn path=/trunk/; revision=48222

Add -t to synopsis.

svn path=/trunk/; revision=48221

Add the appropriate new WS_DLL_* declarations to the wmem API.

svn path=/trunk/; revision=48220

Remove the wmem slab. It was an optimization mimicking the emem slab
(removed in r48218) which did nothing particularly useful. Also lets us remove
another debugging environment variable.

svn path=/trunk/; revision=48219

Remove the emem slab feature (sl_* functions) completely, replacing it with
glib memory slices.

- We weren't doing anything with the emem slab that couldn't be done with glib
  slices.
- Removes a fair bit of code as well as one debugging environment variable.
- Glib slices are much cache-friendlier and are multi-threading friendly (if
  we ever go there).
- Allows glib to actually return slices to the OS on occasion. The emem slab
  would hold onto its memory forever which resulted in a great deal of wasted
  memory after closing a large file.

svn path=/trunk/; revision=48218

Use explicit casts.

svn path=/trunk/; revision=48217

[NPL] Misc fixes

svn path=/trunk/; revision=48216

From Thomas Wiens via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8455

Don't add an extra 16 to offset, we've been updating it as we go. Fixes cases
where valid AMS packets were marked as malformed.

svn path=/trunk/; revision=48215

Two instances of s/delete/delete_id/ to satisfy C++ that were missed in
r48212 and were causing build failures.

svn path=/trunk/; revision=48214

Update lex and yacc

svn path=/trunk/; revision=48213

Use explicit casts.

svn path=/trunk/; revision=48212

Make checkAPIs Happy

svn path=/trunk/; revision=48211

From Jose Pico via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8377 NEW FEATURE IMPLEMENTED: SMB2 SUPPORT FOR EXPORT->OBJECTS->SMB

Added functionality:
- SMB2 support for Export->Objects->SMB
- support for SMB_COM_CREATE, SMB_COM_OPEN, SMB_COM_READ and SMB_COM_WRITE commands
- Ability to choose between File Id and full file name as identifier for file re-building. Implemented as an option under Edit->Preferences->Protocols->SMB and Edit->Preferences->Protocols->SMB2.

Other minor changes and fixes:
- Full filename in file
- Inclusion of IP of SMB server when treeid name (i.e. hostname) is not known
- UTF-8 filenames encoding before passing them to Export Object Window
- Re-written insert_chunk function of export_object_smb.c to make it easier to debug
- Fixed of an error in insert_chunk function of export_object_smb.c (the verification of next free_chunk was always skipped after deleting one free_chunk).
- Removed duplicated code by inserting the function feed_eo_smb in packet-smb.c and packet-smb2.c
- Changed the label of Export->Objects->SMB menu into Export->Objects->SMB/SMB2

svn path=/trunk/; revision=48210

Put items in the tree we just created, mark an unused param as _U_.

Fixes GCC build.

svn path=/trunk/; revision=48209

Update URLs to specs.

svn path=/trunk/; revision=48208

Switch to WinPcap 4.1.3.

svn path=/trunk/; revision=48207

RTPS Cleanup, Part 2

1. Convert more proto_tree_add_text to proto_tree_add_<something else>/expert_info.  checkAPIs.pl is happy (for packet-rtps.c), but the raw number is a bit deceiving because of macros and wrappers.
2. Fixed the offending "uses snprintf + strlen to assemble strings" in packet-rtps.c.  The exact same code is used in packet-rtps2.c, so just proper refactoring will fix it.  There is still too much unnecessary use of g_snprintf/g_strlcpy/strlen, but that's for a later date.
3. Removed most of the "useless" wrapper functions.  Again, the number of proto_tree_add_text is deceiving, so the number of hfs that really need to be created is much larger.
4. Whitespace cleanup.   Removed a lot of whitespace so I could see more code on the screen, to help determine duplication between packet-rtps.c and packet-rtps2.c  Comments/descriptions of fields remain untouched.

The more I trim, the more I think this should all be in a single dissector file, which will be the goal of the next update.  Trying to patch this in somewhat manageable chunks.

Also need to submit sample traces generated for (fuzz)testing.

svn path=/trunk/; revision=48206

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

Fix minor cut-and-paste error in USB-Video dissector caught by cppcheck.

svn path=/trunk/; revision=48205

freekdesktop.org says all themes must fall back to 'hicolor' if they don't find
a theme-specific icon (and as such hicolor is where applications should
install their icons).  So: don't install some of our icons in the gnome area,
install them all in hicolor.

While we're at it, go ahead and install all the icon sizes we have.

If we're on SuSE, use their desktop-file-updater macro; without that they
won't recognize our desktop file.

Fix bug which prevented the MIME database from being updated if our install
prefix is not /usr .

svn path=/trunk/; revision=48204

The Icon is not supposed to have an extension; remove it.

svn path=/trunk/; revision=48203

Use explicit casts.

svn path=/trunk/; revision=48202

Dissect the DCCP header by stepping through it a field at a time and
adding the fields to the protocol tree as we fetch them, rather than
fetching a bunch of them up front.  That way, if the packet was cut
short by a snapshot length, we'll dissect what we have.

Create the top-level tree item at the beginning, with an unknown length,
and set the length when we're done.  If we're putting details into that
item, update the item as we process each field; also update the Info
column as we process each field.

Don't use tvb_bytes_exist() to do our own tvbuff bounds-checking; let
the tvbuffs themselves do that.  Do not use it to do header-length
checking; check the actual header length value instead.

Do not fail if the *captured* data in the tvbuff is too short; we're
selected by an IP protocol number, which we can expect to definitively
identify us, not by a transport-layer port number, which is often an
unreliable identifier.

Do header length checks as we go along, and bail as soon as we identify
the header length as wrong.  (We do the first check once we get the X
bit, so we know whether the generic header is 12 or 16 bytes long.)
Treat a too-*large* header as a protocol violation, not a malformed
packet indication.

Use tvb_get_ntoh24() and tvb_get_ntoh48() to fetch the sequence number,
rather than fetching it in pieces and putting them together ourselves.

Correctly pluralize "byte".

Don't use tvb_length_remaining() to check whether we have a payload, use
tvb_reported_length_remaining(), so we base it on whether the packet
actually had the data, not on whether we actually captured it.

svn path=/trunk/; revision=48201

Add a debug function (ifdef'ed out by default) and fix another two bugs found by
reading it's output.

svn path=/trunk/; revision=48198

Add another pair of assertions.

svn path=/trunk/; revision=48197

Add /SD defaults to every MessageBox call (even if they're commented
out). Fixes bug 8451.

Don't yell at the user so much.

svn path=/trunk/; revision=48196

Update to V11.5.0 (2012-12).

svn path=/trunk/; revision=48195

Use only simple allocators until I can work out the kinks with the block
allocator.

svn path=/trunk/; revision=48194

Should fix assertion failure seen by Anders on -dev.

https://www.wireshark.org/lists/wireshark-dev/201303/msg00081.html

svn path=/trunk/; revision=48193

Add new SIP headers.

svn path=/trunk/; revision=48192

Switch to c-ares 1.9.1.

svn path=/trunk/; revision=48191

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

Dissector for USBVIDEO.

svn path=/trunk/; revision=48190

Don't try to show an IP address if none is present.

svn path=/trunk/; revision=48189

Prettify a bit dissection of sr-ProhibitTimer-r9 and drxShortCycleTimer

svn path=/trunk/; revision=48188

Use explicit casts.

svn path=/trunk/; revision=48187

Use explicit casts.

svn path=/trunk/; revision=48186

From beroset:
remove C++ incompatibilities

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

svn path=/trunk/; revision=48185

From Max Baker via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8431

Add a few member variables to the USB conversation structure, necessary
for a USB PTP dissector.

svn path=/trunk/; revision=48184

Use explicit casts.

svn path=/trunk/; revision=48183

Make the prefernce settings work.

svn path=/trunk/; revision=48182

correctly mark the payload for unknown extension tags
(fixes a bug introduced by my previous commit)

svn path=/trunk/; revision=48181

Properly pluralize "octet" for the Extended Capabilities tag.

If we don't decode a tag, just say "Undecoded" on the top-level item;
the tag name or number is already on that item, and the length is
underneath it.

svn path=/trunk/; revision=48180

Use explicit casts.

svn path=/trunk/; revision=48179

[NPL] Simplify parser for EXPRESSION_FIELD (Expression, ".", ID)

svn path=/trunk/; revision=48178

Fix proto stats window size allocation so it works with GTK3.
(adapted from a suggested patch by Peter Hatina in Bug #8315)
(https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8315)

From me:
 Widen the default width to show all the columns when the window is opened.

svn path=/trunk/; revision=48177

dissect Supplementary Audio Descriptor as defined in EN300468

svn path=/trunk/; revision=48176

use defines for extension tags

svn path=/trunk/; revision=48175

From Ed Beroset via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8416 :
Remove C++ incompatibilities from ssl and ssh files

svn path=/trunk/; revision=48174

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

svn path=/trunk/; revision=48173

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

svn path=/trunk/; revision=48172

Revert part of r48021: those hunks are no more needed with r48170

svn path=/trunk/; revision=48171

From Balint:

[PATCH 1/2] Revert "Try to fix the "LNK4217: locally defined symbol"
warnings.

This reverts commit r48158.

[PATCH 2/2] Employ small hack in editcap to link with a few objects from
libwireshark properly

From me:

Add the ability to reset symbol exports via ws_symbol_export.h's include
guard and do so in capinfos.c and editcap.c. We include ws_symbol_export.h
in over 200 files so it didn't seem to make sense to remove its include
guard entirely.

svn path=/trunk/; revision=48170

Tweak the handling of resizing the window.

svn path=/trunk/; revision=48169

Use if_string instead og if_string->name.

svn path=/trunk/; revision=48168

Use ifdefs for buffer size.

svn path=/trunk/; revision=48167

Use ifdefs for buffer size.

svn path=/trunk/; revision=48166

More ifdefs. Maybe this makes the mac buildbots happy.

svn path=/trunk/; revision=48165

Try to fix the buildbots.
The patch for capture_dlg.c is from Anders...

svn path=/trunk/; revision=48164

Add buffersize and snap length to the preferences.

svn path=/trunk/; revision=48163

Use explicit casts.

svn path=/trunk/; revision=48162

Use explicit casts.

svn path=/trunk/; revision=48161

List more (obvious) build dependencies.

The problem with listing these package names (which I think is convenient)
is that different distros have different names for some packages.  So:
update to work on OpenSuSE.

svn path=/trunk/; revision=48160

Avoid a potential divide-by-zero condition.
#BACKPORT(1.8)

svn path=/trunk/; revision=48159

Try to fix the "LNK4217: locally defined symbol" warnings.

svn path=/trunk/; revision=48158

Update the changelog.

svn path=/trunk/; revision=48151

Enable c-ares support by default to match the Windows and Debian
packages (and to make things less painful if the user enables name
resolution). Untested.

svn path=/trunk/; revision=48150

Always build with GDK_PIXBUF_DISABLE_DEPRECATED, GDK_DISABLE_DEPRECATED, GTK_DISABLE_DEPRECATED
Also: # Enable GSEAL when building with GTK > 2.20

svn path=/trunk/; revision=48149