metze/wireshark/wip.git
9 years agoHave the makefile run windepoyqt if it exists to copy the required dlls to the instal...
AndersBroman [Wed, 25 Jun 2014 12:18:40 +0000 (14:18 +0200)]
Have the makefile run windepoyqt if it exists to copy the required dlls to the install dir.

Change-Id: If87c2670e1ffef116d4497241f070fa41a70015b
Reviewed-on: https://code.wireshark.org/review/2644
Reviewed-by: Anders Broman <a.broman58@gmail.com>
9 years agoUse g_queue_free_full() only on GLib > 2.32
Balint Reczey [Fri, 27 Jun 2014 10:00:11 +0000 (12:00 +0200)]
Use g_queue_free_full() only on GLib > 2.32

Change-Id: I7a72fe8243fc809b444f1b026ed872cd2aa92245
Reviewed-on: https://code.wireshark.org/review/2686
Reviewed-by: Balint Reczey <balint@balintreczey.hu>
Tested-by: Balint Reczey <balint@balintreczey.hu>
9 years agoUse GQueue for generating statistics
Balint Reczey [Sun, 22 Jun 2014 19:53:48 +0000 (12:53 -0700)]
Use GQueue for generating statistics

This makes reversing the list back and forth to keep adding data
at O(n) complexity obsolete.

Bug: 9696

Change-Id: Ice77328b8f6c5bf72bbfcfd82e08d09d4f986d3f
Reviewed-on: https://code.wireshark.org/review/2571
Reviewed-by: Balint Reczey <balint@balintreczey.hu>
Tested-by: Balint Reczey <balint@balintreczey.hu>
9 years agoReport more information on a sync_xxx_open() call failure.
Guy Harris [Fri, 27 Jun 2014 08:43:53 +0000 (01:43 -0700)]
Report more information on a sync_xxx_open() call failure.

Trying to get more information for

http://ask.wireshark.org/questions/34057/waitpid-error-starting

Change-Id: Ifc0eb58f5b69c2c4a4638aa95f6b5f8f628bef02
Reviewed-on: https://code.wireshark.org/review/2685
Reviewed-by: Guy Harris <guy@alum.mit.edu>
9 years agoWe're not currently using dissect_rtcp_psfb_remb(); #if 0 it out.
Guy Harris [Fri, 27 Jun 2014 07:39:52 +0000 (00:39 -0700)]
We're not currently using dissect_rtcp_psfb_remb(); #if 0 it out.

Also, note that we need to determine how to handle Application Layer
Feedback messages based on the SDP setup traffic for the session; recent
changes disabled dissection of REMB Application Layer Feedback messages
in favor of MS-RTP Application Layer Feedback messages.  (This is why we
shouldn't remove dissect_rtcp_psfb_remb() unless REMB isn't being used
any more.)

Change-Id: Ib320bdf4a64263fdef29fc4ea2583eaae1cc4bee
Reviewed-on: https://code.wireshark.org/review/2684
Reviewed-by: Guy Harris <guy@alum.mit.edu>
9 years agoIn function 'dissect_rtcp_psfb':
AndersBroman [Fri, 27 Jun 2014 04:21:36 +0000 (06:21 +0200)]
In function 'dissect_rtcp_psfb':
packet-rtcp.c:1232: warning: unused parameter 'top_item'

Change-Id: I76522a9c6094473ce0eeeb7cc929a66e6da21909
Reviewed-on: https://code.wireshark.org/review/2683
Reviewed-by: Anders Broman <a.broman58@gmail.com>
9 years agoRTCP: Add support for MS-RTP
Britt McKinley [Mon, 23 Jun 2014 20:18:04 +0000 (16:18 -0400)]
RTCP: Add support for MS-RTP
Support for Profile Specific Extensions from MS-RTP
Support for RTCP Feedback Messages
Support for Application Layer Feedback Messages.
MS-RTP: Real-time Transport Protocol (RTP) Extensions
http://msdn.microsoft.com/en-us/library/office/cc431492.aspx

Change-Id: I1f1e6e60b5f9d09b1dffd7e308426c0b67914441
Reviewed-on: https://code.wireshark.org/review/2586
Reviewed-by: Anders Broman <a.broman58@gmail.com>
9 years agoTighten the dump-glossary test.
Evan Huus [Fri, 27 Jun 2014 03:43:51 +0000 (23:43 -0400)]
Tighten the dump-glossary test.

 1. The only indication we get of an out-of-order value string is a message on
 STDERR, so check that and fail the test if STDERR wasn't empty.

 2. This exposes an out-of-order value string in packet-stun.c; fix it.

 3. This triggered the pre-commit hook on packet-stun.c, which noticed an API
 error (ENC_ASCII -> ENC_ASCII|ENC_NA); fix that too.

Change-Id: I36f87a2a87b40537119562f22a7e3012716ff239
Lesson: automated testing/tooling is both wonderful and scary.
Reviewed-on: https://code.wireshark.org/review/2682
Reviewed-by: Evan Huus <eapache@gmail.com>
9 years agoIf we have no fields, don't try to iterate through them.
Evan Huus [Fri, 27 Jun 2014 03:25:08 +0000 (23:25 -0400)]
If we have no fields, don't try to iterate through them.

Otherwise glib throws an assertion since the array we pass it is NULL.

Change-Id: I9159c1f5ad99b280c040cd790df3cf352738601f
Reviewed-on: https://code.wireshark.org/review/2680
Reviewed-by: Evan Huus <eapache@gmail.com>
9 years agoshow an expert info if a non-control urb contains a setup packet
Martin Kaiser [Thu, 26 Jun 2014 03:30:13 +0000 (23:30 -0400)]
show an expert info if a non-control urb contains a setup packet

Change-Id: I9339869defa47a862b6174d8821cdd8e6186f5c5
Reviewed-on: https://code.wireshark.org/review/2678
Reviewed-by: Martin Kaiser <wireshark@kaiser.cx>
Tested-by: Martin Kaiser <wireshark@kaiser.cx>
9 years agoa USB urb of an unknown transfer type does not contain a setup packet
Martin Kaiser [Tue, 24 Jun 2014 03:21:29 +0000 (23:21 -0400)]
a USB urb of an unknown transfer type does not contain a setup packet

