Use the 'proto_tree *' (not the 'proto_item *') for a proto_tree_add_item(): Coverity 963 {UNUSED]

svn path=/trunk/; revision=37369

Pass tap info into sequence analysis function.

svn path=/trunk/; revision=37368

Detect when padding non-final padding comes after other control subheaders.

svn path=/trunk/; revision=37367

(Trivial) Testing _MSC_FULL_VER can be done at compile time.

svn path=/trunk/; revision=37366

Fix compilation.

svn path=/trunk/; revision=37365

Remove an unneeded redundant line of code: Fixes Coverity 1001.
Also:
 Use a separate ett variable for a subtree;
 Convert what appears to be '4 space tabs' to '4 spaces'.

svn path=/trunk/; revision=37364

Use correct subtree in two cases: Fixes Coverity 920;
Also:
  Remove a number of unneeded #include statements;
  Do some whitespace cleanup.

svn path=/trunk/; revision=37363

Fix compilation.

svn path=/trunk/; revision=37362

Use correct subtree in a few cases: Fixes Coverity 921,894,878 & 1136;

svn path=/trunk/; revision=37361

Use correct subtree in several 'proto_tree_add_text(...)' statements: Fixes Coverity 938 [UNUSED].

svn path=/trunk/; revision=37360

Use correct subtree in 'call dissector(...)': Fixes Coverity 962 [UNUSED].

svn path=/trunk/; revision=37359

Add a missing line of code: Fixes Coverity 967 [UNUSED].

svn path=/trunk/; revision=37358

Use correct subtree for proto_tree_add_item in several places: Coverity 1130 [UNUSED];
Also: Reformat a few lines and do some whitespace cleanup.

svn path=/trunk/; revision=37357

Add missing braces.

svn path=/trunk/; revision=37356

7.3.2.25 or 802.11-2007 gives an example of length 20, so it is unclear where the 24 came from (unintentionally fixes bug 5942)

svn path=/trunk/; revision=37355

Clean duplicated contributors.

svn path=/trunk/; revision=37354

Delete unused header field entries found by running checkhf.pl.

svn path=/trunk/; revision=37353

Added Michael Mann for his contributions to bug 2794.

svn path=/trunk/; revision=37352

From Micheal Mann via bug 2794: Fix some display filter field names.

svn path=/trunk/; revision=37351

From Yaniv Kaul: Some 'set but not used' compilation fixes;
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5858

From me: 2 corrections to the patch and sevral other minor changes.

svn path=/trunk/; revision=37350

[Automatic manuf, services and enterprise-numbers update for 2011-05-22]

svn path=/trunk/; revision=37346

Fix the case where you start with interfaces on the command line
(so using ifaces) and then using the GUI (which only deals with
iface).

These kind of problems will completely disappear when the
old strucuture components like iface have completely be
retired. This is the next thing we will work on.

svn path=/trunk/; revision=37345

Use appropriate temporary filename when capturing from multiple interfaces.

svn path=/trunk/; revision=37344

As suggested on the dev list: Use pcapng and threads
when capturing on multiple interfaces even if not specified
on the command line.

svn path=/trunk/; revision=37343

Fix the capturing from the GUI. Honor the iface variable.

The next step is to retire the iface variable and use
the ifaces array only. This should simplify things and
fix the cases, where you start wireshark with command
line arguments and also use the GUI.

svn path=/trunk/; revision=37342

Pass a pcap_options * instead of a pcap_options **.

svn path=/trunk/; revision=37341

Don't assign to a variable which is then not referenced: Coverity 998 [UNUSED].

svn path=/trunk/; revision=37340

Don't assign to an unused variable: Fixes Coverity 874 [UNUSED].

svn path=/trunk/; revision=37339

Fix a typo ...

svn path=/trunk/; revision=37338

Remove unneeded code: Fixes Coverity 890 [UNUSED].

svn path=/trunk/; revision=37337

Use (what appears to be) the correct tvb for a function call: Fixes Coverity 919 [UNUSED];
#include <stdlib.h> not req'd.

