tuexen [Sat, 21 May 2011 20:57:00 +0000 (20:57 +0000)]
Use appropriate temporary filename when capturing from multiple interfaces.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37344
f5534014-38df-0310-8fa8-
9805f1628bb7
tuexen [Sat, 21 May 2011 18:51:25 +0000 (18:51 +0000)]
As suggested on the dev list: Use pcapng and threads
when capturing on multiple interfaces even if not specified
on the command line.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37343
f5534014-38df-0310-8fa8-
9805f1628bb7
tuexen [Sat, 21 May 2011 13:22:38 +0000 (13:22 +0000)]
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.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37342
f5534014-38df-0310-8fa8-
9805f1628bb7
gerald [Sat, 21 May 2011 00:10:57 +0000 (00:10 +0000)]
Pass a pcap_options * instead of a pcap_options **.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37341
f5534014-38df-0310-8fa8-
9805f1628bb7
wmeier [Fri, 20 May 2011 20:41:23 +0000 (20:41 +0000)]
Don't assign to a variable which is then not referenced: Coverity 998 [UNUSED].
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37340
f5534014-38df-0310-8fa8-
9805f1628bb7
wmeier [Fri, 20 May 2011 20:29:19 +0000 (20:29 +0000)]
Don't assign to an unused variable: Fixes Coverity 874 [UNUSED].
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37339
f5534014-38df-0310-8fa8-
9805f1628bb7
wmeier [Fri, 20 May 2011 20:22:59 +0000 (20:22 +0000)]
Fix a typo ...
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37338
f5534014-38df-0310-8fa8-
9805f1628bb7
wmeier [Fri, 20 May 2011 20:21:14 +0000 (20:21 +0000)]
Remove unneeded code: Fixes Coverity 890 [UNUSED].
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37337
f5534014-38df-0310-8fa8-
9805f1628bb7
wmeier [Fri, 20 May 2011 20:10:41 +0000 (20:10 +0000)]
Use (what appears to be) the correct tvb for a function call: Fixes Coverity 919 [UNUSED];
#include <stdlib.h> not req'd.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37336
f5534014-38df-0310-8fa8-
9805f1628bb7
wmeier [Fri, 20 May 2011 20:00:16 +0000 (20:00 +0000)]
Remove unused code: Fixes Coverity 972 [UNUSED].
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37335
f5534014-38df-0310-8fa8-
9805f1628bb7
wmeier [Fri, 20 May 2011 19:00:09 +0000 (19:00 +0000)]
Add #include <stdlib.h> to files which reference fcns declared in stdlib.h;
(In many cases I previously incorrectly removed the #include <stdlib.h>).
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37334
f5534014-38df-0310-8fa8-
9805f1628bb7
gerald [Fri, 20 May 2011 17:02:20 +0000 (17:02 +0000)]
Add bugs 4985 and 5580 to the list of known problems.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37333
f5534014-38df-0310-8fa8-
9805f1628bb7
wmeier [Fri, 20 May 2011 15:44:25 +0000 (15:44 +0000)]
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.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37331
f5534014-38df-0310-8fa8-
9805f1628bb7
wmeier [Fri, 20 May 2011 15:34:29 +0000 (15:34 +0000)]
Add to comment about using a heuristic; Fix a typo.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37330
f5534014-38df-0310-8fa8-
9805f1628bb7
wmeier [Fri, 20 May 2011 15:22:58 +0000 (15:22 +0000)]
Use correct parent tree when creating a subtree: Fixes Coverity 902 [UNUSED];
Remove unneeded #includes;
Do some whitespace cleanup.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37329
f5534014-38df-0310-8fa8-
9805f1628bb7
wmeier [Fri, 20 May 2011 15:19:26 +0000 (15:19 +0000)]
'#if 0' code related to already commented out code: Fixes Coverity 881.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37328
f5534014-38df-0310-8fa8-
9805f1628bb7
wmeier [Fri, 20 May 2011 14:21:02 +0000 (14:21 +0000)]
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.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37327
f5534014-38df-0310-8fa8-
9805f1628bb7
etxrab [Fri, 20 May 2011 13:55:21 +0000 (13:55 +0000)]
Prettify som 3GPP AVP:s
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37326
f5534014-38df-0310-8fa8-
9805f1628bb7
stig [Fri, 20 May 2011 11:58:00 +0000 (11:58 +0000)]
gtk_menu_item_set_label() is from gtk 2.16.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37325
f5534014-38df-0310-8fa8-
9805f1628bb7
stig [Fri, 20 May 2011 11:30:51 +0000 (11:30 +0000)]
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.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37324
f5534014-38df-0310-8fa8-
9805f1628bb7
morriss [Fri, 20 May 2011 03:22:27 +0000 (03:22 +0000)]
Copy data from the TVB into ep_alloc'd memory, rather than saving a pointer into the TVB.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37323
f5534014-38df-0310-8fa8-
9805f1628bb7
morriss [Fri, 20 May 2011 03:20:20 +0000 (03:20 +0000)]
Use tvb_get_ephemeral_string() instead of tvb_get_ptr().
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37322
f5534014-38df-0310-8fa8-
9805f1628bb7
morriss [Fri, 20 May 2011 03:17:01 +0000 (03:17 +0000)]
Use tvb_bytes_to_str() instead of a homegrown version.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37321
f5534014-38df-0310-8fa8-
9805f1628bb7
morriss [Fri, 20 May 2011 03:15:18 +0000 (03:15 +0000)]
Use TVB accessor functions instead of tvb_get_ptr()
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37320
f5534014-38df-0310-8fa8-
9805f1628bb7
morriss [Fri, 20 May 2011 03:12:56 +0000 (03:12 +0000)]
Use tvb_ether_to_str()
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37319
f5534014-38df-0310-8fa8-
9805f1628bb7
guy [Fri, 20 May 2011 02:44:12 +0000 (02:44 +0000)]
Fix annoying typo.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37318
f5534014-38df-0310-8fa8-
9805f1628bb7
morriss [Fri, 20 May 2011 02:42:14 +0000 (02:42 +0000)]
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.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37317
f5534014-38df-0310-8fa8-
9805f1628bb7
guy [Fri, 20 May 2011 02:35:05 +0000 (02:35 +0000)]
In at least some versions of OS X SystemStarter gets cranky if the
startup item is group-writable.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37316
f5534014-38df-0310-8fa8-
9805f1628bb7
guy [Thu, 19 May 2011 21:11:44 +0000 (21:11 +0000)]
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?)
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37314
f5534014-38df-0310-8fa8-
9805f1628bb7
gerald [Thu, 19 May 2011 21:09:29 +0000 (21:09 +0000)]
Update the User's Guide.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37312
f5534014-38df-0310-8fa8-
9805f1628bb7
tuexen [Thu, 19 May 2011 20:48:58 +0000 (20:48 +0000)]
Drop privileges after opening all pcap devices, not after the first one...
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37311
f5534014-38df-0310-8fa8-
9805f1628bb7
gerald [Thu, 19 May 2011 20:37:36 +0000 (20:37 +0000)]
1.6 → 1.7.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37309
f5534014-38df-0310-8fa8-
9805f1628bb7
wmeier [Thu, 19 May 2011 20:21:08 +0000 (20:21 +0000)]
Don't assign to an unused 'proto item *': Coverity 1033 & 906
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37308
f5534014-38df-0310-8fa8-
9805f1628bb7
tuexen [Thu, 19 May 2011 20:20:35 +0000 (20:20 +0000)]
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.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37307
f5534014-38df-0310-8fa8-
9805f1628bb7
wmeier [Thu, 19 May 2011 20:16:10 +0000 (20:16 +0000)]
Create subtree under the correct parent tree: Coverity 1138.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37306
f5534014-38df-0310-8fa8-
9805f1628bb7
wmeier [Thu, 19 May 2011 20:13:54 +0000 (20:13 +0000)]
Don't assign a value which is never used to a 'proto item *': Covertiy 1135;
Remove some unneeded #includes.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37305
f5534014-38df-0310-8fa8-
9805f1628bb7
gerald [Thu, 19 May 2011 20:13:33 +0000 (20:13 +0000)]
Fix markup.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37304
f5534014-38df-0310-8fa8-
9805f1628bb7
gerald [Thu, 19 May 2011 19:52:56 +0000 (19:52 +0000)]
Update the version and command output.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37303
f5534014-38df-0310-8fa8-
9805f1628bb7
wmeier [Thu, 19 May 2011 19:48:39 +0000 (19:48 +0000)]
#if 0 some not currently used code: Fixes Coverity 966 [UNUSED]
Convert apparent "4 space tabs" to "4 spaces".
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37302
f5534014-38df-0310-8fa8-
9805f1628bb7
wmeier [Thu, 19 May 2011 19:22:49 +0000 (19:22 +0000)]
Remove unneeded variable and code; Rename a function arg; Fix usage of _U_: Fixes Coverity 1146 [UNUSED]
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37301
f5534014-38df-0310-8fa8-
9805f1628bb7
gerald [Thu, 19 May 2011 19:03:22 +0000 (19:03 +0000)]
FOP 0.95 → 1.0.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37300
f5534014-38df-0310-8fa8-
9805f1628bb7
gerald [Thu, 19 May 2011 18:10:21 +0000 (18:10 +0000)]
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.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37298
f5534014-38df-0310-8fa8-
9805f1628bb7
guy [Thu, 19 May 2011 16:22:21 +0000 (16:22 +0000)]
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.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37297
f5534014-38df-0310-8fa8-
9805f1628bb7
tuexen [Thu, 19 May 2011 16:11:02 +0000 (16:11 +0000)]
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.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37296
f5534014-38df-0310-8fa8-
9805f1628bb7
tuexen [Thu, 19 May 2011 15:38:44 +0000 (15:38 +0000)]
Fix compilation.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37295
f5534014-38df-0310-8fa8-
9805f1628bb7
etxrab [Thu, 19 May 2011 15:23:19 +0000 (15:23 +0000)]
Fix a typo.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37294
f5534014-38df-0310-8fa8-
9805f1628bb7
wmeier [Thu, 19 May 2011 14:48:05 +0000 (14:48 +0000)]
Don't assign a value to a variable which is then not used: Coverity 1145
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37293
f5534014-38df-0310-8fa8-
9805f1628bb7
etxrab [Thu, 19 May 2011 14:41:58 +0000 (14:41 +0000)]
Subdissect 2 3GPP AVP:s that are actually text.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37292
f5534014-38df-0310-8fa8-
9805f1628bb7
wmeier [Thu, 19 May 2011 14:32:14 +0000 (14:32 +0000)]
Don't assign to an unused variable; remove variable definition: Coverity 1144.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37291
f5534014-38df-0310-8fa8-
9805f1628bb7
wmeier [Thu, 19 May 2011 14:11:01 +0000 (14:11 +0000)]
Don't assign a value to a variable which is then never used: Coverity 1160 [UNUSED]
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37290
f5534014-38df-0310-8fa8-
9805f1628bb7
tuexen [Thu, 19 May 2011 12:19:03 +0000 (12:19 +0000)]
When capturing from multiple interfaces, indicate this in the window
title of the wireshark main window.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37289
f5534014-38df-0310-8fa8-
9805f1628bb7
tuexen [Thu, 19 May 2011 12:14:27 +0000 (12:14 +0000)]
Remove unused define.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37288
f5534014-38df-0310-8fa8-
9805f1628bb7
tuexen [Thu, 19 May 2011 11:58:18 +0000 (11:58 +0000)]
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.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37287
f5534014-38df-0310-8fa8-
9805f1628bb7
guy [Thu, 19 May 2011 09:47:03 +0000 (09:47 +0000)]
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.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37286
f5534014-38df-0310-8fa8-
9805f1628bb7
tuexen [Thu, 19 May 2011 05:51:52 +0000 (05:51 +0000)]
Actually use the defaults from the last commit message.
... and switch off debug output.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37285
f5534014-38df-0310-8fa8-
9805f1628bb7
tuexen [Thu, 19 May 2011 05:49:46 +0000 (05:49 +0000)]
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.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37284
f5534014-38df-0310-8fa8-
9805f1628bb7
jmayer [Thu, 19 May 2011 05:46:07 +0000 (05:46 +0000)]
Fix warnings about unused variables in UI_MANAGER code.
Move gsm_a_stat.c back to clean dissectors.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37283
f5534014-38df-0310-8fa8-
9805f1628bb7
jmayer [Thu, 19 May 2011 05:44:32 +0000 (05:44 +0000)]
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?
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37282
f5534014-38df-0310-8fa8-
9805f1628bb7
jmayer [Thu, 19 May 2011 05:08:42 +0000 (05:08 +0000)]
Fix another old style function declaration warning
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37281
f5534014-38df-0310-8fa8-
9805f1628bb7
tuexen [Thu, 19 May 2011 05:00:28 +0000 (05:00 +0000)]
When capturing from multiple interfaces, indicate this in the
capture info dialog.
This is based on wirk from Irene Ruengeler.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37280
f5534014-38df-0310-8fa8-
9805f1628bb7
tuexen [Thu, 19 May 2011 04:56:04 +0000 (04:56 +0000)]
Fix the capture info dialog. Reset the EOF state such that the
number get actually updated.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37279
f5534014-38df-0310-8fa8-
9805f1628bb7
jmayer [Thu, 19 May 2011 04:42:57 +0000 (04:42 +0000)]
Fix an old style function declaration warning.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37278
f5534014-38df-0310-8fa8-
9805f1628bb7
jmayer [Thu, 19 May 2011 04:35:04 +0000 (04:35 +0000)]
Update CMakeLists to follow commit 37219.
Remove a duplicate check in configure.in
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37277
f5534014-38df-0310-8fa8-
9805f1628bb7
tuexen [Thu, 19 May 2011 04:30:05 +0000 (04:30 +0000)]
Move the debug output for the generated command line arguments to the correct place.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37276
f5534014-38df-0310-8fa8-
9805f1628bb7
wmeier [Thu, 19 May 2011 01:42:49 +0000 (01:42 +0000)]
For old times' sake: Fix bad dissection of Ethernet Router Hello Messages: Coverity 876 & 877 [UNUSED]
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37275
f5534014-38df-0310-8fa8-
9805f1628bb7
martinm [Thu, 19 May 2011 01:21:48 +0000 (01:21 +0000)]
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.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37274
f5534014-38df-0310-8fa8-
9805f1628bb7
martinm [Thu, 19 May 2011 01:09:05 +0000 (01:09 +0000)]
Only declare a variable if its going to be used.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37273
f5534014-38df-0310-8fa8-
9805f1628bb7
martinm [Thu, 19 May 2011 01:05:26 +0000 (01:05 +0000)]
Improve AM sequence analysis, check ACK looks sensible compared with latest SN received in opposite direction.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37272
f5534014-38df-0310-8fa8-
9805f1628bb7
martinm [Thu, 19 May 2011 01:03:56 +0000 (01:03 +0000)]
Set subheader root length to 1, so if contentsinvalid will already be correctl length.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37271
f5534014-38df-0310-8fa8-
9805f1628bb7
wmeier [Wed, 18 May 2011 23:24:50 +0000 (23:24 +0000)]
Don't create 2 unneeded subtrees; Fixes Coverity 879 & 880 [UNUSED];
#include <stdio.h> & <time.h> not needed.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37269
f5534014-38df-0310-8fa8-
9805f1628bb7
wmeier [Wed, 18 May 2011 23:09:18 +0000 (23:09 +0000)]
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.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37268
f5534014-38df-0310-8fa8-
9805f1628bb7
wmeier [Wed, 18 May 2011 23:01:05 +0000 (23:01 +0000)]
Create 2 subtrees using the correct 'proto_item *'; Fixes Coverity 900 & 901 [UNUSED];
#inclide <stdlib.h> not req'd.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37267
f5534014-38df-0310-8fa8-
9805f1628bb7
wmeier [Wed, 18 May 2011 22:49:33 +0000 (22:49 +0000)]
#include <stdlib.h> not req'd;
Reformat hf[] definition;
Misc whitespace cleanup;
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37266
f5534014-38df-0310-8fa8-
9805f1628bb7
wmeier [Wed, 18 May 2011 22:01:29 +0000 (22:01 +0000)]
Add a missing creation of a subtree: Fixes Coverity 907 [UNUSED];
Don't create an unused subtree: Fixes Coverity 908 [UNUSED];
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37265
f5534014-38df-0310-8fa8-
9805f1628bb7
darkjames [Wed, 18 May 2011 21:33:05 +0000 (21:33 +0000)]
Fix tvb_get_bits_buf for LSB.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37264
f5534014-38df-0310-8fa8-
9805f1628bb7
wmeier [Wed, 18 May 2011 21:20:22 +0000 (21:20 +0000)]
(Trivial) Fix a typo; Add spme punctuation in a coment.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37263
f5534014-38df-0310-8fa8-
9805f1628bb7
morriss [Wed, 18 May 2011 21:07:59 +0000 (21:07 +0000)]
Fix some (value) mistakes in value_strings. Found by Martin's patch on the -dev list.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37262
f5534014-38df-0310-8fa8-
9805f1628bb7
morriss [Wed, 18 May 2011 20:56:03 +0000 (20:56 +0000)]
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.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37261
f5534014-38df-0310-8fa8-
9805f1628bb7
morriss [Wed, 18 May 2011 20:26:55 +0000 (20:26 +0000)]
Fix a (value) mistake in a value_string. Found by Martin's patch on the -dev list.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37260
f5534014-38df-0310-8fa8-
9805f1628bb7
morriss [Wed, 18 May 2011 20:06:17 +0000 (20:06 +0000)]
Fix a couple (value) mistakes in value_strings. Found by Martin's patch on the -dev list.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37259
f5534014-38df-0310-8fa8-
9805f1628bb7
guy [Wed, 18 May 2011 19:29:47 +0000 (19:29 +0000)]
We don't have the ability to set the capture buffer size on all platforms.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37255
f5534014-38df-0310-8fa8-
9805f1628bb7
darkjames [Wed, 18 May 2011 18:49:33 +0000 (18:49 +0000)]
- Optimize tvb_get_bits_buf
- Add lsb0 (read: least significant bit is 0) parameter to tvb_get_bits_buf()
(not tested, might be buggy...)
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37254
f5534014-38df-0310-8fa8-
9805f1628bb7
tuexen [Wed, 18 May 2011 18:35:57 +0000 (18:35 +0000)]
Don't print too many interface names...
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37253
f5534014-38df-0310-8fa8-
9805f1628bb7
tuexen [Wed, 18 May 2011 18:04:50 +0000 (18:04 +0000)]
Add support for multiple interfaces for the -L option.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37250
f5534014-38df-0310-8fa8-
9805f1628bb7
tuexen [Wed, 18 May 2011 18:03:58 +0000 (18:03 +0000)]
Support multiple interfaces when used with -L option.
Show multiple interfaces when capturing from them.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37249
f5534014-38df-0310-8fa8-
9805f1628bb7
tuexen [Wed, 18 May 2011 18:02:30 +0000 (18:02 +0000)]
Use the interface specific parameter array.
This enables the capturing from mulitple interfaces using
tshark and wireshark (at the command line).
(tshark -i lo0 -i en0) or (wireshark -k -i lo0 -i en0)
works. You can capture from multiple remote and local
interfaces.
Based on work from Irene Ruengeler.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37248
f5534014-38df-0310-8fa8-
9805f1628bb7
morriss [Wed, 18 May 2011 17:31:48 +0000 (17:31 +0000)]
Finish removing tabs and fix up some indentation.
Add a break after decoding CSData Codec Type.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37247
f5534014-38df-0310-8fa8-
9805f1628bb7
wmeier [Wed, 18 May 2011 17:19:42 +0000 (17:19 +0000)]
Mark a function arg as _U_
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37246
f5534014-38df-0310-8fa8-
9805f1628bb7
wmeier [Wed, 18 May 2011 17:17:08 +0000 (17:17 +0000)]
Do PROTO_ITEM_SET_GENERATED for two 'proto_item *': Fixes Coverity 914 & 915 [UNUSED]
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37245
f5534014-38df-0310-8fa8-
9805f1628bb7
morriss [Wed, 18 May 2011 17:09:55 +0000 (17:09 +0000)]
Add a default (else) case in the default case for cause values to avoid passing
a NULL string pointer to format routines.
While we're there, update the cause list from 48.008 v10.1.0.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37244
f5534014-38df-0310-8fa8-
9805f1628bb7
wmeier [Wed, 18 May 2011 16:52:45 +0000 (16:52 +0000)]
Don't assign to an unused 'proto_item *': Coverity 939 [UNUSED];
Don't create a subtree which is never used: Coverity 940 [UNUSED].
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37243
f5534014-38df-0310-8fa8-
9805f1628bb7
wmeier [Wed, 18 May 2011 16:19:15 +0000 (16:19 +0000)]
Use subtree rather than parent tree to add items; Fixes Coverity 973 & 974;
#include <stdlib.h> not needed;
Minor whitespace cleanup.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37242
f5534014-38df-0310-8fa8-
9805f1628bb7
cmaynard [Wed, 18 May 2011 15:56:46 +0000 (15:56 +0000)]
Per destination flags is a 1-byte field. Grab it with tvb_get_guint8(), not
with tvb_get_letohs(). (Bug surfaced while trying out MESH_OVERRIDES.)
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37241
f5534014-38df-0310-8fa8-
9805f1628bb7
alagoutte [Wed, 18 May 2011 15:34:34 +0000 (15:34 +0000)]
From discussion in dev mailing ( wireshark.org/lists/wireshark-dev/201105/msg00235.html )
* Fix a duplicate value_string in cp_version table
* Fix a typo error (Cersion => Version)
Also fix some whitespaces/4tab-spaces (Need more work...)
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37240
f5534014-38df-0310-8fa8-
9805f1628bb7
morriss [Wed, 18 May 2011 15:33:07 +0000 (15:33 +0000)]
Fix https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5930 :
with a variant of the patch submitted by Alexander Stein on the -dev list:
Allow FT_FRAMENUMs to be added (as base BASE_NONE). My change from Alexander's
patch is to disallow BASE_NONE for non-framenum integers.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37239
f5534014-38df-0310-8fa8-
9805f1628bb7
rbalint [Wed, 18 May 2011 13:08:02 +0000 (13:08 +0000)]
Refer to "Capture Help" as "Capture Help below" when there are no interfaces
to capture on.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37238
f5534014-38df-0310-8fa8-
9805f1628bb7
stig [Wed, 18 May 2011 08:10:37 +0000 (08:10 +0000)]
Removed "Copy" from the profiles popup menu in favour of a "Create from"
dropdown in the "Create New Profile" window.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37237
f5534014-38df-0310-8fa8-
9805f1628bb7
guy [Wed, 18 May 2011 07:20:45 +0000 (07:20 +0000)]
You cannot just make the "len" field of a GByteArray larger, if there's
no data to back that length; you can only make it smaller.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37236
f5534014-38df-0310-8fa8-
9805f1628bb7
guy [Wed, 18 May 2011 03:28:26 +0000 (03:28 +0000)]
Make some more routines not used outside epan/proto.c static.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37235
f5534014-38df-0310-8fa8-
9805f1628bb7
wmeier [Wed, 18 May 2011 01:37:06 +0000 (01:37 +0000)]
Convert tabs to "4 spaces" since it appears that tabs are "4 space tabs";
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@37234
f5534014-38df-0310-8fa8-
9805f1628bb7