Change-Id: Iac6a259a1081b907149c49023614a5053440e560
Reviewed-on: https://code.wireshark.org/review/2677
Reviewed-by: Martin Kaiser <wireshark@kaiser.cx>
Tested-by: Martin Kaiser <wireshark@kaiser.cx>
9 years agoCheck the validity of fields given to tshark with "-e": complain and exit if
Jeff Morriss [Wed, 25 Jun 2014 19:37:36 +0000 (15:37 -0400)]
Check the validity of fields given to tshark with "-e": complain and exit if
they aren't valid.

Should help avoid confusion like that in
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=10201

Bug: 10201
Change-Id: Iff7f51ae042ca1a92d1c803b6cb61aa5d81ce205
Reviewed-on: https://code.wireshark.org/review/2654
Reviewed-by: Michael Mann <mmann78@netscape.net>
Reviewed-by: Evan Huus <eapache@gmail.com>
9 years agoFixed linker flags for MSVC to not attempt to make and link a manifest
Graham Bloice [Tue, 24 Jun 2014 23:05:08 +0000 (00:05 +0100)]
Fixed linker flags for MSVC to not attempt to make and link a manifest
Added solution "folder" for tests.

Change-Id: I96dfb4a28fb7aaf3e84cea026e1f5d15afe6c0c6
Reviewed-on: https://code.wireshark.org/review/2635
Tested-by: Graham Bloice <graham.bloice@trihedral.com>
Reviewed-by: Evan Huus <eapache@gmail.com>
Reviewed-by: Graham Bloice <graham.bloice@trihedral.com>
9 years agoadd a cast, the mask makes it safe (stupid compiler)
Evan Huus [Thu, 26 Jun 2014 15:37:06 +0000 (11:37 -0400)]
add a cast, the mask makes it safe (stupid compiler)

Change-Id: Iaf24c62295a93a1abd4fe3daf7e4c9587c3ef76b
Reviewed-on: https://code.wireshark.org/review/2670
Reviewed-by: Evan Huus <eapache@gmail.com>
9 years agoiSER: Add iSER dissector support
Yan Burman [Wed, 11 Jun 2014 07:29:01 +0000 (10:29 +0300)]
iSER: Add iSER dissector support

Bug: 10189
Change-Id: Ie99d99a1736b3c6446d5a00edf201a49dfcd4780
Reviewed-on: https://code.wireshark.org/review/2247
Reviewed-by: Michael Mann <mmann78@netscape.net>
Reviewed-by: Evan Huus <eapache@gmail.com>
9 years agoFix Function call argument is an uninitialized value
Michael Mann [Thu, 26 Jun 2014 13:57:13 +0000 (09:57 -0400)]
Fix Function call argument is an uninitialized value

Change-Id: I716e80e53477edd419164c61937800795a550869
Reviewed-on: https://code.wireshark.org/review/2651
Reviewed-by: Michael Mann <mmann78@netscape.net>
Reviewed-by: Evan Huus <eapache@gmail.com>
9 years agoFix several compilation warnings
Pascal Quantin [Thu, 26 Jun 2014 12:52:58 +0000 (14:52 +0200)]
Fix several compilation warnings

Followup of gf798709

Change-Id: I0afddfe2e9b9ac454377f2358a29b4ecdd011b91
Reviewed-on: https://code.wireshark.org/review/2668
Reviewed-by: Anders Broman <a.broman58@gmail.com>
9 years agoconvert to proto_tree_add_subtree[_format]
Michael Mann [Thu, 26 Jun 2014 02:59:50 +0000 (22:59 -0400)]
convert to proto_tree_add_subtree[_format]

Change-Id: I525ac2aae2bdbfd5f3a2f3b35f1bf10dde053f66
Reviewed-on: https://code.wireshark.org/review/2667
Tested-by: Michael Mann <mmann78@netscape.net>
Reviewed-by: Michael Mann <mmann78@netscape.net>
9 years agoSTUN: Add support of MS-ICE2 (Used by Lync)
Britt McKinley [Mon, 23 Jun 2014 18:22:35 +0000 (14:22 -0400)]
STUN: Add support of MS-ICE2 (Used by Lync)

Interactive Connectivity Establishment ICE Extensions 2.0
http://msdn.microsoft.com/en-us/library/office/cc431504.aspx

Change from review:
1) Change encoding for foundation to ASCII
2) Move case for MS_IMPLEMENTATION_VER.

Change-Id: Ic524a2fe811695478aba81af9cbb3dbd031bbce3
Reviewed-on: https://code.wireshark.org/review/2579
Reviewed-by: Evan Huus <eapache@gmail.com>
Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
9 years agoNew HP DTD dictionary file
Basil [Tue, 24 Jun 2014 14:08:17 +0000 (18:08 +0400)]
New HP DTD dictionary file

Bug: 10206
Change-Id: Ibe056ed37d3619aabfb17d3c2c5e85e67deae4e5
Reviewed-on: https://code.wireshark.org/review/2617
Reviewed-by: Jeff Morriss <jeff.morriss.ws@gmail.com>
Reviewed-by: Evan Huus <eapache@gmail.com>
9 years agoFix Dead Store (Dead assignement/Dead increment) warning found by Clang
Alexis La Goutte [Wed, 25 Jun 2014 19:27:55 +0000 (21:27 +0200)]
Fix Dead Store (Dead assignement/Dead increment) warning found by Clang

Change-Id: Icec39c64d952ccc9739df95135ed79b3196a427a
Reviewed-on: https://code.wireshark.org/review/2652
Reviewed-by: Jeff Morriss <jeff.morriss.ws@gmail.com>
9 years agoFix Dead Store (Dead assignement/Dead increment) warning found by Clang
Alexis La Goutte [Wed, 25 Jun 2014 19:28:23 +0000 (21:28 +0200)]
Fix Dead Store (Dead assignement/Dead increment) warning found by Clang

Change-Id: I5c66a456b7986e3fd7672deaf7ce773467873324
Reviewed-on: https://code.wireshark.org/review/2653
Reviewed-by: Jeff Morriss <jeff.morriss.ws@gmail.com>
9 years agoDecrease TRAFFIC_CAPTURE_DURATION from 60s to 15s.
Gerald Combs [Wed, 25 Jun 2014 23:00:11 +0000 (16:00 -0700)]
Decrease TRAFFIC_CAPTURE_DURATION from 60s to 15s.

Change-Id: Ie6d6ac0489f0fcb506360a3900d33c44cb3477e9
Reviewed-on: https://code.wireshark.org/review/2664
Reviewed-by: Gerald Combs <gerald@wireshark.org>
9 years agoUpdate a comment.
Guy Harris [Wed, 25 Jun 2014 21:05:36 +0000 (14:05 -0700)]
Update a comment.