svn path=/trunk/; revision=37336

Remove unused code: Fixes Coverity 972 [UNUSED].

svn path=/trunk/; revision=37335

Add #include <stdlib.h> to files which reference fcns declared in stdlib.h;
 (In many cases I previously incorrectly removed the #include <stdlib.h>).

svn path=/trunk/; revision=37334

Add bugs 4985 and 5580 to the list of known problems.

svn path=/trunk/; revision=37333

Don't assign to a proto_item* if the value won't be used:
 Fixes Coverity 903,935,968,985,997,999,1024,1025,1134;
Remove unneeded #includes;
Do whitespace cleanup.

svn path=/trunk/; revision=37331

Add to comment about using a heuristic; Fix a typo.

svn path=/trunk/; revision=37330

Use correct parent tree when creating a subtree: Fixes Coverity 902 [UNUSED];
Remove unneeded #includes;
Do some whitespace cleanup.

svn path=/trunk/; revision=37329

'#if 0' code related to already commented out code: Fixes Coverity 881.

svn path=/trunk/; revision=37328

Don't assign to a proto_item* if the value won't be used: Coverity 975 [UNUSED]
Remove unneeded prefs callback in prefs registration;
Remove 'if (!initialized)' test from proto_reg_handoff...();
Remove unneeded #include;
Do some whitespace cleanup.

svn path=/trunk/; revision=37327

Prettify som 3GPP AVP:s

svn path=/trunk/; revision=37326

gtk_menu_item_set_label() is from gtk 2.16.

svn path=/trunk/; revision=37325

Include the profile name in the Delete item in the profiles popup.
Fixed creating a new profile from the popup without setting a create from.
Removed some unneeded ep_strdup_printf.

svn path=/trunk/; revision=37324

Copy data from the TVB into ep_alloc'd memory, rather than saving a pointer into the TVB.

svn path=/trunk/; revision=37323

Use tvb_get_ephemeral_string() instead of tvb_get_ptr().

svn path=/trunk/; revision=37322

Use tvb_bytes_to_str() instead of a homegrown version.

svn path=/trunk/; revision=37321

Use TVB accessor functions instead of tvb_get_ptr()

svn path=/trunk/; revision=37320

Use tvb_ether_to_str()

svn path=/trunk/; revision=37319

Fix annoying typo.

svn path=/trunk/; revision=37318

Use proto_tree_add_item() instead of proto_tree_add_xxx() with a pointer into
the TVB.

Introduce and start using tvb_get_manuf_name() and tvb_get_manuf_name_if_known()
which do the same as the non-tvb versions but take a tvb and an offset instead
of (commonly) a pointer into a TVB.

svn path=/trunk/; revision=37317

In at least some versions of OS X SystemStarter gets cranky if the
startup item is group-writable.

svn path=/trunk/; revision=37316

Add pdml2html.xsl to the source tarball.

(Can we have a macro that has everything in pkgdata_DATA except for
COPYING, and use that macro in both the definition of pkgdata_DATA and
EXTRA_DIST?)

svn path=/trunk/; revision=37314

Update the User's Guide.

svn path=/trunk/; revision=37312

Drop privileges after opening all pcap devices, not after the first one...

svn path=/trunk/; revision=37311

1.6 → 1.7.

svn path=/trunk/; revision=37309

Don't assign to an unused 'proto item *': Coverity 1033 & 906

svn path=/trunk/; revision=37308

In case we have no interface information in the array, use the global value.
This fixes https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5935.

svn path=/trunk/; revision=37307

Create subtree under the correct parent tree: Coverity 1138.

svn path=/trunk/; revision=37306

Don't assign a value which is never used to a 'proto item *': Covertiy 1135;
Remove some unneeded #includes.

svn path=/trunk/; revision=37305

Fix markup.

svn path=/trunk/; revision=37304

Update the version and command output.

svn path=/trunk/; revision=37303

#if 0 some not currently used code: Fixes Coverity 966 [UNUSED]
Convert apparent "4 space tabs" to "4 spaces".

svn path=/trunk/; revision=37302

Remove unneeded variable and code; Rename a function arg; Fix usage of _U_: Fixes Coverity 1146 [UNUSED]

svn path=/trunk/; revision=37301

FOP 0.95 → 1.0.

svn path=/trunk/; revision=37300

From Dirk Jagdmann via bug 5875:

My attachment adds a link to a XSLT file to the preamble of the PDML.
The XSLT will transform the PDML to a HTML page, and the HTML page
features a look similar to Wireshark. See
http://cubic.org/~doj/ebay/a.pdml for an example.

The patch also contains a small perl program which converts the
Wireshark colortable into javascript code which is used in the XSLT
file. If you want to use a different color scheme you would execute the
perl program and insert the generated javascript function into your XSLT
file.

To view the HTML you could either place the PDML and XSLT file on your
webserver and verify that your webserver sends the PDML file as
"text/xml".  Then your webbrowser will find the linked XSLT file,
download that as well and convert the PDML to HTML on the fly.

You could also use an XSLT processor like xsltproc to convert the PDML
and XSLT into a static HTML file.

From me:

Minor fixups.

svn path=/trunk/; revision=37298

Give some routines used for info levels that appear both in set and get
names beginning with dissect_qspi_, and give some of them names with the
info level structure in them rather than the SNIA CIFS specification
section number.

Have separate routines for SMB_INFO_STANDARD and SMB_INFO_QUERY_EA_SIZE;
SMB_INFO_STANDARD is specified differently in the SNIA CIFS
specification and the MS-CIFS specification, and some captures have the
SNIA CIFS version, with the EA length and some have the MS-CIFS version
without it.  The dissector for SMB_INFO_STANDARD will dissect it if it's
there and not say "this structure is truncated" if it's not there.

Rename dissect_qfi_SMB_FILE_ALTERNATE_NAME_INFO() to
dissect_qfi_SMB_FILE_NAME_INFO(), as it also dissects
SMB_QUERY_FILE_NAME_INFO.

Merge the dissectors for SMB_FILE_ALLOCATION_INFO and
SMB_SET_FILE_ALLOCATION_INFO, and for SMB_FILE_END_OF_FILE_INFO and
SMB_SET_FILE_END_OF_FILE_INFO, as the structures are the same.

Dissect some presumed "passthrough info levels" the same way the
corresponding official SMB infos are dissected.

Expand some comments for info level dissectors to give the MS-CIFS
section number and to give some other details.

If an info level is truncated, put in an expert info error.

If we don't know about a given info level, just dissect the body as
"Information level unknown", rather than having it dissected as an
"unknown information" trailer.

svn path=/trunk/; revision=37297

When capturing from multiple interface, indicate this in the main status bar.

This patch is, as all other patches related to capturing from multiple interfaces,
based on work of Irene Ruengeler.

svn path=/trunk/; revision=37296

Fix compilation.

svn path=/trunk/; revision=37295

Fix a typo.

svn path=/trunk/; revision=37294

Don't assign a value to a variable which is then not used: Coverity 1145

svn path=/trunk/; revision=37293

Subdissect 2 3GPP AVP:s that are actually text.

svn path=/trunk/; revision=37292

Don't assign to an unused variable; remove variable definition: Coverity 1144.

svn path=/trunk/; revision=37291

Don't assign a value to a variable which is then never used: Coverity 1160 [UNUSED]

svn path=/trunk/; revision=37290

When capturing from multiple interfaces, indicate this in the window
title of the wireshark main window.

svn path=/trunk/; revision=37289

Remove unused define.

svn path=/trunk/; revision=37288

It is not an error (anymore) to specify multiple capture filters.
This was
wireshark -i lo0 -f sctp -i en0 -f udp -k
will work.

svn path=/trunk/; revision=37287

The routines declared in packet-smb.h are defined in packet-smb.c;
include packet-smb.h in packet-smb.c so that we check the declarations
against the definitions.

In query ops, info level 2 is Query EA Size, not Query EAs From List.
In set ops, info level 2 is Set EAs, not Query EA Size.

Expand the constants for the Trans2 subcodes to 16 bits.

The tvb argument to dissect_find_file_unix_info2() is used.

svn path=/trunk/; revision=37286

Actually use the defaults from the last commit message.
... and switch off debug output.

svn path=/trunk/; revision=37285

Add a queue and byte limit to the capture queue. Current default
values: 1000 packets, 1 MB. Should be made configurable via
command line options.

svn path=/trunk/; revision=37284

Fix warnings about unused variables in UI_MANAGER code.
Move gsm_a_stat.c back to clean dissectors.

svn path=/trunk/; revision=37283

Fix two warning in case of UIMANAGER:
- Unused variable
- ‘graph_type’ may be used uninitialized

Is the second part of the patch (added inside the else clause)
correct?

svn path=/trunk/; revision=37282

Fix another old style function declaration warning

svn path=/trunk/; revision=37281

When capturing from multiple interfaces, indicate this in the
capture info dialog.
This is based on wirk from Irene Ruengeler.

svn path=/trunk/; revision=37280

Fix the capture info dialog. Reset the EOF state such that the
number get actually updated.

svn path=/trunk/; revision=37279

Fix an old style function declaration warning.

svn path=/trunk/; revision=37278

Update CMakeLists to follow commit 37219.
Remove a duplicate check in configure.in

svn path=/trunk/; revision=37277

Move the debug output for the generated command line arguments to the correct place.

svn path=/trunk/; revision=37276

For old times' sake: Fix bad dissection of Ethernet Router Hello Messages: Coverity 876 & 877 [UNUSED]

svn path=/trunk/; revision=37275

Add a check that (plain) value_strings that are used in hf items don't have
any conflicting entries.  i.e. lots of value_strings have repeated items, but
for now only warn for cases where the same numeric value appears with a
different string.

Because this will slow down startup and output distracting warnings, it has been
#if 0'd out for now.  As discussed on the dev mailing list, it'd be good to
create a #define for developer/non-release builds so that tests such as this
can regularly be run.

svn path=/trunk/; revision=37274

Only declare a variable if its going to be used.

svn path=/trunk/; revision=37273

Improve AM sequence analysis, check ACK looks sensible compared with latest SN received in opposite direction.

svn path=/trunk/; revision=37272

Set subheader root length to 1, so if contentsinvalid will already be correctl length.

svn path=/trunk/; revision=37271

Don't create 2 unneeded subtrees; Fixes Coverity 879 & 880 [UNUSED];
#include <stdio.h> & <time.h> not needed.

svn path=/trunk/; revision=37269

Don't assign to an unused 'proto_item *': Fixes Coverity 883 [UNUSED];
Use correct 'tree' arg in a dissect(...) fcn call: Fixes Coverity 884 [UNUSED];
#include <stdlib.h> not req'd.

svn path=/trunk/; revision=37268

Create 2 subtrees using the correct 'proto_item *'; Fixes Coverity 900 & 901 [UNUSED];
#inclide <stdlib.h> not req'd.

svn path=/trunk/; revision=37267

#include <stdlib.h> not req'd;
Reformat hf[] definition;
Misc whitespace cleanup;

svn path=/trunk/; revision=37266

Add a missing creation of a subtree: Fixes Coverity 907 [UNUSED];
Don't create an unused subtree: Fixes Coverity 908 [UNUSED];

svn path=/trunk/; revision=37265

Fix tvb_get_bits_buf for LSB.

svn path=/trunk/; revision=37264

(Trivial) Fix a typo; Add spme punctuation in a coment.

svn path=/trunk/; revision=37263

Fix some (value) mistakes in value_strings.  Found by Martin's patch on the -dev list.

svn path=/trunk/; revision=37262

Fix a couple (value) mistakes in a value_string, taking the values from the IDL
file.  Found by Martin's patch on the -dev list.

Add a comment to answer the question of whether this file is autogenerated by
pointing to the IDL file.

svn path=/trunk/; revision=37261