aboutsummaryrefslogtreecommitdiffstats
path: root/epan/dissectors/packet-vp8.c
AgeCommit message (Collapse)AuthorFilesLines
2018-02-12dissectors: use SPDX identifiers.Dario Lombardo1-13/+1
Change-Id: I92c94448e6641716d03158a5f332c8b53709423a Reviewed-on: https://code.wireshark.org/review/25756 Petri-Dish: Dario Lombardo <lomato@gmail.com> Reviewed-by: Anders Broman <a.broman58@gmail.com>
2017-08-08We really only need one range_foreach routine.Guy Harris1-4/+4
Rename range_foreach_r() to range_foreach(), getting rid of the old range_foreach(). If your callback doesn't require an additional argument, just pass NULL when calling range_foreach(), and declare the argument as unused. Change-Id: I49a56f90610e39cf2ddc398c9e30ed11a6ca90db Reviewed-on: https://code.wireshark.org/review/23025 Reviewed-by: Guy Harris <guy@alum.mit.edu>
2017-05-06Fixed the column info when displaying VP8 sub protocol to RTP.Robert Jonbgloed1-1/+0
The RTP header information was no longer displayed, and this is very useful information. It certainly looked like the VP8 information was supposed to be extra to the existing info (the dash in the text), and should have been tacked on the end. Change-Id: I1cbefac109a7de7fbc78ac2c274071e20261dd4f Reviewed-on: https://code.wireshark.org/review/21496 Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Michael Mann <mmann78@netscape.net>
2017-01-07Convert range API to always use wmem memory.Michael Mann1-2/+2
This is mostly to address memory leaks in range preferences (the biggest user of range functionality) on shutdown. Now range preferences must use epan scoped memory when referencing internal preference structures to keep consistency. Change-Id: Idc644f59b5b42fa1d46891542b53ff13ea754157 Reviewed-on: https://code.wireshark.org/review/19387 Petri-Dish: Michael Mann <mmann78@netscape.net> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Peter Wu <peter@lekensteyn.nl> Reviewed-by: Michael Mann <mmann78@netscape.net>
2016-12-31Dissectors don't need a journey of self discovery.Michael Mann1-2/+1
They already know who they are when they register themselves. Saving the handle then to avoid finding it later. Not sure if this will increase unnecessary register_dissector functions (instead of using create_dissector_handle in proto_reg_handoff function) when other dissectors copy/paste, but it should make startup time a few microseconds better. Change-Id: I3839be791b32b84887ac51a6a65fb5733e9f1f43 Reviewed-on: https://code.wireshark.org/review/19481 Petri-Dish: Michael Mann <mmann78@netscape.net> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Anders Broman <a.broman58@gmail.com>
2015-12-09new_register_dissector -> register_dissector for dissector directory.Michael Mann1-1/+1
Change-Id: Ie39ef054a4a942687bd079f3a4d8c2cc55d5f22c Reviewed-on: https://code.wireshark.org/review/12485 Petri-Dish: Michael Mann <mmann78@netscape.net> Reviewed-by: Michael Mann <mmann78@netscape.net>
2015-11-14register_dissector -> new_register_dissectorMichael Mann1-3/+5
Picking off "easy" dissectors that only have one or two exit points at most. This concludes a "first pass" over the dissector directory. Change-Id: If5ce5484214be50fe541cba478da1de62e354297 Reviewed-on: https://code.wireshark.org/review/11830 Petri-Dish: Michael Mann <mmann78@netscape.net> Reviewed-by: Michael Mann <mmann78@netscape.net>
2014-09-08Eliminate proto_tree_add_text from some of the dissectors.Michael Mann1-6/+14
Other minor cleanups while in the area. Change-Id: I623d941e53128f169e55dfc629547b4221fa72fc Reviewed-on: https://code.wireshark.org/review/4021 Reviewed-by: Michael Mann <mmann78@netscape.net> Petri-Dish: Michael Mann <mmann78@netscape.net> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Evan Huus <eapache@gmail.com>
2014-08-05Various minor dissector fixes/changesBill Meier1-41/+48
- Fix a few minor bugs; - Remove unneeded #includes; - Do some whitespace/long_lines formatting changes. Change-Id: I97239fa20727498604682239cda0e1b87b10f4bc Reviewed-on: https://code.wireshark.org/review/3434 Petri-Dish: Bill Meier <wmeier@newsguy.com> Reviewed-by: Bill Meier <wmeier@newsguy.com>
2014-07-06convert to proto_tree_add_subtree[_format]Michael Mann1-10/+4
Change-Id: I3efa2a81ab2685cde6eae0a00b24520478a545ce Reviewed-on: https://code.wireshark.org/review/2900 Reviewed-by: Michael Mann <mmann78@netscape.net>
2014-04-12Fix packet-vp8.c:168:6: warning: redundant redeclaration of ↵Alexis La Goutte1-2/+0
'proto_reg_handoff_vp8' [-Wredundant-decls] packet-vp8.c:30:6: note: previous declaration of 'proto_reg_handoff_vp8' was here Change-Id: Ibbae1eeffaa672224e5ea77bda8e207d077b7262 Reviewed-on: https://code.wireshark.org/review/1071 Reviewed-by: Michael Mann <mmann78@netscape.net>
2014-03-04Remove all $Id$ from top of fileAlexis La Goutte1-2/+0
(Using sed : sed -i '/^ \* \$Id\$/,+1 d') Fix manually some typo (in export_object_dicom.c and crc16-plain.c) Change-Id: I4c1ae68d1c4afeace8cb195b53c715cf9e1227a8 Reviewed-on: https://code.wireshark.org/review/497 Reviewed-by: Anders Broman <a.broman58@gmail.com>
2014-03-02Remove unneeded line of code.Bill Meier1-2/+0
Change-Id: Ic2769d5c125d8db7d32b6ad3ab0e4363f513f599 Reviewed-on: https://code.wireshark.org/review/451 Reviewed-by: Bill Meier <wmeier@newsguy.com> Tested-by: Bill Meier <wmeier@newsguy.com>
2014-02-10Fix indent (use 4 spaces and modelines info)Alexis La Goutte1-86/+99
Change-Id: Ib5c57a8302ca065233dd7d827605f8dc333d220c Reviewed-on: https://code.wireshark.org/review/166 Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com> Tested-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
2014-02-10VP8 RTP: fix Windows build failurePascal Quantin1-27/+24
Fix Windows build errors: packet-vp8.c(179) : error C2220: warning treated as error - no 'object' file generated packet-vp8.c(179) : warning C4101: 'vp8_payload_tree' : unreferenced local variable packet-vp8.c(291) : warning C4101: 'ti' : unreferenced local variable packet-vp8.c(599) : warning C4101: 'vp8_name_handle' : unreferenced local variable While we are at it, remove useless _U_ modifiers and NULL assignments. Change-Id: I3d28de7f27a8019cc5424dafc4264107742d7c86 Reviewed-on: https://code.wireshark.org/review/158 Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
2014-02-10VP8 RTP dissectorOwen Williams1-0/+613
Change-Id: Ic21d13bf8ca77ff37c2d2d229c4e8c9d8f053677 Reviewed-on: https://code.wireshark.org/review/124 Reviewed-by: Evan Huus <eapache@gmail.com> Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>