Also, make the block of code containing that comment intended
consistently with spaces.

Change-Id: I8e8eb346833662f15c53ece5869b12cc430bad11
Reviewed-on: https://code.wireshark.org/review/2661
Reviewed-by: Guy Harris <guy@alum.mit.edu>
9 years agoBoost the maximum snapshot length to 262144, for USBPcap.
Guy Harris [Wed, 25 Jun 2014 20:42:36 +0000 (13:42 -0700)]
Boost the maximum snapshot length to 262144, for USBPcap.

Change-Id: I104bd9d1519298a0e4ff818f1cf821df594b3491
Reviewed-on: https://code.wireshark.org/review/2658
Reviewed-by: Guy Harris <guy@alum.mit.edu>
9 years agoBoost the maximum packet size to 131072.
Guy Harris [Wed, 25 Jun 2014 20:26:39 +0000 (13:26 -0700)]
Boost the maximum packet size to 131072.

Libpcap 1.6/tcpdump 4.6 will support up to 131072, as the MTU on the
Linux loopback device is 65536 on at least some versions of the kernel,
and that doesn't count the fake Ethernet header, so the maximum packet
size is 65549; they went to the next power of 2 up.

Change-Id: Ibfc66d01ef8ef7387887a75c2b567159bb78ac0f
Reviewed-on: https://code.wireshark.org/review/2655
Reviewed-by: Guy Harris <guy@alum.mit.edu>
9 years agoDLSw: better fix, as suggested by Evan
Pascal Quantin [Wed, 25 Jun 2014 18:15:32 +0000 (20:15 +0200)]
DLSw: better fix, as suggested by Evan

Change-Id: I82556c40cc4f0618e9e1c927beafa19ea4659683
Reviewed-on: https://code.wireshark.org/review/2650
Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
9 years agoIndicate tshark can read (-r) from stdin and named pipes now.
Jeff Morriss [Wed, 25 Jun 2014 15:03:12 +0000 (11:03 -0400)]
Indicate tshark can read (-r) from stdin and named pipes now.

(Though it works only with certain formats.)  As requested in:
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=2874#c4

Change-Id: I2ce0509d8750b21ae0167d12459074ba0f9506f8
Reviewed-on: https://code.wireshark.org/review/2646
Reviewed-by: Evan Huus <eapache@gmail.com>
9 years agoDLSw: ett2 proto_item variable is an input parameter for dissect_dlsw_capex()
Pascal Quantin [Wed, 25 Jun 2014 17:50:57 +0000 (19:50 +0200)]
DLSw: ett2 proto_item variable is an input parameter for dissect_dlsw_capex()

Regression introduced in g888f22d

Change-Id: I6ae451ef31b188540db4e4687c9fa492e8aa195e
Reviewed-on: https://code.wireshark.org/review/2649
Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
9 years agossl-utils: add missing ett registration
Peter Wu [Wed, 25 Jun 2014 08:54:57 +0000 (10:54 +0200)]
ssl-utils: add missing ett registration

Fixes the dissector bug warning:
"epan/proto.c:4657: failed assertion "idx >= 0 && idx < num_tree_types"

Add some comments to avoid future mistakes, add folding markers and fix
alignment of curves hf.

Change-Id: Ibcb57bfeb09a9777324682704a86f1ce260d345e
Reviewed-on: https://code.wireshark.org/review/2642
Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
9 years agoDTPT: remove unused variables
Pascal Quantin [Wed, 25 Jun 2014 17:03:36 +0000 (19:03 +0200)]
DTPT: remove unused variables

Change-Id: I9c8e26af6e0d71012b010dee24a0d0d7220b49f7
Reviewed-on: https://code.wireshark.org/review/2648
Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
9 years agoconvert to proto_tree_add_subtree[_format]
Michael Mann [Wed, 25 Jun 2014 03:24:14 +0000 (23:24 -0400)]
convert to proto_tree_add_subtree[_format]

Change-Id: I2ea1892b5963cc5578cbdd2b03029ca8424f2267
Reviewed-on: https://code.wireshark.org/review/2640
Tested-by: Michael Mann <mmann78@netscape.net>
Reviewed-by: Michael Mann <mmann78@netscape.net>
9 years agofix_bug_10199: enhancement Telephony VoIP Call add ability to show all SIP Flows.
James Coleman [Thu, 19 Jun 2014 16:47:49 +0000 (17:47 +0100)]
fix_bug_10199: enhancement Telephony VoIP Call add ability to show all SIP Flows.
Add SIP Flows menu option beside VoIP Calls.
Flow for all SIP message types (which have a call-id) is shown in SIP Flow.
Add useful info(original flow method, response code, cseq) to comment field in conversation and flow dialogs.

Change-Id: I4801a633ed9b6594b2d89629c9d6fec6352da150
Reviewed-on: https://code.wireshark.org/review/2479
Reviewed-by: Evan Huus <eapache@gmail.com>
Reviewed-by: James Coleman <gaoithe@gmail.com>
Reviewed-by: Anders Broman <a.broman58@gmail.com>
9 years agoRemove wiretap/AUTHORS (after ensuring all names are present in the top-level AUTHORS...
Jeff Morriss [Wed, 25 Jun 2014 14:26:33 +0000 (10:26 -0400)]
Remove wiretap/AUTHORS (after ensuring all names are present in the top-level AUTHORS file).

Change-Id: Ic3aa46c770f373541572e8507ec08b3439f3077f
Reviewed-on: https://code.wireshark.org/review/2645
Reviewed-by: Evan Huus <eapache@gmail.com>
9 years agoList elided_label.h only once.
Irene Ruengeler [Wed, 25 Jun 2014 07:59:50 +0000 (09:59 +0200)]
List elided_label.h only once.

Change-Id: Ibd00e55fb31d571b0b1f6f1c4595a35835e22d4d
Reviewed-on: https://code.wireshark.org/review/2641
Reviewed-by: Anders Broman <a.broman58@gmail.com>
9 years agoGet rid of an unused variable.
Guy Harris [Wed, 25 Jun 2014 02:38:02 +0000 (19:38 -0700)]
Get rid of an unused variable.

Change-Id: I8e436b7e4724c85943200087783d3cf7dd7c1535
Reviewed-on: https://code.wireshark.org/review/2639
Reviewed-by: Guy Harris <guy@alum.mit.edu>
9 years agoUpdate my email address in a couple more spots.
Jeff Morriss [Wed, 25 Jun 2014 02:14:41 +0000 (22:14 -0400)]
Update my email address in a couple more spots.

Change-Id: I787ae2651a8083bc3082e5cb85dbd849823a16ca
Reviewed-on: https://code.wireshark.org/review/2638
Reviewed-by: Jeff Morriss <jeff.morriss.ws@gmail.com>
9 years agoReduce maximum permitted leakage to 100KB
Evan Huus [Tue, 24 Jun 2014 22:30:23 +0000 (18:30 -0400)]
Reduce maximum permitted leakage to 100KB

Down from 500KB. The old value only triggered once that I can recall, and the
"average" leakage I'm seeing on most captures is only a few KB now, so this
shouldn't flood us with issues (which was the original concern leaving it so
high).

