aboutsummaryrefslogtreecommitdiffstats
path: root/epan/dissectors/packet-osc.c
AgeCommit message (Collapse)AuthorFilesLines
2014-06-20Rename dissector_add_handle() to dissector_add_for_decode_as().Guy Harris1-1/+2
Hopefully that name makes it clear what the routiner's purpose is, and will encourage people to use it rather than using dissector_add_uint() with a bogus integer value. Change-Id: Ic5be456d0ad40b176aab01712ab7b13aed5de2a8 Reviewed-on: https://code.wireshark.org/review/2483 Reviewed-by: Guy Harris <guy@alum.mit.edu>
2014-05-13isprint() -> g_ascii_isprint()AndersBroman1-1/+1
Change-Id: Ia586ef8ce500d5fc7578c52014206fa7a7eaea41 Reviewed-on: https://code.wireshark.org/review/1624 Reviewed-by: Anders Broman <a.broman58@gmail.com>
2014-04-15Fix "might be clobbered by 'longjmp' or 'vfork' [-Werror=clobbered]" warning ↵Bill Meier1-5/+5
from gcc 4.9. Change-Id: I0d8f2a03d615840a1733602d0a5a6524caba6ff5 Reviewed-on: https://code.wireshark.org/review/1152 Reviewed-by: Bill Meier <wmeier@newsguy.com> Tested-by: Bill Meier <wmeier@newsguy.com>
2014-03-31packet-osc dissector bug fix of path validity funcHanspeter Portner1-2/+3
- 'is_valid_path' function has been corrected - an OSC path is valid if: - it consists of printable characters only - does not contain ' ' and '#' - characters '*' ',' '?' '[' ']' '{' '}' are valid, but have special meaning at the receiving end (used for pattern matching) Change-Id: I4ff4308d0955da2ef377d606b7778819b97754a0 Reviewed-on: https://code.wireshark.org/review/868 Reviewed-by: Evan Huus <eapache@gmail.com> Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
2014-03-28Add handling of OSC over TCP; fix a bug in the UDP heuristic.Bill Meier1-26/+92
Bug Fixed: UDP heuristic wasn't properly setting the dissector for the UDP conversation. From https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9837#c3: I did a little survey about alternative OSC transmission, but UDP. As OSC is packet-based, it is tramitted raw via packet-oriented protocols (e.g. UDP). For reliable stream-based protocols (TCP, USB), the raw OSC packet is prefixed with the packets Int32 size as a packetization scheme. For unreliable stream-based protocols (RS232 and other serial lines), the raw OSC packet is SLIP and/or double SLIP encoded as packetization scheme. There was discussion in the past to make SLIP encoding the default for all stream-based protocols, but apparently it has never been adopted for any OSC via TCP implementation I've found in the web. As OSC is used in networked Audio, most implementations run with the Nagle algorithm disable, and send the prefixed length and the raw OSC packet separately. Change-Id: Ife690cc5ea0575c65124a7b441431e1cc6ba5091 Reviewed-on: https://code.wireshark.org/review/858 Reviewed-by: Bill Meier <wmeier@newsguy.com> Tested-by: Bill Meier <wmeier@newsguy.com>
2014-03-17Try to fix trivial varningsAndersBroman1-0/+3
Change-Id: I7b09999fe34ffded73f0b26c3e561b39d2120643 Reviewed-on: https://code.wireshark.org/review/714 Reviewed-by: Anders Broman <a.broman58@gmail.com>
2014-03-14Fix two bugs & do various other changes.Bill Meier1-79/+99
Bugs fixed: - Catch exceptions during heuristics test so that the sequence of dissector heuristics tests is not terminated abnormally; (Prevents incorrect tshark "one-pass" dissection); - Comment out registration of heuristic for TCP; TCP dissection requires different code than for UDP. ("XXX: ToDo" added) Misc: - Create/use two extended value_strings; - "UL" is not needed as a modifier for several constants; - Remove some unneeded initializers; - Localize some variables; - Remove unneeded '#include <stdlib.h> - Do some whitespace changes. Change-Id: Ida11cb6b26911c0032155fde7491dd2a6f136c34 Reviewed-on: https://code.wireshark.org/review/650 Reviewed-by: Bill Meier <wmeier@newsguy.com> Tested-by: Bill Meier <wmeier@newsguy.com>
2014-03-06Open Sound Control dissector.Hanspeter Portner1-0/+848
Dissector for specification at http://opensoundcontrol.org/spec-1_0. - use value_string - add integer oferflow protection - remove trailing white space - add capture file to bug tracker: Bug 9837 - fix warnings by tools/fix-encodings-args.pl - make use of VALS - dissect MIDI controller messages separately Change-Id: Iab0dc01d4bf1c08eac175b6af1dd07809090c35a Reviewed-on: https://code.wireshark.org/review/454 Reviewed-by: Evan Huus <eapache@gmail.com> Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com> Tested-by: Alexis La Goutte <alexis.lagoutte@gmail.com>