Commit Graph

35049 Commits

Author SHA1 Message Date
Anders Broman 312ad4e850 From Mathieu Parent:
- use 4-bytes hex for messages ids
- reorder skinny messages ids
- remove obsolete C-comments about status
- rewrite field names (from camel case to plain words)

From me:
Replaced meaningless hf blurbs with NULL

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

svn path=/trunk/; revision=36638
2011-04-14 06:07:42 +00:00
Anders Broman 7e17419e6c From Mathieu Parent:
packet-skinny:Add DialedPhoneBookMessage and DialedPhoneBookAckMessage

Replaced meaningless hf blurbs with NULL

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

svn path=/trunk/; revision=36637
2011-04-14 05:37:58 +00:00
Guy Harris 7fa71ab07e Check for _setmode() failing, Just In Case. Squelches some MSVC static
analyzer warnings.

Return an actual error if we're failing because we're trying to write to
the standard output in compressed mode.

svn path=/trunk/; revision=36636
2011-04-14 02:53:18 +00:00
Guy Harris c1b3e350ce Check for pkg-config's path as well, just in case it's installed
somewhere other than GLib (or if GLib isn't installed at all; that
configure script will fail in that case, but that's better than failing
because PKG_CHECK_MODULES wasn't defined, as that's a confusing failure
mode).

svn path=/trunk/; revision=36635
2011-04-14 02:49:20 +00:00
Chris Maynard ed054833be Mention that tshark can now display ICMP and ICMPv6 service response times.
svn path=/trunk/; revision=36634
2011-04-14 02:16:39 +00:00
Chris Maynard f627b9911a Via bug5774, Andrew Feren requested V9TEMPLATE_MAX_FIELDS be increased to 100.
svn path=/trunk/; revision=36633
2011-04-14 02:08:19 +00:00
Bill Meier b9784b4f21 From Solomon Peachy:Fix "Missing indication that we recognize the Radiotap MCS header field."
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5827

The radiotap header has a pile of optional fields, and uses a bitmask field to
indicate which fields are present.  Wireshark marks the fields it decodes, and
then decodes the payload.

The recently-added MCS field is decoded now, but the bit is not displayed as
recognized. This was presumably an oversight, as this bit is recognized and the
field decoded properly.

svn path=/trunk/; revision=36632
2011-04-13 22:07:53 +00:00
Bill Meier 9b36f03fc9 Don't assign to a proto_item * if the value won't be used: Coverity 1054-1056.
svn path=/trunk/; revision=36631
2011-04-13 21:27:08 +00:00
Bill Meier 8c4a4c09b9 Comment out code to create a subtree since it's (currently ?) unused: Coverity 1039;
Don't assign to a proto_item * if the value won't be used: Coverity 1040 & 1041.
Remove some unneede #includes;
Do some minor indentation & whitespace cleanup.

svn path=/trunk/; revision=36630
2011-04-13 21:00:44 +00:00
Chris Maynard 39013df13f Allow TCP segment data to be filterable via "tcp.data".
svn path=/trunk/; revision=36629
2011-04-13 18:15:41 +00:00
Gerald Combs dd2f52fd3c Update to reflect recent changes in epan/strutil.[ch].
svn path=/trunk/; revision=36628
2011-04-13 17:53:28 +00:00
Stephen Fisher 9246c41703 Change ws_strdup_escape_underscore() function to be more general, by
accepting any character as the escaped character.  Change existing
uses to use '_' for the underscore escaping.


svn path=/trunk/; revision=36627
2011-04-13 16:56:24 +00:00
Gerald Combs 763603d3b6 capinfos and editcap need PYTHON_DIR when we have Python enabled.
svn path=/trunk/; revision=36626
2011-04-13 16:05:11 +00:00
Alexis La Goutte f163f3fd8c Add Tag Quiet (40) and IBSS DFS (41)
Enhance Tag ERP Information (42), TS Delay (43), TCLAS Processing (44)

* Replace proto_tree_add_uint*/boolean/text by proto_tree_add_item
* use ENC_* macro for encoding


svn path=/trunk/; revision=36625
2011-04-13 15:59:22 +00:00
Stephen Fisher fcfcfa347c Rollback r36614 and change dissect_SMUX_PDUs_PDU() to use previously
unused smux_tree instead of tree.  Per suggestion from Stig on -dev.


svn path=/trunk/; revision=36624
2011-04-13 15:48:24 +00:00
Anders Broman 25ab8f9594 Ok let's try GTK 2.22 on Win64 too - fix a typo.
svn path=/trunk/; revision=36623
2011-04-13 15:46:39 +00:00
Jeff Morriss 4f510bc54d Based on https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5821#c2 :
Move the setting of debug environment variables out of the while loop.