Change-Id: Ie4c98696b3fb7a533a7dc4f83c7ac8c458b499c8
Reviewed-on: https://code.wireshark.org/review/2633
Reviewed-by: Michael Mann <mmann78@netscape.net>
9 years agoconvert to proto_tree_add_subtree[_format]
Michael Mann [Tue, 24 Jun 2014 00:39:28 +0000 (20:39 -0400)]
convert to proto_tree_add_subtree[_format]

Change-Id: I5f573dffabb8685a8e5a334ff2bfb24d9838daa6
Reviewed-on: https://code.wireshark.org/review/2601
Tested-by: Michael Mann <mmann78@netscape.net>
Reviewed-by: Michael Mann <mmann78@netscape.net>
9 years agoAdd back initializers dropped in g9356d5c689fa
Evan Huus [Tue, 24 Jun 2014 22:25:15 +0000 (18:25 -0400)]
Add back initializers dropped in g9356d5c689fa

They were actually necessary.

Bug:10224
Change-Id: I9973bf7bab670d12e5b90bb2a57e99f9125d6a07
Reviewed-on: https://code.wireshark.org/review/2632
Reviewed-by: Evan Huus <eapache@gmail.com>
9 years agoGTK3: Fix changing filter expression's bg color based on validity
Balint Reczey [Tue, 24 Jun 2014 12:58:46 +0000 (14:58 +0200)]
GTK3: Fix changing filter expression's bg color based on validity

Bug: 8598
Change-Id: Id5a52c05cab25df9e228b1b4d2caf07335c1f4c8
Reviewed-on: https://code.wireshark.org/review/2614
Reviewed-by: Anders Broman <a.broman58@gmail.com>
Reviewed-by: Evan Huus <eapache@gmail.com>
9 years agoFix two bugs in kafka dissection
Evan Huus [Tue, 24 Jun 2014 19:38:15 +0000 (15:38 -0400)]
Fix two bugs in kafka dissection

 - Respect the length field when dissecting message sets
 - Don't "wrap around" in capture when doing request/response matches

Also convert one instance to proto_tree_add_subtree, as an experiment.

Change-Id: Id161687865afa7ca83e6943a643bc54582f65554
Reviewed-on: https://code.wireshark.org/review/2624
Reviewed-by: Evan Huus <eapache@gmail.com>
9 years agoSupport -w even if we're not built with pcap.
Guy Harris [Tue, 24 Jun 2014 20:48:40 +0000 (13:48 -0700)]
Support -w even if we're not built with pcap.

You can, for example, do

tshark -r file1 -Y filter -w file2

to read a file, apply a read filter, and write the packets that match
the filter to another file even if you can't capture traffic.

Change-Id: Ifd5e1d5c0e745edef5e98ec4babc720bfbcee6d9
Reviewed-on: https://code.wireshark.org/review/2627
Reviewed-by: Guy Harris <guy@alum.mit.edu>
9 years agoAdd the capchild directory to some lists of rules.
Guy Harris [Tue, 24 Jun 2014 20:10:56 +0000 (13:10 -0700)]
Add the capchild directory to some lists of rules.

Change-Id: Ic06cf8005a16cc61c8e3ae506dba4c4916ab6471
Reviewed-on: https://code.wireshark.org/review/2626
Reviewed-by: Guy Harris <guy@alum.mit.edu>
9 years agoLoad WinPcap before we get the run-time information string.
Guy Harris [Tue, 24 Jun 2014 20:04:22 +0000 (13:04 -0700)]
Load WinPcap before we get the run-time information string.

That way, we can include the WinPcap version in that string.

Change-Id: I01fa0defce158e122d1c602fdfbc81916a9e80ef
Reviewed-on: https://code.wireshark.org/review/2625
Reviewed-by: Guy Harris <guy@alum.mit.edu>
9 years agoOSPF Cleanup. Remove unnecessary TFS declarations
Anish Bhatt [Fri, 20 Jun 2014 08:48:47 +0000 (01:48 -0700)]
OSPF Cleanup. Remove unnecessary TFS declarations

Change-Id: I9b4205f6d579ea9e707d83baf81f77393e3098b4
Signed-off-by: Anish Bhatt <anish@chelsio.com>
Reviewed-on: https://code.wireshark.org/review/2478
Reviewed-by: Anders Broman <a.broman58@gmail.com>
9 years agoFix sorting RTP Streams by jitter
Balint Reczey [Tue, 24 Jun 2014 14:53:58 +0000 (16:53 +0200)]
Fix sorting RTP Streams by jitter

Bug: 9460
Change-Id: I80d991053eb47b8650561e8af4cc8dec512e2c9c
Reviewed-on: https://code.wireshark.org/review/2619
Reviewed-by: Anders Broman <a.broman58@gmail.com>
Reviewed-by: Michael Mann <mmann78@netscape.net>
9 years agoAdd quotes around MAKEDIR.
Ben Burwell [Tue, 24 Jun 2014 15:34:40 +0000 (11:34 -0400)]
Add quotes around MAKEDIR.

Bug: 7870
Change-Id: I6cea057c4953f5ecc0a146a24570d089e79f8352
Reviewed-on: https://code.wireshark.org/review/2620
Reviewed-by: Michael Mann <mmann78@netscape.net>
9 years agoCorrect indentation in make-dissector-reg
Роман Донченко [Tue, 24 Jun 2014 17:33:41 +0000 (21:33 +0400)]
Correct indentation in make-dissector-reg

The indented portions are inside an if.

Change-Id: I3343a7aa7e777466ec9f40e8a02a8218bef62017
Reviewed-on: https://code.wireshark.org/review/2622
Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
Reviewed-by: Michael Mann <mmann78@netscape.net>
9 years agoEnsure proto_item from subtree is initialized
Michael Mann [Tue, 24 Jun 2014 18:07:10 +0000 (14:07 -0400)]
Ensure proto_item from subtree is initialized

