Commit Graph

34971 Commits

Author SHA1 Message Date
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
Anders Broman b5125be726 Build with zlib-1.2.5 for win32
zlib for GTK hasd this comment:
/* LFS conventions have no meaning on Windows. Looking for feature
 * macros like _LARGEFILE64_SOURCE or _FILE_OFFSET_BITS on Windows is
 * wrong. So make sure any such macros misguidedly defined by the
 * user have no effect. Windows has large file support, but the
 * official zlib DLL has not been built to provide the 64-bit offset
 * APIs, sigh.  So we have just patched out the 64-bit offset API
 * from this header file.
 */

svn path=/trunk/; revision=36586
2011-04-12 15:44:05 +00:00
Bill Meier fc600d4bf9 Fix a few cases of proto_item_add_subtree() using the wrong tree: Coverity 916-918;
Also: Fix some indentation: e.g., "4 space tabs".

svn path=/trunk/; revision=36585
2011-04-12 15:39:19 +00:00
Martin Mathieson b76b05b75b A parameter isn't used if ZLIB isn't enabled.
svn path=/trunk/; revision=36584
2011-04-12 14:33:31 +00:00
Bill Meier e07cb22a47 Rework whitespace.
svn path=/trunk/; revision=36583
2011-04-12 14:33:30 +00:00
Alexis La Goutte 2e9f0857f6 Fix tshark doc (ip.src==1.2.3.4) is not a valid filter for ICMPv6 trafic ;)
svn path=/trunk/; revision=36582
2011-04-12 13:30:43 +00:00
Bill Meier c424bd7c83 Don't assign to a proto_item * if the value won't be used: Coverity 1158 & 1159.
svn path=/trunk/; revision=36581
2011-04-12 13:27:08 +00:00
Bill Meier e3f4dc17a9 localtime() can return a NULL ptr.
svn path=/trunk/; revision=36580
2011-04-12 13:21:32 +00:00
Stig Bjørlykke 42f41ddf0b Corrected switch-case flow.
Coverity 1184.

svn path=/trunk/; revision=36579
2011-04-12 09:05:43 +00:00
Guy Harris bbcb51276a If we hit an EOF - wth->subtype_read returns FALSE, but sets *err to 0 -
call file_error() to fetch any deferred error, so we report an error
rather than an "everything's OK" EOF.

svn path=/trunk/; revision=36578
2011-04-12 03:27:10 +00:00
Guy Harris b28ee8b331 From Jakub Zawadzki: speed up random access to gzipped files, as per the
zran.c example in the zlib source.

This means that problems in the file's contents might not be reported
when a packet is read, as long as there's no problem in the contents of
the file up to the last bit of compressed data for the packet; we now
check for errors after finishing the sequential read of the file, at
least in some programs, so that shouldn't be an issue (the other
programs need to be changed to do so as well).  This is necessary in
order to be able to read all the packets we saw in the sequential pass;
it also lets us get a few more packets from truncated files in some
cases.