Export MALLOC_OPTIONS=AJ to increase memory allocation debug on FreeBSD (and
others).

Export a number of environment variables that MacOS looks at.

(I don't have easy access to either of these OS so this has not been tested.)

Add a comments for each entry explaining which OS uses it and what it does.

svn path=/trunk/; revision=36622
2011-04-13 15:36:05 +00:00
Jörg Mayer 0c484c8cf9 + Fix bug listed in CID 1182:
Move test out of switch statement.
 Remove unused variable.

Remove trailing whitespace.

Whitespace in that file is a mess.

svn path=/trunk/; revision=36621
2011-04-13 15:16:41 +00:00
Anders Broman 81e330976e Ok let's try GTK 2.22 on Win64 too.
svn path=/trunk/; revision=36620
2011-04-13 14:38:58 +00:00
Anders Broman 2d38d00656 Try out GTK 2.22
svn path=/trunk/; revision=36619
2011-04-13 13:42:41 +00:00
Anders Broman 8532a6f8af Restore defaults
svn path=/trunk/; revision=36618
2011-04-13 05:43:07 +00:00
Anders Broman efeaabdd9a Cast away size_t warnings on win64.
svn path=/trunk/; revision=36617
2011-04-13 04:53:02 +00:00
Anders Broman 3db53e54ba Build win64 with a modified zlib (google is your friend).
http://www.apachehaus.com/forum/index.php?action=printpage;topic=143.0

First problem: ZLIB build must be fixed for x64, otherwise there will be one unresolved external symbol later. Quick fix is to open build\win32\build_zlib.bat and insert this at line 51:
set ASM_OPTS=AS=ml64 LOC="-DASMV -DASMINF" OBJA="inffasx64.obj gvmat64.obj inffas8664.obj"
(info found in zlib\win32\Makefile.msc) and then open zlib\contrib\masmx64\inffas8664.c and prepend "../../" to four includes at the beginning.

svn path=/trunk/; revision=36616
2011-04-13 04:47:38 +00:00
Stephen Fisher 4b416d92d0 Fix Coverity 524: NEGATIVE_RETURNS
Coverity sees that two calls to tvb_length_remaining() with an offset
of 0 may return -1 (for an "out of bounds" condition), which would lead
to an assignment of -1 to an unsigned variable and then passing it
to functions further down that cannot accept negative values.

Use tvb_ensure_length_remaining() to throw an exception in a case of
out of bounds access (though unlikely with an offset of 0) instead of
tvb_length_remaining(), which would return -1.


svn path=/trunk/; revision=36615
2011-04-12 22:23:03 +00:00
Stephen Fisher ce2eea5b95 Fix Coverity 1047: UNUSED_VALUE in dissect_smux
svn path=/trunk/; revision=36614
2011-04-12 22:12:05 +00:00
Anders Broman a0e2d8ec0a Use the new label(It will still break on zlib as zlib 1.2.5 isn't used yet).
svn path=/trunk/; revision=36611
2011-04-12 21:09:34 +00:00
Stig Bjørlykke e98f5fc23d Cleanup ProcessError().
Coverity 865.

svn path=/trunk/; revision=36610
2011-04-12 21:00:26 +00:00
Gerald Combs 01e072411f Regenerate more ASN.1 dissectors for Coverity.
svn path=/trunk/; revision=36609
2011-04-12 20:45:25 +00:00
Gerald Combs ca01b3a876 Add x509if_frame_end() which is passed to register_frame_end_routine()
and which resets our global variables. Remove a boolean which was used
to track a char *; just check if the pointer is NULL instead. Fixes bugs
5754 and 5793, hopefully without breaking anything.

svn path=/trunk/; revision=36608
2011-04-12 20:39:48 +00:00
Stephen Fisher ef5823c210 Put back in one of the includes removed in r36606: ctype.h for isdigit()
function.


svn path=/trunk/; revision=36607
2011-04-12 20:29:42 +00:00
Bill Meier c85689f108 Don't assign to a proto_item * if the value won't be used: Coverity 1034;
Comment out two calls to tvb_)get_ephemeral_string since the result is never used: Coverity 1035;
Also:
 Remove some unneeded #includes;
 Use consistent indentation.

svn path=/trunk/; revision=36606
2011-04-12 20:19:10 +00:00
Stig Bjørlykke 1bb94ef56d Added a FALLTHRU comment to avoid a missing break warning.
Coverity 437.

svn path=/trunk/; revision=36605
2011-04-12 20:03:21 +00:00
Bill Meier 706a2fd0a4 Don't assign to a variable if the value won't be used: Coverity 1002 & 1003.
svn path=/trunk/; revision=36604
2011-04-12 19:59:26 +00:00
Bill Meier fa5a06231c Fix two instances of apparent use of incorrect tree: Coverity 982 & 983;
Remove an uneeded line of code: Coverity 984;
#include <string.h> not needed;
Use consistent indentation & do other whitespace cleanup.

svn path=/trunk/; revision=36603
2011-04-12 19:41:26 +00:00
Stig Bjørlykke b862371815 Removed some unused variables.
svn path=/trunk/; revision=36602
2011-04-12 19:06:51 +00:00
Chris Maynard 1914330274 Don't bother to strcpy() an empty string; just set buf[0] = '\0': Coverity 626
svn path=/trunk/; revision=36601
2011-04-12 18:43:44 +00:00
Chris Maynard 7b43d9c705 Use g_snprintf instead of sprintf: (Part of) Covertiy 665.
svn path=/trunk/; revision=36600
2011-04-12 18:29:52 +00:00
Guy Harris e5f767cb50 #if 0 out some unused variables; we may make use of them later, but this
keeps GCC 4.6.0 from complaining about them and failing to build with
-Werror, and may also squelch some Coverity (and other static analyzer)
complaints.

svn path=/trunk/; revision=36599
2011-04-12 18:21:05 +00:00
Chris Maynard ee36a5cfbe Use g_strlcat instead of strcat: Coverity 632.
svn path=/trunk/; revision=36598
2011-04-12 18:16:55 +00:00
Chris Maynard 5773340023 Use g_snprintf instead of sprintf and g_strlcat instead of strcat: Coverity 698
svn path=/trunk/; revision=36597
2011-04-12 17:57:22 +00:00
Bill Meier 8587721abb Fix nasm pathname: nasm-2.09-08\nasm.exe ==> nasm-2.09.08\nasm.exe
svn path=/trunk/; revision=36596
2011-04-12 17:53:23 +00:00
Chris Maynard f9fa7e8def Use g_snprintf instead of sprintf: Coverity 622.
Use g_strlcpy instead of strcpy: Coverity 632.

svn path=/trunk/; revision=36595
2011-04-12 17:52:52 +00:00
Bill Meier bbea8c7cf7 Don't assign to a proto_item * if the value won't be used: Coverity 978;
Use what appears to be the intended correct tree for various proto_add...: Coverity 979;
Minor indentation cleanup.

svn path=/trunk/; revision=36594
2011-04-12 17:44:22 +00:00
Guy Harris 92c974f2f7 Now that wtap_read() checks for delayed errors on EOF, there's no need
to check for it on close.

svn path=/trunk/; revision=36593
2011-04-12 17:37:05 +00:00
Bill Meier a90aff50bb Comment out some (currently ?) unused code: Coverity 969-971;
Fix up whitespace & indentation (e.g., "4 space tabs").

svn path=/trunk/; revision=36592
2011-04-12 16:48:58 +00:00
Guy Harris fd96236cdf From Jakub Zawadski: some small fixes.
From me: small indentation fix (use spaces rather than tabs in all lines
in that routine).

svn path=/trunk/; revision=36591
2011-04-12 16:31:16 +00:00
Guy Harris 91e1769ac5 Update and expand some comments.
In the end-of-stream code, when we're checking the CRC and length, don't
check the CRC or length if we failed to read them, and don't check the
length if the CRC is bad.

We define O_BINARY as 0 on UN*X in <wsutil/file_util.h>, so we don't
need to avoid using it on UN*X.

In file_gets(), check for delayed errors.

svn path=/trunk/; revision=36590
2011-04-12 16:21:14 +00:00
Bill Meier fe8caf9040 Don't assign to a proto_item * if the value won't be used: Coverity 954 & 955;
Remove some unneeded #includes;
Minor whitespace cleanup.

svn path=/trunk/; revision=36589
2011-04-12 16:12:03 +00:00
Bill Meier e9618c5548 Don't assign to a proto_item * if the value won't be used: Coverity 933;
Fix apparent use of incorrect tree: Coverity 934.

svn path=/trunk/; revision=36588
2011-04-12 16:00:28 +00:00
Bill Meier 81672fe2f4 Don't assign to a proto_item * if the value won't be used: Coverity 916-918;
Also: use consistent indentation.

svn path=/trunk/; revision=36587
2011-04-12 15:49:29 +00:00