bug:10223
Change-Id: I40ed92bcf436723ad3927a93181f7c732cf4690f
Reviewed-on: https://code.wireshark.org/review/2623
Reviewed-by: Michael Mann <mmann78@netscape.net>
Reviewed-by: Evan Huus <eapache@gmail.com>
9 years agoMBIM: add an option to force SMS PDU decoding format if MBIM_DEVICE_CAPS_INFO message...
Pascal Quantin [Tue, 24 Jun 2014 14:17:09 +0000 (16:17 +0200)]
MBIM: add an option to force SMS PDU decoding format if MBIM_DEVICE_CAPS_INFO message was not captured

Change-Id: Iff78a00b463a7a33e1705c76ea49618af532f3aa
Reviewed-on: https://code.wireshark.org/review/2621
Tested-by: Pascal Quantin <pascal.quantin@gmail.com>
Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
9 years agoFix OS X CMake build.
Guy Harris [Tue, 24 Jun 2014 07:07:38 +0000 (00:07 -0700)]
Fix OS X CMake build.

I have ***NO*** idea why this makes a difference, but, without this
change, APPLE_CORE_FOUNDATION_LIBRARY is apparently *not* set correctly
for wsutil/CMakeLists.txt, and, with this change, it is.  I guess
there's something magic involved here with "global" CMake variables or
something crazy such as that.

Change-Id: I7a0046b9c249568cd666720838104f48e854e203
Reviewed-on: https://code.wireshark.org/review/2612
Reviewed-by: Guy Harris <guy@alum.mit.edu>
9 years agoproto_tree_add_uint() takes a value, not an encoding, as an argument.
Guy Harris [Tue, 24 Jun 2014 06:56:59 +0000 (23:56 -0700)]
proto_tree_add_uint() takes a value, not an encoding, as an argument.

Change-Id: Ie2c0523e32b54cd13506501d98215934a8d1304e
Reviewed-on: https://code.wireshark.org/review/2611
Reviewed-by: Guy Harris <guy@alum.mit.edu>
9 years agoAnother capchild.lib -> libcapchild.lib.
Guy Harris [Tue, 24 Jun 2014 03:53:48 +0000 (20:53 -0700)]
Another capchild.lib -> libcapchild.lib.

Change-Id: I4c7750f05c831f12d0d34e2edf0d982d501d90d8
Reviewed-on: https://code.wireshark.org/review/2610
Reviewed-by: Guy Harris <guy@alum.mit.edu>
9 years agoAnother file to generate in the configure script.
Guy Harris [Tue, 24 Jun 2014 03:35:20 +0000 (20:35 -0700)]
Another file to generate in the configure script.

Change-Id: I6dafc9ab9fafd465cb2ad83e5e56e4d22b36d41b
Reviewed-on: https://code.wireshark.org/review/2609
Reviewed-by: Guy Harris <guy@alum.mit.edu>
9 years agoCreate capchild/doxygen.cfg
Joerg Mayer [Tue, 24 Jun 2014 03:18:37 +0000 (05:18 +0200)]
Create capchild/doxygen.cfg

Change-Id: I215a9d9849c752f9e8e3216c0f6479d1ba1cfafd
Reviewed-on: https://code.wireshark.org/review/2608
Reviewed-by: Jörg Mayer <jmayer@loplof.de>
9 years agousb iso urbs on linux can't possibly contain a setup packet
Martin Kaiser [Sat, 21 Jun 2014 20:10:43 +0000 (16:10 -0400)]
usb iso urbs on linux can't possibly contain a setup packet

see mon_bin_event() in the linux kernel where the setup_flag is set only
for control urbs

clean up various things related to this assertion:
remove type_2 parameter
show the iso descriptors in any case
calculate the end offset correctly, the end offset is the byte after the
iso data

Change-Id: Iebfbe6443c224a958a1697563aa8fb853d7aa8c2
Reviewed-on: https://code.wireshark.org/review/2541
Reviewed-by: Martin Kaiser <wireshark@kaiser.cx>
Tested-by: Martin Kaiser <wireshark@kaiser.cx>
9 years agoRemove unneeded include <sys/stat.h>
Joerg Mayer [Tue, 24 Jun 2014 02:27:04 +0000 (04:27 +0200)]
Remove unneeded include <sys/stat.h>

Change-Id: I3be8f29d2b4fba2cb1d7ee2f29bdb27e42dd40a5
Reviewed-on: https://code.wireshark.org/review/2607
Reviewed-by: Jörg Mayer <jmayer@loplof.de>
9 years agoAdd libcapchild, remove the files that are now in it.
Guy Harris [Tue, 24 Jun 2014 02:12:34 +0000 (19:12 -0700)]
Add libcapchild, remove the files that are now in it.

Change-Id: I79c16bbea47e507859c904e4142fc06fa6bfc127
Reviewed-on: https://code.wireshark.org/review/2606
Reviewed-by: Guy Harris <guy@alum.mit.edu>
9 years agoGet rid of trailing blanks.
Guy Harris [Tue, 24 Jun 2014 02:08:11 +0000 (19:08 -0700)]
Get rid of trailing blanks.

Change-Id: Idee9c273a74d37f1e754fcada0a06dcd67f306c3
Reviewed-on: https://code.wireshark.org/review/2605
Reviewed-by: Guy Harris <guy@alum.mit.edu>
9 years agoRemove unneeded sys/types.h
Joerg Mayer [Tue, 24 Jun 2014 02:06:07 +0000 (04:06 +0200)]
Remove unneeded sys/types.h

Change-Id: I03cd66cb9a2d01ea40308b338955756d08a36516
Reviewed-on: https://code.wireshark.org/review/2604
Reviewed-by: Jörg Mayer <jmayer@loplof.de>
9 years agocapture_ifinfo.c and capture_sync.c are now in libcapchild.
Guy Harris [Tue, 24 Jun 2014 02:06:33 +0000 (19:06 -0700)]
capture_ifinfo.c and capture_sync.c are now in libcapchild.

Change-Id: I8203a3a1a77f231c4286fd32fcc6372a2ac9db48
Reviewed-on: https://code.wireshark.org/review/2603
Reviewed-by: Guy Harris <guy@alum.mit.edu>
9 years agoRestore correct tree assignment.
Michael Mann [Tue, 24 Jun 2014 01:54:48 +0000 (21:54 -0400)]
Restore correct tree assignment.