svn path=/trunk/; revision=36577
2011-04-12 02:40:14 +00:00
Guy Harris e5fe0128d7 Allow wtap_sequential_close() and wtap_close() to return an error; this
may happen if, when reading a compressed file, we find an error in the
file's contents past the last packet (e.g., the file being cut short so
that we can't get a full buffer worth of compressed data), and that
reporting of that error is delayed (so that you can get all of the
packets that we *can* decompress).  Check for those errors, at least on
the sequential read pass (the only errors we should see when closing the
random stream are errors we've already seen in the sequential stream).

svn path=/trunk/; revision=36576
2011-04-12 01:36:12 +00:00
Guy Harris 2b8ebd389b "This file format can't be written to a pipe" and "this file format
can't be saved in compress form" are both equivalent to "this file file
format requires seeking when writing it".  Change the "can compress"
Boolean in the file format table to "writing requires seeking", give all
the entries the proper value, and do the checks for attempting to write
a file format to a pipe or write it in compressed format to common code.

This means we don't need to pass the "can't seek" flag to the dump open
routines.

svn path=/trunk/; revision=36575
2011-04-12 00:44:44 +00:00
Gerald Combs 793d683575 Regenerate the ANSI MAP, LDAP, and TETRA dissectors so that Coverity can
locate their template files.

svn path=/trunk/; revision=36569
2011-04-11 23:07:53 +00:00
Guy Harris f5f74bed3d Use ws_open(), not open(), so we handle UTF-8 pathnames on Windows.
Update or remove some additional "we don't have ferror() in zlib"
comments to reflect the current reality.

svn path=/trunk/; revision=36568
2011-04-11 22:19:01 +00:00
Guy Harris 35c66a1745 Remove a no-longer-valid comment (we're not using zlib's I/O routines,
and the routines we're now using guarantee that an error has occured in
that case).

svn path=/trunk/; revision=36567
2011-04-11 22:11:44 +00:00
Martin Mathieson e7a23fb4fe Don't assign to a proto_item* that is not used before being overwritten.
Coverity 1017.

svn path=/trunk/; revision=36566
2011-04-11 21:40:49 +00:00
Martin Mathieson deac0b8af4 Show Segment offset number in AM header root item.
svn path=/trunk/; revision=36565
2011-04-11 21:34:47 +00:00
Martin Mathieson a8f1a17b68 - for UL/DL HARQ, show number of ms since previous tx
- for SR state-machine problems, show UEId in report
- incomplete beginnings of support for grouping/counting UEs per TTI

svn path=/trunk/; revision=36564
2011-04-11 21:33:33 +00:00
Guy Harris b184c69559 Don't use the zlib I/O routines for writing compressed files, either;
this frees us from worrying about zlib large file issues on the write
side, and also lets us clean up a few other things.

svn path=/trunk/; revision=36563
2011-04-11 21:33:22 +00:00
Chris Maynard 26e4e13c06 Neither checksum_item nor length are used uninitialized, but for some reason,
the OSX buildbots think they can be.  Initialize them to avoid a build failure.

svn path=/trunk/; revision=36562
2011-04-11 18:40:36 +00:00
Chris Maynard 5abfa49525 Add conversation tracking and tshark tap support to ICMPv6. Fixes bug 5810.
TODO: Add a Wireshark tap or look into possibly using the stats tree instead.
Also, like ICMP, the ICMPv6 payload appears to carry the sender's timestamp, so
it might be possible to make use of this information to estimate the total SRT.
(See bug 5770 for more details.)

svn path=/trunk/; revision=36561
2011-04-11 17:42:01 +00:00
Jörg Mayer d181b588b7 CID 898: Remove a not really used variable
svn path=/trunk/; revision=36560
2011-04-11 17:25:41 +00:00
Jörg Mayer a6691d79c6 Fix coverity CID 855: remove an unused variable
svn path=/trunk/; revision=36559
2011-04-11 17:21:15 +00:00
Bill Meier 0e95d4dbf5 Don't assign to a proto_item * if the value won't be used: Coverity 1036;
Fix what appears to be a use of an incorrect tree: Coverity 1037;
Also:
- Remove uneeded #includes & re-order #includes;
- Do indentation and whitespace cleanup (e.g., "4 space tabs").

svn path=/trunk/; revision=36558
2011-04-11 15:17:22 +00:00
Bill Meier 8d8043e4f6 Don't assign to a proto_item * if the value won't be used: Coverity 912 & 913.
svn path=/trunk/; revision=36557
2011-04-11 14:45:54 +00:00
Bill Meier 5f57b0d828 Don't assign to a variable if the value won't be used: Coverity 964 & 965;
Also:
- Move proto_register...() & proto_reg_handoff...() to the end of the file;
- Remove some unneeded #includes;
- Do some indentation and whitespace cleanup (e.g., "4-space tabs").

svn path=/trunk/; revision=36556
2011-04-11 14:41:14 +00:00
Bill Meier ea8b282a75 Don't assign to a proto_item * if the value won't be used: Coverity 928-932;
Also: remove some unneeded #includes.

svn path=/trunk/; revision=36555
2011-04-11 14:28:18 +00:00
Chris Maynard a460d35272 Add proto_field_is_referenced. Fixes bug 5816.
svn path=/trunk/; revision=36554
2011-04-11 13:44:31 +00:00