Stig Bjørlykke [Thu, 31 Mar 2011 12:14:59 +0000 (12:14 -0000)]
Added dissector_handle_get_long_name().
svn path=/trunk/; revision=36412
Stig Bjørlykke [Thu, 31 Mar 2011 11:01:06 +0000 (11:01 -0000)]
Use correct length for clientaddr4 callback address entry.
svn path=/trunk/; revision=36411
Chris Maynard [Wed, 30 Mar 2011 15:54:28 +0000 (15:54 -0000)]
Fix a couple of typos.
svn path=/trunk/; revision=36410
Chris Maynard [Wed, 30 Mar 2011 14:53:09 +0000 (14:53 -0000)]
From: wireshark.org/lists/wireshark-dev/201103/msg00157.html
Change RTT references to SRT. (This should have been checked in with r36297.)
svn path=/trunk/; revision=36409
Chris Maynard [Wed, 30 Mar 2011 14:44:18 +0000 (14:44 -0000)]
Wrap all text lines of help at column 80.
svn path=/trunk/; revision=36408
Ronnie Sahlberg [Wed, 30 Mar 2011 04:36:09 +0000 (04:36 -0000)]
remove two pointless const in casts
coverity 580 and 581
svn path=/trunk/; revision=36407
Ronnie Sahlberg [Wed, 30 Mar 2011 03:39:58 +0000 (03:39 -0000)]
remove some assignments we dont use
svn path=/trunk/; revision=36406
Chris Maynard [Tue, 29 Mar 2011 21:40:54 +0000 (21:40 -0000)]
Change TCP_PORT_MANGO to TCP_PORT_MONGO.
svn path=/trunk/; revision=36405
Jeff Morriss [Tue, 29 Mar 2011 21:39:15 +0000 (21:39 -0000)]
Don't put a 65536 byte variable on the stack: instead, malloc/free it.
Coverity 760-761.
svn path=/trunk/; revision=36404
Anders Broman [Tue, 29 Mar 2011 21:20:36 +0000 (21:20 -0000)]
Add a missing break.
CID 487
svn path=/trunk/; revision=36403
Anders Broman [Tue, 29 Mar 2011 21:11:26 +0000 (21:11 -0000)]
Add a missing break.
CID 488
svn path=/trunk/; revision=36402
Anders Broman [Tue, 29 Mar 2011 21:08:34 +0000 (21:08 -0000)]
Add a missing break.
CID 490
svn path=/trunk/; revision=36401
Anders Broman [Tue, 29 Mar 2011 21:05:29 +0000 (21:05 -0000)]
Added FALLTHRU comment to avoid a "missing break" warning.
CID 489
svn path=/trunk/; revision=36400
Chris Maynard [Tue, 29 Mar 2011 20:58:57 +0000 (20:58 -0000)]
From Alexis La Goutte via bug 5790: Add partial bit flag in query.
svn path=/trunk/; revision=36399
Chris Maynard [Tue, 29 Mar 2011 20:05:33 +0000 (20:05 -0000)]
Grab URL from frame info value instead of from tvb. Fixes bug 5780.
svn path=/trunk/; revision=36398
Jeff Morriss [Tue, 29 Mar 2011 14:29:45 +0000 (14:29 -0000)]
Don't put a 196808 byte variable on the stack: instead, malloc/free it.
Coverity 789-790.
Since we've been keeping track of how many bytes we put in the buffer,
use that value instead of calling strlen() find it again.
Also, some white space/indentation cleanup.
svn path=/trunk/; revision=36397
Anders Broman [Tue, 29 Mar 2011 14:23:21 +0000 (14:23 -0000)]
Remove check_col().
svn path=/trunk/; revision=36396
Martin Mathieson [Tue, 29 Mar 2011 14:23:13 +0000 (14:23 -0000)]
Don't assign proto_item pointers that are not used.
Coverity 1174
svn path=/trunk/; revision=36395
Jeff Morriss [Tue, 29 Mar 2011 14:05:28 +0000 (14:05 -0000)]
Don't put a 65536 byte variable on the stack: instead, malloc/free it.
Coverity 791-794.
Also, some white space/indentation cleanup.
svn path=/trunk/; revision=36394
Stig Bjørlykke [Tue, 29 Mar 2011 11:09:59 +0000 (11:09 -0000)]
Guard against NULL return from find_protocol_by_id().
svn path=/trunk/; revision=36393
Stig Bjørlykke [Tue, 29 Mar 2011 11:08:15 +0000 (11:08 -0000)]
From Pascal Quantin:
Added guard against proto_name's not resolved as protocols.
svn path=/trunk/; revision=36392
Tomas Kukosa [Tue, 29 Mar 2011 10:56:30 +0000 (10:56 -0000)]
RTP: display payload type name from dynamic payload type table
H.223: use just one protocol for bot dissectors for more clear protocol disabling
svn path=/trunk/; revision=36391
Jeff Morriss [Tue, 29 Mar 2011 01:27:20 +0000 (01:27 -0000)]
Prohibit (memory-unsafe) g_sprintf() and g_vsprintf().
svn path=/trunk/; revision=36390
Chris Maynard [Mon, 28 Mar 2011 22:22:05 +0000 (22:22 -0000)]
Cast to guint to avoid a Windows compiler warning.
svn path=/trunk/; revision=36389
Chris Maynard [Mon, 28 Mar 2011 22:12:53 +0000 (22:12 -0000)]
Move -D option into "pre-scan" group as there's no need to start gtk+ for it.
svn path=/trunk/; revision=36388
Chris Maynard [Mon, 28 Mar 2011 22:10:41 +0000 (22:10 -0000)]
Use fprintf_stderr() instead of printf() so "wireshark -D" works (on Windows).
svn path=/trunk/; revision=36387
Stephen Fisher [Mon, 28 Mar 2011 20:53:20 +0000 (20:53 -0000)]
Fix for bug #4334 "VNC Hextile encoding is incomplete" from Yaniv Kaul.
svn path=/trunk/; revision=36386
Jeff Morriss [Mon, 28 Mar 2011 19:44:41 +0000 (19:44 -0000)]
Use g_ascii_strdown instead of g_strdown
svn path=/trunk/; revision=36385
Guy Harris [Mon, 28 Mar 2011 17:23:06 +0000 (17:23 -0000)]
More updates to the wtap_open_offline() comment.
svn path=/trunk/; revision=36384
Chris Maynard [Mon, 28 Mar 2011 17:04:58 +0000 (17:04 -0000)]
Don't use prohibited strncpy(). Use g_strlcpy() instead. Ensures that "orig"
will be NULL-terminated, even if len >= SCS_HUGE_SIZE.
svn path=/trunk/; revision=36383
Chris Maynard [Mon, 28 Mar 2011 16:42:25 +0000 (16:42 -0000)]
Don't use prohibited strncpy(). In this case, use g_strlcat() instead.
svn path=/trunk/; revision=36382
Jeff Morriss [Mon, 28 Mar 2011 15:28:03 +0000 (15:28 -0000)]
From Alexis La Goutte via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5522 :
Fix a Clang warning, add email address, and add some stuff to COL_INFO.
svn path=/trunk/; revision=36381
Jeff Morriss [Mon, 28 Mar 2011 15:23:27 +0000 (15:23 -0000)]
From Stephane Bryant via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5787 :
Register the reload-framing dissector by name. Needed when RELOAD is not used
over udp/tcp (but tls, for example)
svn path=/trunk/; revision=36380
Jeff Morriss [Mon, 28 Mar 2011 15:20:16 +0000 (15:20 -0000)]
From Alexis La Goutte via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5768 :
Patch 5
Enchance MySQL :
* Add expert info of when undecoded data
* Fix some white space
svn path=/trunk/; revision=36379
Chris Maynard [Mon, 28 Mar 2011 14:53:35 +0000 (14:53 -0000)]
Make ip.fragments an FT_BYTES field so a PDML Export will display the bytes in
the value="..." attribute. Fixes bug 1520.
svn path=/trunk/; revision=36378
Stig Bjørlykke [Mon, 28 Mar 2011 14:18:29 +0000 (14:18 -0000)]
Added payload, header and trailer information to the DLT tree entry.
svn path=/trunk/; revision=36377
Anders Broman [Mon, 28 Mar 2011 13:09:47 +0000 (13:09 -0000)]
Update a comment.
svn path=/trunk/; revision=36376
Guy Harris [Sun, 27 Mar 2011 17:26:21 +0000 (17:26 -0000)]
If there are more than 4 billion characters in the printable form of a
network address, we have bigger problems. Cast the result of strlen()
to int to squelch a compiler warning with MSVC++.
svn path=/trunk/; revision=36375
Bill Meier [Sun, 27 Mar 2011 16:15:19 +0000 (16:15 -0000)]
Fix Coverity 393: "DEADCODE"; Add an XXX comment
svn path=/trunk/; revision=36373
Gerald Combs [Sun, 27 Mar 2011 16:06:03 +0000 (16:06 -0000)]
Add "-z hosts", which dumps name resolution information in hosts format.
svn path=/trunk/; revision=36372
Gerald Combs [Sun, 27 Mar 2011 15:58:52 +0000 (15:58 -0000)]
Add the IPv4 address, not the first 4 bytes of its name, to the
resolution table.
svn path=/trunk/; revision=36370
Bill Meier [Sun, 27 Mar 2011 15:39:06 +0000 (15:39 -0000)]
1. Fix bug in dissection of a field; Add two missing lines of code.
2. #if 0 if test "which will never be true"; Add XXX comment
svn path=/trunk/; revision=36369
Gerald Combs [Sun, 27 Mar 2011 15:03:43 +0000 (15:03 -0000)]
[Automatic manuf, services and enterprise-numbers update for 2011-03-27]
svn path=/trunk/; revision=36368
Bill Meier [Sun, 27 Mar 2011 14:34:19 +0000 (14:34 -0000)]
Removed uneeded 'continue'.
Fixes Coverity 815 ("UNREACHABLE").
svn path=/trunk/; revision=36367
Stig Bjørlykke [Sun, 27 Mar 2011 14:11:40 +0000 (14:11 -0000)]
Added a FALLTHRU comment to avoid a "missing break" warning.
Coverity 472.
svn path=/trunk/; revision=36366
Anders Broman [Sun, 27 Mar 2011 14:02:42 +0000 (14:02 -0000)]
Added FALLTHRU comment to avoid a "missing break" warning.
svn path=/trunk/; revision=36365
Stig Bjørlykke [Sun, 27 Mar 2011 13:56:53 +0000 (13:56 -0000)]
Added a missing break.
Coverity 452.
svn path=/trunk/; revision=36364
Stig Bjørlykke [Sun, 27 Mar 2011 13:54:53 +0000 (13:54 -0000)]
Added a missing break.
Coverity 461.
svn path=/trunk/; revision=36363
Stig Bjørlykke [Sun, 27 Mar 2011 13:51:47 +0000 (13:51 -0000)]
Added some missing breaks.
Coverity 462 & 463.
svn path=/trunk/; revision=36362
Stig Bjørlykke [Sun, 27 Mar 2011 13:40:54 +0000 (13:40 -0000)]
Added some missing breaks.
Coverity 443 & 444.
svn path=/trunk/; revision=36361
Stig Bjørlykke [Sun, 27 Mar 2011 13:35:30 +0000 (13:35 -0000)]
Added some missing breaks.
Coverity 447.
svn path=/trunk/; revision=36360
Stig Bjørlykke [Sun, 27 Mar 2011 13:33:28 +0000 (13:33 -0000)]
Added a FALLTHRU comment to avoid a "missing break" warning.
Coverity 458.
svn path=/trunk/; revision=36359
Stig Bjørlykke [Sun, 27 Mar 2011 13:30:39 +0000 (13:30 -0000)]
Added some missing breaks.
Coverity 466.
svn path=/trunk/; revision=36358
Stig Bjørlykke [Sun, 27 Mar 2011 13:16:50 +0000 (13:16 -0000)]
Added a missing break.
Coverity 491.
svn path=/trunk/; revision=36357
Stig Bjørlykke [Sun, 27 Mar 2011 13:15:14 +0000 (13:15 -0000)]
Added a missing break.
Coverity 492.
svn path=/trunk/; revision=36356
Bill Meier [Sat, 26 Mar 2011 16:39:30 +0000 (16:39 -0000)]
Don't assign to a proto_item* if the value won't be used: Coverity 895-897
Also: Fix some indentation & whitespace.
svn path=/trunk/; revision=36355
Jaap Keuter [Sat, 26 Mar 2011 16:07:30 +0000 (16:07 -0000)]
From Alexis La Goutte:
A patch to add ATM over TCP Dissector.
The dissector dissect only the ATMTCP header (VCI, VPI, Payload Length)
The data are not yet dissect, it is necessary to add a "UAT" (As with the K12
dissector) to indicate the type (ILMI, AAL, ATM...) of data (based on VCI/VPI)
svn path=/trunk/; revision=36354
Stig Bjørlykke [Sat, 26 Mar 2011 15:59:57 +0000 (15:59 -0000)]
Added a sanity check to remove a coverity warning.
Coverity 537.
svn path=/trunk/; revision=36353
Stig Bjørlykke [Sat, 26 Mar 2011 15:53:55 +0000 (15:53 -0000)]
Added sanity checks for preferences.
Coverity 551 & 553.
svn path=/trunk/; revision=36352
Stig Bjørlykke [Sat, 26 Mar 2011 15:39:35 +0000 (15:39 -0000)]
Added a sanity check in output_fields_set_option.
Coverity 552.
svn path=/trunk/; revision=36351
Stig Bjørlykke [Sat, 26 Mar 2011 15:24:36 +0000 (15:24 -0000)]
Added a FALLTHRU comment to avoid a coverity warning.
Coverity 480.
svn path=/trunk/; revision=36350
Stig Bjørlykke [Sat, 26 Mar 2011 15:20:37 +0000 (15:20 -0000)]
Added a missing break in graph_select_segment.
Coverity 482.
svn path=/trunk/; revision=36349
Stig Bjørlykke [Sat, 26 Mar 2011 15:16:40 +0000 (15:16 -0000)]
Added a missing break in hostlist_get_filter_name.
Coverity 481.
svn path=/trunk/; revision=36348
Jaap Keuter [Sat, 26 Mar 2011 11:23:46 +0000 (11:23 -0000)]
From Hadar Shoham:
Add support to correlation ID under DHCPv6 according to
"CL-SP-CANN-DHCP-Reg-I06-110210"
DHCPv6 option 17 Vendor-specific Information, sub option 2172 (paragraph 5.4.3)
svn path=/trunk/; revision=36347
Ronnie Sahlberg [Sat, 26 Mar 2011 05:39:45 +0000 (05:39 -0000)]
remove a proto_item variable we never use
coverity 866
svn path=/trunk/; revision=36346
Ronnie Sahlberg [Sat, 26 Mar 2011 05:18:09 +0000 (05:18 -0000)]
remove a proto_item variable we dont need
coverity 1046
svn path=/trunk/; revision=36345
Jeff Morriss [Sat, 26 Mar 2011 02:39:04 +0000 (02:39 -0000)]
Introduce ep_strconcat (copied from the glib version)
svn path=/trunk/; revision=36344
Jeff Morriss [Sat, 26 Mar 2011 02:11:42 +0000 (02:11 -0000)]
Remove some duplicated string manipulation code.
svn path=/trunk/; revision=36343
Jeff Morriss [Sat, 26 Mar 2011 01:22:31 +0000 (01:22 -0000)]
Simplify the generation of the 'name' passed to tick_stat_node().
svn path=/trunk/; revision=36342
Jeff Morriss [Sat, 26 Mar 2011 01:21:43 +0000 (01:21 -0000)]
The 'name's passed to tick_stat_node() do not need to be static.
svn path=/trunk/; revision=36341
Gerald Combs [Fri, 25 Mar 2011 23:45:48 +0000 (23:45 -0000)]
Fix Coverity CIDs 810 & 811: uninitialized variables.
svn path=/trunk/; revision=36340
Stephen Fisher [Fri, 25 Mar 2011 22:35:29 +0000 (22:35 -0000)]
Fix Coverity 331, 332, 333: Wrap the three remaining
gtk_tree_selection_get_selected() calls with if(!..) return;
like the rest of them. These functions shouldn't be called
unless a packet was selected, so this call should always return
TRUE for success.
svn path=/trunk/; revision=36339
Jeff Morriss [Fri, 25 Mar 2011 22:20:27 +0000 (22:20 -0000)]
Reword the comments from rev 36304.
Replace the tabs in the file (mostly added by me) with spaces (for consistency).
svn path=/trunk/; revision=36338
Stephen Fisher [Fri, 25 Mar 2011 22:16:23 +0000 (22:16 -0000)]
Fix coverity 268, 269, 546, 547, 548, 549 by checking for null return
from p_get_proto_data() since it should have been set already by the
calling function. Any other case would be a bug, so call
DISSECTOR_ASSERT in those cases. Hopefully Coverity realizes that this
is a null return value check.
svn path=/trunk/; revision=36337
Bill Meier [Fri, 25 Mar 2011 22:13:33 +0000 (22:13 -0000)]
Don't assign to a proto_item* if the value won't be used: Coverity 941
svn path=/trunk/; revision=36336
Bill Meier [Fri, 25 Mar 2011 21:52:56 +0000 (21:52 -0000)]
Don't assign to a proto_item* if the value won't be used: Coverity 958-961.
svn path=/trunk/; revision=36335
Bill Meier [Fri, 25 Mar 2011 21:09:28 +0000 (21:09 -0000)]
Don't assign to a proto_item* if the value won't be used: Coverity 852,854;
Add a hidden item to the correct subtree: Coverity 853.
svn path=/trunk/; revision=36334
Jeff Morriss [Fri, 25 Mar 2011 20:04:54 +0000 (20:04 -0000)]
As suggested by Didier in https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=3055 :
Apply rev 25869 to some more dissectors.
svn path=/trunk/; revision=36333
Jeff Morriss [Fri, 25 Mar 2011 19:02:18 +0000 (19:02 -0000)]
From Didier via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=3055 :
Apply rev 25869 to most of the rest of the TCP-desegmenting dissectors.
(The SSL dissector was already updated in one of two spots with bug 4535/rev
32456.)
A couple of the patches had to be manually applied.
From me: Fix the comments to match the change (including in the TCP and SSL
dissectors.)
svn path=/trunk/; revision=36332
Chris Maynard [Fri, 25 Mar 2011 18:51:12 +0000 (18:51 -0000)]
Change urb_type from FT_STRING to FT_UINT8. This fixes the display filtering
for this field. This change was prompted because of the following query:
http://ask.wireshark.org/questions/3113/usburb_type-values
Also, remove 2 unused fields found by running tools/checkhf.pl.
svn path=/trunk/; revision=36331
Jeff Morriss [Fri, 25 Mar 2011 15:45:37 +0000 (15:45 -0000)]
Fix the problem reported in https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=3785#c14 :
If we thought we finished reassembly (and called the subdissector) only to find
out that the subdissector asked for more data, handle the case where the
subdissector asked for DESEGMENT_UNTIL_FIN. Previously we only handled the
possibility that the subdissector would ask for a specific number of bytes or
DESEGMENT_ONE_MORE_SEGMENT.
svn path=/trunk/; revision=36330
Ronnie Sahlberg [Fri, 25 Mar 2011 07:26:17 +0000 (07:26 -0000)]
remove two unused assignments
coverity 1050
svn path=/trunk/; revision=36329
Ronnie Sahlberg [Fri, 25 Mar 2011 07:22:23 +0000 (07:22 -0000)]
use the newly created tree to store the new items instead of the parent tree
coverity 842
svn path=/trunk/; revision=36328
Ronnie Sahlberg [Fri, 25 Mar 2011 07:19:15 +0000 (07:19 -0000)]
remove unused variable
coverity 229
svn path=/trunk/; revision=36327
Ronnie Sahlberg [Fri, 25 Mar 2011 07:15:48 +0000 (07:15 -0000)]
remove an unused variable
coverity 1044
svn path=/trunk/; revision=36326
Anders Broman [Fri, 25 Mar 2011 06:42:59 +0000 (06:42 -0000)]
Make it build on Windows 7
(pcapng.c(1817) : warning C4267: '=' : conversion from 'size_t' to 'gint', possible loss of data)
svn path=/trunk/; revision=36325
Ronnie Sahlberg [Fri, 25 Mar 2011 03:07:05 +0000 (03:07 -0000)]
remove assignment of an unused value
coverity 1178
svn path=/trunk/; revision=36324
Gerald Combs [Fri, 25 Mar 2011 00:28:47 +0000 (00:28 -0000)]
If a segment looks like a retransmission, don't mark it out-of-order.
svn path=/trunk/; revision=36323
Bill Meier [Thu, 24 Mar 2011 23:34:13 +0000 (23:34 -0000)]
Don't assign to a proto_item* if the value won't be used:
Also: Fix a bug wherein PPP_RTP_CUD8 was being registered
to "ppp.protocol" with the cudp16 handle rather than the cudp8 handle.
Coverity 1004-1013
svn path=/trunk/; revision=36322
Gerald Combs [Thu, 24 Mar 2011 23:33:38 +0000 (23:33 -0000)]
Add a missing include.
svn path=/trunk/; revision=36321
Bill Meier [Thu, 24 Mar 2011 23:05:36 +0000 (23:05 -0000)]
Fix a compile warning: unused variable
svn path=/trunk/; revision=36320
Gerald Combs [Thu, 24 Mar 2011 22:58:45 +0000 (22:58 -0000)]
Add an item about pcapng name resolution.
svn path=/trunk/; revision=36319
Gerald Combs [Thu, 24 Mar 2011 22:47:57 +0000 (22:47 -0000)]
Add initial pcapng name resolution record support. Wireshark has read
support; TShark has read+write support. Additionally TShark can read a
"hosts" file and write those records to a capture file.
This uses "struct addrinfo" in many places and probably won't compile on
some platforms.
svn path=/trunk/; revision=36318
Bill Meier [Thu, 24 Mar 2011 22:27:20 +0000 (22:27 -0000)]
Don't assign to a proto_item* if the value won't be used: Coverity 942-951
Also: minor Code & Whitespace cleanup.
svn path=/trunk/; revision=36317
Stephen Fisher [Thu, 24 Mar 2011 21:22:07 +0000 (21:22 -0000)]
Only display the 3 digits of precision after the decimal that we have
from the ICMP conversation response time tracking.
svn path=/trunk/; revision=36316
Bill Meier [Thu, 24 Mar 2011 21:19:39 +0000 (21:19 -0000)]
Don't assign to a proto_item* if the value won't be used: Coverity 1155-1156;
Also:
- Fix a few cases of SET_HIDDEN on the wrong item. (The SET_HIDDEN was being
done on an already hidden item).
- Minor indentation cleanup;
svn path=/trunk/; revision=36315
Bill Meier [Thu, 24 Mar 2011 20:17:22 +0000 (20:17 -0000)]
Don't assign to a proto_item* if the value won't be used: Coverity 1124-1128;
Also: minorwhitespace cleanup.
svn path=/trunk/; revision=36314
Chris Maynard [Thu, 24 Mar 2011 18:05:10 +0000 (18:05 -0000)]
Display the value of the ICMP echo request/reply response time in milliseconds.
svn path=/trunk/; revision=36313
Bill Meier [Thu, 24 Mar 2011 17:37:07 +0000 (17:37 -0000)]
Don't assign to a proto_item* if the value won't be used: Coverity 1151-1154;
svn path=/trunk/; revision=36312
Tomas Kukosa [Thu, 24 Mar 2011 16:21:31 +0000 (16:21 -0000)]
Export next_tvb_... functions
svn path=/trunk/; revision=36311