Change-Id: Id06bd486114a80fb899f8dc148d48928e99e775e
Reviewed-on: https://code.wireshark.org/review/2602
Reviewed-by: Michael Mann <mmann78@netscape.net>
9 years agoAdd a doxygen.cfg.in file for the capture child library.
Guy Harris [Tue, 24 Jun 2014 01:47:23 +0000 (18:47 -0700)]
Add a doxygen.cfg.in file for the capture child library.

Change-Id: I79c83c8583b86e1d05b7f486078a0e0a4ca5dbdd
Reviewed-on: https://code.wireshark.org/review/2599
Reviewed-by: Guy Harris <guy@alum.mit.edu>
9 years agodos2unix is now in Utils, not Base.
Guy Harris [Tue, 24 Jun 2014 00:55:26 +0000 (17:55 -0700)]
dos2unix is now in Utils, not Base.

Change-Id: Ib430582270a2b788843785fd5aa7148ebe0f8871
Reviewed-on: https://code.wireshark.org/review/2597
Reviewed-by: Guy Harris <guy@alum.mit.edu>
9 years agoFix name of library.
Guy Harris [Tue, 24 Jun 2014 00:47:45 +0000 (17:47 -0700)]
Fix name of library.

Change-Id: I463637a6218a86ed6dbd95019f33d8b999bcaaf2
Reviewed-on: https://code.wireshark.org/review/2596
Reviewed-by: Guy Harris <guy@alum.mit.edu>
9 years agoFix some more capchild includes.
Guy Harris [Tue, 24 Jun 2014 00:35:57 +0000 (17:35 -0700)]
Fix some more capchild includes.

Change-Id: I6d6cd79ee9e630fe81eb6deca9691249b44bdaf4
Reviewed-on: https://code.wireshark.org/review/2595
Reviewed-by: Guy Harris <guy@alum.mit.edu>
9 years agoFor capchild headers, include <capchild/XXX.h>.
Guy Harris [Tue, 24 Jun 2014 00:27:28 +0000 (17:27 -0700)]
For capchild headers, include <capchild/XXX.h>.

Change-Id: I780c69ee637dcd9846756a2e2d6a35baf02d826d
Reviewed-on: https://code.wireshark.org/review/2594
Reviewed-by: Guy Harris <guy@alum.mit.edu>
9 years agoInclude ZLIB_CFLAGS so we file zlib.h.
Guy Harris [Tue, 24 Jun 2014 00:03:56 +0000 (17:03 -0700)]
Include ZLIB_CFLAGS so we file zlib.h.

Change-Id: I9026f99397b1dfbc550af550d0c71597b3c87950
Reviewed-on: https://code.wireshark.org/review/2592
Reviewed-by: Guy Harris <guy@alum.mit.edu>
9 years agoMove the routines to talk to dumpcap into a static libcapchild.
Guy Harris [Mon, 23 Jun 2014 23:48:41 +0000 (16:48 -0700)]
Move the routines to talk to dumpcap into a static libcapchild.

This pulls some stuff out of the top-level directory, and means we don't
have to build them once for every program using them.

Change-Id: I37b31fed20f2d5c3563ecd2bae9fd86af70afff5
Reviewed-on: https://code.wireshark.org/review/2591
Reviewed-by: Guy Harris <guy@alum.mit.edu>
9 years agossl: fix ClientKeyExchange, fix TLSv1.2 SKE for DH
Peter Wu [Mon, 23 Jun 2014 21:14:06 +0000 (23:14 +0200)]
ssl: fix ClientKeyExchange, fix TLSv1.2 SKE for DH

Since DTLS and TLS do not differ in handling ClientKeyExchange and
ServerKeyExchange, its dissection got moved to ssl-utils. The code is
based on the SSL dissector, with header field names adjusted to the
DTLS ones (those got capitalized). Besides a version difference (for
signatures), the header field and function names, the DTLS and SSL code
are equal (this is verified).

This patch refactors the dissectors for DHE_RSA and ECDHE to make use of
a common function to dissect the signed_params field. All offset
tracking is also removed in favor of exception handling by the
proto_tree_add_item function. Occurrences of proto_tree_add_uint are
also replaced by proto_tree_add_item for simplicity.

After those changes, the SKE dissector for DH key exchanges is updated
to handle the mandatory signature field in TLSv1.2, using the newly
added function. (bug 9208)

Another bug occurred after the length check removal, pre-TLS and
OpenSSL's old DTLS implemenation do not include a vector length in
the CKE. This is now also fixed. (bug 10222)

Other minor changes: comments added/corrected, renamed
keyex_dh -> keyex_dhe (includes DHE_RSA and DHE_DSS).

Bug: 9208
Bug: 10222
Change-Id: I76e835d56a65c91facce46840d79c1c48ce8d5dd
Reviewed-on: https://code.wireshark.org/review/2542
Reviewed-by: Evan Huus <eapache@gmail.com>
9 years agoRemove some unneeded includes
Joerg Mayer [Mon, 23 Jun 2014 16:51:28 +0000 (18:51 +0200)]
Remove some unneeded includes

Change-Id: I4327ead0451244daa0d876ae3a770cbbf80760c8
Reviewed-on: https://code.wireshark.org/review/2590
Reviewed-by: Jörg Mayer <jmayer@loplof.de>
9 years agoDon't report the run-time version of pcap if we weren't built with pcap.
Guy Harris [Mon, 23 Jun 2014 22:49:11 +0000 (15:49 -0700)]
Don't report the run-time version of pcap if we weren't built with pcap.

Change-Id: Ib879290b72938cc999e00b6e044f917161ba5801
Reviewed-on: https://code.wireshark.org/review/2589
Reviewed-by: Guy Harris <guy@alum.mit.edu>
9 years agoHave individual programs get libpcap and libz version info.
Guy Harris [Mon, 23 Jun 2014 21:55:56 +0000 (14:55 -0700)]
Have individual programs get libpcap and libz version info.

That way, the code that constructs the runtime version string doesn't
itself have to call libpcap and libz, and could be usable in programs
that don't call them.

While we're at it, add "with" to the run-time version information for
GnuTLS and libgcrypt, to match the compile-time version information, and
add the version information from libwireshark to TShark.

