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

Update to the latest config.guess and config.sub (I needed these so configure
would recognize OpenSuSE 12.1).

Note: these are now GPLv3 but there's an exception which allows us to
distribute these files under our distribution terms.

svn path=/trunk/; revision=48148

Add support for software updates. Remove qtshark.rc and use
image/wireshark.rc (which contains resources needed by WinSparkle)
instead.

svn path=/trunk/; revision=48147

look for /usr/include/lua5.x directories to determine the lua version

(Debian squeeze installs lua includes in /usr/include/lua5.1 and libs
 under /usr/lib, there's no --with-lua=<dir> setting for this
 configuration)

see also https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8438

svn path=/trunk/; revision=48146

Always build with GDK_DISABLE_DEPRECATED, GDK_PIXBUF_DISABLE_DEPRECATED, GTK_DISABLE_DEPRECATED

Also: remove "experimental" comment from Win32 GTK3 build config;

(I'm not able to try the WIN64 GTK3 build so I've left that marked as experimental).

svn path=/trunk/; revision=48145

Use explicit casts.

svn path=/trunk/; revision=48144

Get rid of xor_gc altogether, I think it's a remnant from the "old" cross-hair code.

svn path=/trunk/; revision=48141

Get rid of xor_gc altogether, I think it's a remnant from the "old" cross-hair code.

svn path=/trunk/; revision=48140

Use explicit casts.

svn path=/trunk/; revision=48137

Add asn2deb.pod and idl2deb.pod to the source distribution.

svn path=/trunk/; revision=48136

Don't use (GNU-specific) pattern rules: use old-fashioned suffix rules instead.

Since there's now a suffix rule for the .1 man pages, don't create explicit
rules for them.  (Leave a comment explaining why the .html and .4 rules need
to stay.)

Simplify CLEANFILES: there's no need to list each man page, just remove them
all.

svn path=/trunk/; revision=48135

Fix distcheck in autotools based builds

svn path=/trunk/; revision=48134

Merge asn2deb and idl2deb from official Debian package

svn path=/trunk/; revision=48133

gtk_alignment_new() args are gfloat; use constants of form 0.0F.

(A floating point constant without a suffix (e.g., 0.0) is a 'double'.

svn path=/trunk/; revision=48129

Actually add IPv6 addresses to the display.

svn path=/trunk/; revision=48128

Fix the plurality of the length of a data source if it has only 1 byte (I'm
looking at a "Bitstring tvb" that is only 1 byte long).

svn path=/trunk/; revision=48127

Followup to r48011, answering my own XXX comment:

No, ReportedBoundsError is not the right thing to throw, ReassemblyError is.
That's why I added it in the first place!

svn path=/trunk/; revision=48123

From Peter Hatina: Fix for:  Gtk3 widgets are not properly expanded.
(Part of suggested patch for WIreshark 1.8 applied to trunk instead)

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

From me: Add an XXX xomment about how, on Windows, closing the top_level
Wireshark window while the capture_interfaces dialog window is open
will leave the associated dumpcap process running.

svn path=/trunk/; revision=48122

Make the GTP and GTP' dissectors new-style dissectors, so that they can
reject packets that don't look enough like GTP/GTP' packets.  This fixes
bug 1706, and fixes some other cases where non-GTP/GTP' packets are
being dissected as GTP/GTP'.

svn path=/trunk/; revision=48118

Don't say "Warn!", say "Warning!"

svn path=/trunk/; revision=48110

Update the information on building RPMs based on recent work.  Include the information that the Qt development package needs to be installed.

svn path=/trunk/; revision=48109

Use explicit casts.

svn path=/trunk/; revision=48108

Remove a comment left over from the "old days".

Also: Add an XXX comment re one effect of the
      fact that, on Windows, dlg_window_new() does *not*
      set the newly created dialog window as 'transient for'
      the Wireshark top_level window.

svn path=/trunk/; revision=48106

Don't have a separate srpm-package target: we already build the SRPM while building the RPM (since r47574).

svn path=/trunk/; revision=48105

Move the install_desktop_files target so it's not in the middle of the various package-making targets.

svn path=/trunk/; revision=48103

Use XDLC_IS_INFORMATION() to determine whether there's a payload to hand
to the X.25 dissector or not.

svn path=/trunk/; revision=48097

it seems that g_free() needs a gpointer

using a gconstpointer creates a warning
capture_sync.c: In function ‘sync_pipe_open_command’:
capture_sync.c:823:13: error: passing argument 1 of ‘g_free’ discards ‘const’ qualifier from pointer target type [-Werror]

svn path=/trunk/; revision=48096

fix
packet-bssgp.c: In function 'de_bssgp_unconfim_send_state_var':
packet-bssgp.c:3318:13: error: variable 'state_var' set but not used [-Werror=unused-but-set-variable]

svn path=/trunk/; revision=48095

Fix some variable names to better indicate what they do.

Fix indentation.

Fix a proto_tree_add_uint_format_value() call not to include the name of
the field - proto_tree_add_uint_format_value() will add that for you.

Have dte_address_util() take the offset of the address as an argument;
it's not always at the same offset from the beginning of the facility.

Have it return the pointer to the generated string directly, rather than
through a pointer argument.

Create only one subtree for each facility, and give it a text description
of the facility code rather than the numerical value of the facility
code.  Make the top-level item for the facility cover all the bytes of
the facility, including code, length if present, and parameters.

Dissect the end-to-end transit delay and priorities facilities
completely.  Also, fix an incorrect use of "transmit delay" to say
"transit delay".

Get rid of the last of the spaces preceding colons in "Field: value"
descriptions and in a "default:" case label.

Do the data vs. non-data packet thing ith

if (PACKET_IS_DATA(pkt_type)) {
...
} else {
...
}

rather than, in effect, doing the "else" with a break; that makes the
code a bit clearer.

Put the logical channel number into the protocol tree in common code for
the default case, rather than doing it separately for data and non-data
packets.  Clean up the dissection of non-data packets to add entries
before updating the columns, so that we don't throw an exception
updating the columns before we get to add items that wouldn't throw
exceptions.  Clear the Info column early in the dissection, in case we
throw an exception before getting to set it and thus leave behind the
column information for the protocol atop which we're running.

svn path=/trunk/; revision=48093

"HKEY_CURRENT_USER\Software\The Wireshark developer community, wireshark.org/\Wireshark\WinSparkle" isn't the prettiest registry path.

svn path=/trunk/; revision=48089

Use explicit casts.

svn path=/trunk/; revision=48088

From Joseph Chai:
Add aditional IE dissection.
From me patch cleanup and add set fence for multiple DTAP messages in one frame.

svn path=/trunk/; revision=48087

Use consistent indentation (4 spaces);
Reformat  long single line comments for readability;
Add editor modelines.

svn path=/trunk/; revision=48086