Change-Id: I3726a027d032270b032292da9314c1cec535dcd2
Reviewed-on: https://code.wireshark.org/review/2587
Reviewed-by: Guy Harris <guy@alum.mit.edu>
9 years agoInitialize whole buffer in GPRS Mobility and Session Management dissector
Pascal Quantin [Mon, 23 Jun 2014 20:06:05 +0000 (22:06 +0200)]
Initialize whole buffer in GPRS Mobility and Session Management dissector

Bug: 10216
Change-Id: I572a7a6ce0f816063f02397b667dd46c990cf73e
Reviewed-on: https://code.wireshark.org/review/2583
Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
9 years ago[WIP] qt: Add support for Polish language
Michal Labedzki [Wed, 30 Oct 2013 12:31:53 +0000 (13:31 +0100)]
[WIP] qt: Add support for Polish language

Change-Id: Ib63937e741b737f171a9b383a9cbabb55dfdd8ef
Reviewed-on: https://code.wireshark.org/review/2553
Reviewed-by: Jörg Mayer <jmayer@loplof.de>
9 years agoPoint the user at http://www.cygwin.com/install.html for downloads.
Guy Harris [Mon, 23 Jun 2014 17:04:11 +0000 (10:04 -0700)]
Point the user at cygwin.com/install.html for downloads.

That page gives a bit more information.

Change-Id: Id0c708ede50aa9e6c6583f6957c355a630fa7e7e
Reviewed-on: https://code.wireshark.org/review/2578
Reviewed-by: Guy Harris <guy@alum.mit.edu>
9 years agoReplace AF_ values by COMMON_AF_ values.
Joerg Mayer [Mon, 23 Jun 2014 16:44:14 +0000 (18:44 +0200)]
Replace AF_ values by COMMON_AF_ values.
Remove no longer needed system includes

Change-Id: Id9ffffaa7da5185041db63fa7611d348a1cc4b68
Reviewed-on: https://code.wireshark.org/review/2577
Reviewed-by: Jörg Mayer <jmayer@loplof.de>
9 years agoAdd all AF_ types for Windows that are in WinSock.h or WinSock2.h
Joerg Mayer [Mon, 23 Jun 2014 16:39:43 +0000 (18:39 +0200)]
Add all AF_ types for Windows that are in WinSock.h or WinSock2.h

Change-Id: Ia45f4aaea75d8cb08bb9016f9147f47db4434b06
Reviewed-on: https://code.wireshark.org/review/2576
Reviewed-by: Jörg Mayer <jmayer@loplof.de>
9 years agoRevert "Optimize sip_is_known_sip_header()"
Evan Huus [Mon, 23 Jun 2014 16:21:49 +0000 (16:21 +0000)]
Revert "Optimize sip_is_known_sip_header()"

This reverts commit c9a5fbeb1da9b5566ac9ac36bfdcdee6172ef5c8.

Change-Id: Ic2e5d531f719ed1107ef7bb1de12175d4601fd6d
Reviewed-on: https://code.wireshark.org/review/2574
Reviewed-by: Evan Huus <eapache@gmail.com>
9 years agoRevert "Cheat and try to fix the generated file manually."
Evan Huus [Mon, 23 Jun 2014 16:22:05 +0000 (16:22 +0000)]
Revert "Cheat and try to fix the generated file manually."

This reverts commit 9079e3ad1d32c594309a52ccef5936d11a93a55d.

Change-Id: I0430408e139ff8de068c970d02e36122552614fe
Reviewed-on: https://code.wireshark.org/review/2575
Reviewed-by: Evan Huus <eapache@gmail.com>
9 years agoCheat and try to fix the generated file manually.
AndersBroman [Mon, 23 Jun 2014 15:28:36 +0000 (17:28 +0200)]
Cheat and try to fix the generated file manually.

Change-Id: Iabf1821aa0ef676ac4d1d7f2983460b2e671a98a
Reviewed-on: https://code.wireshark.org/review/2573
Reviewed-by: Anders Broman <a.broman58@gmail.com>
9 years agoOptimize sip_is_known_sip_header()
Jakub Zawadzki [Tue, 10 Jun 2014 21:17:01 +0000 (23:17 +0200)]
Optimize sip_is_known_sip_header()

Profling SIP shows that gperf generated hashing code, is
3 times faster than using GHashTable & g_str_hash/_equal()

This result in about 1% improve of whole dissection (sip traffic with filter).

Change-Id: Id6bf64bacd872e2d1c30a1b6356db444b25ba326
Reviewed-on: https://code.wireshark.org/review/2116
Reviewed-by: Anders Broman <a.broman58@gmail.com>
9 years agoFix a typo.
AndersBroman [Mon, 23 Jun 2014 08:21:14 +0000 (10:21 +0200)]
Fix a typo.

Change-Id: I6874a6f4a340c2b8e82d1ca5333cbeb31ff27f2d
Reviewed-on: https://code.wireshark.org/review/2570
Reviewed-by: Anders Broman <a.broman58@gmail.com>
9 years agoAdd col_append_lstr(), speedup column generation for TCP.
Jakub Zawadzki [Sat, 21 Jun 2014 21:06:54 +0000 (23:06 +0200)]
Add col_append_lstr(), speedup column generation for TCP.

When dissecting with columns TCP dissector spends
around 1/4 time in col_append_fstr(), add col_append_lstr()
and do formatting by ourselves.

Change-Id: If90bc26242761884b4991e8db0db62c8f9e32690
Reviewed-on: https://code.wireshark.org/review/2527
Reviewed-by: Evan Huus <eapache@gmail.com>
Reviewed-by: Anders Broman <a.broman58@gmail.com>
9 years agoGet rid of set-but-not-used variable.
Guy Harris [Sun, 22 Jun 2014 22:17:48 +0000 (15:17 -0700)]
Get rid of set-but-not-used variable.

Change-Id: Ic0f2c79b4bd9fc737bf33ef64512f4142d74de6b
Reviewed-on: https://code.wireshark.org/review/2568
Reviewed-by: Guy Harris <guy@alum.mit.edu>
9 years agoGet rid of some unused variables.
Guy Harris [Sun, 22 Jun 2014 21:48:51 +0000 (14:48 -0700)]
Get rid of some unused variables.

Change-Id: I2a806af639e5f0519ba93b0048ec7a4624fa33fc
Reviewed-on: https://code.wireshark.org/review/2567
Reviewed-by: Guy Harris <guy@alum.mit.edu>
9 years agoMake dissect_lltd() a new-style dissector and register it as such.
Guy Harris [Sun, 22 Jun 2014 21:41:09 +0000 (14:41 -0700)]
Make dissect_lltd() a new-style dissector and register it as such.

Presumably that was the intent.

Change-Id: Icf8529a23a9a36e7f12e446d67f3867771b221d8
Reviewed-on: https://code.wireshark.org/review/2566
Reviewed-by: Guy Harris <guy@alum.mit.edu>
9 years agoconvert to proto_tree_add_subtree[_format] for ASN.1 dissectors
Michael Mann [Sat, 21 Jun 2014 01:35:40 +0000 (21:35 -0400)]
convert to proto_tree_add_subtree[_format] for ASN.1 dissectors

Change-Id: I753ca95e2e1b38bad2c09955317e648c525e40ef
Reviewed-on: https://code.wireshark.org/review/2509
Tested-by: Michael Mann <mmann78@netscape.net>
Reviewed-by: Michael Mann <mmann78@netscape.net>
9 years agoLLTD dissector
Michael Mann [Sat, 21 Jun 2014 18:07:27 +0000 (14:07 -0400)]
LLTD dissector

bug: 6071
Change-Id: If7b544a762df10ffc13aeaf8886cf74a1757c37c
Reviewed-on: https://code.wireshark.org/review/2512
Tested-by: Michael Mann <mmann78@netscape.net>
Reviewed-by: Michael Mann <mmann78@netscape.net>
9 years agoconvert to proto_tree_add_subtree[_format]
Michael Mann [Sun, 22 Jun 2014 20:01:36 +0000 (16:01 -0400)]
convert to proto_tree_add_subtree[_format]

Change-Id: Ib60ca75b7da8cfa21cfe2999c9b9448a02c332df
Reviewed-on: https://code.wireshark.org/review/2560
Tested-by: Michael Mann <mmann78@netscape.net>
Reviewed-by: Michael Mann <mmann78@netscape.net>
9 years agoInitialize whole buffer in GTP dissector
Evan Huus [Sun, 22 Jun 2014 20:53:35 +0000 (16:53 -0400)]
Initialize whole buffer in GTP dissector

Bug:10216
Change-Id: Ib7de616d50937eb43b16daa4067ee0de9edc8ec7
Reviewed-on: https://code.wireshark.org/review/2562
Reviewed-by: Evan Huus <eapache@gmail.com>
9 years agoFix leak in nbap init function
Evan Huus [Sun, 22 Jun 2014 19:33:06 +0000 (15:33 -0400)]
Fix leak in nbap init function

Change-Id: I9a70d4da936ad5fd847fc1ba0b29b7220030b977
Reviewed-on: https://code.wireshark.org/review/2558
Reviewed-by: Evan Huus <eapache@gmail.com>
9 years agoFix a leak in the SCTP dissector
Evan Huus [Sun, 22 Jun 2014 19:31:01 +0000 (15:31 -0400)]
Fix a leak in the SCTP dissector

use wmem instead of glib

Change-Id: I326d2dd71b13ae45b4434c86fdacf9f3cec6c069
Reviewed-on: https://code.wireshark.org/review/2557
Reviewed-by: Evan Huus <eapache@gmail.com>
9 years agoReplace C++-style comments with "#if 0"/"#endif".
Guy Harris [Sun, 22 Jun 2014 18:48:57 +0000 (11:48 -0700)]
Replace C++-style comments with "#if 0"/"#endif".

checkapi complains about C++-style comments, as some C compilers (IBM
XLC, for one) reject them by default, and gcc -pedantic might do so as
well.

Change-Id: I1719da03d2fed0fe97574e200dd79434b3d760cd
Reviewed-on: https://code.wireshark.org/review/2556
Reviewed-by: Guy Harris <guy@alum.mit.edu>
9 years agoPoint to the Cygwin mirrors page.
Guy Harris [Sun, 22 Jun 2014 18:38:44 +0000 (11:38 -0700)]
Point to the Cygwin mirrors page.

If you choose to install from the Intertubes, you probably want to
choose a nearby mirror site, rather than downloading from halfway across
the world; the list of download sites in setup*.exe's UI gives no
indication of where the mirror sites are, so you could end up picking
one to which you have a slow Internet path.  Tell the user about the
list of mirror sites on the Cygwin Web site, as that list *does* give
geographical locations.

Change-Id: Idf035d288885ee45db7b3627af969e64270487a1
Reviewed-on: https://code.wireshark.org/review/2555
Reviewed-by: Guy Harris <guy@alum.mit.edu>
9 years ago64-bit Cygwin's dependencies for Bison are wrong; it doesn't require m4.
Guy Harris [Sun, 22 Jun 2014 18:32:09 +0000 (11:32 -0700)]
64-bit Cygwin's dependencies for Bison are wrong; it doesn't require m4.

Change-Id: Id6a6c0a7c09ed1d9eb1e1d522930e3f94e7b20d6
Reviewed-on: https://code.wireshark.org/review/2554
Reviewed-by: Guy Harris <guy@alum.mit.edu>
9 years agoLogcat: Set data-text-lines dissectors for log
Michal Labedzki [Sun, 22 Jun 2014 12:47:02 +0000 (14:47 +0200)]
Logcat: Set data-text-lines dissectors for log

Some binary logcat packets has more then one line, show them in
a convenient form.

Change-Id: I008aac6fe5589f2b10db51f7221853f9d79bbc7a
Reviewed-on: https://code.wireshark.org/review/2549
Reviewed-by: Michal Labedzki <michal.labedzki@tieto.com>
9 years agoLogcat: Fix dump formats
Michal Labedzki [Mon, 9 Jun 2014 14:07:22 +0000 (16:07 +0200)]
Logcat: Fix dump formats

Some dump formats are not exactly what should be done,
so fix them and try to little improve them (mostly by space padding %-8s)

Change-Id: I8ee38479c848abc0a2eaff30ce733e4b60930ac4
Reviewed-on: https://code.wireshark.org/review/2550
Reviewed-by: Michal Labedzki <michal.labedzki@tieto.com>
9 years agoBluetooth: A2DP: Fix fuzz failture
Michal Labedzki [Wed, 18 Jun 2014 15:10:09 +0000 (17:10 +0200)]
Bluetooth: A2DP: Fix fuzz failture

Fix invalid structure casting by using defaults values,
this also fix DecodeAs for A2DP.

Do the same for VDP.

Change-Id: I360787af648ed65205eb54732ab6d88f8532cf15
Reviewed-on: https://code.wireshark.org/review/2551
Reviewed-by: Michal Labedzki <michal.labedzki@tieto.com>