aboutsummaryrefslogtreecommitdiffstats
path: root/epan/dissectors/packet-ua3g.c
AgeCommit message (Collapse)AuthorFilesLines
2021-11-29Fix some spelling errorsMartin Mathieson1-1/+1
2021-09-01tvbuff: convert helper methods to pinfo->poolEvan Huus1-1/+1
A few of them just needed scratch memory, so allocate and free it manually after doing any exception-raising checks. A few others were returning memory, and needed conversion to accept a wmem scope argument.
2021-07-21First pass pinfo->pool conversionEvan Huus1-2/+2
Automated find/replace of wmem_packet_scope() with pinfo->pool in all files where it didn't cause a build failure. I also tweaked a few of the docs which got caught up.
2021-02-02Address more non-static, non-shared symbols.Martin Mathieson1-6/+6
2020-09-28Fix some more wrong filter names.Martin Mathieson1-1/+1
These are duplicates detected seen by running check_typed_item_calls.py --consecutive. There are still quite a few more to go.
2020-08-31More spelling fixes, last part of 2nd pass of dissectors.Martin Mathieson1-3/+3
2020-06-19Fix the type of arrays of pointers to hf_ values for bitfield routines.Guy Harris1-4/+4
The static arrays are supposed to be arrays of const pointers to int, not arrays of non-const pointers to const int. Fixing that means some bugs (scribbling on what's *supposed* to be a const array) will be caught (see packet-ieee80211-radiotap.c for examples, the first of which inspired this change and the second of which was discovered while testing compiles with this change), and removes the need for some annoying casts. Also make some of those arrays static while we're at it. Update documentation and dissector-generator tools. Change-Id: I789da5fc60aadc15797cefecfd9a9fbe9a130ccc Reviewed-on: https://code.wireshark.org/review/37517 Petri-Dish: Guy Harris <gharris@sonic.net> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2019-07-26HTTPS (almost) everywhere.Guy Harris1-1/+1
Change all wireshark.org URLs to use https. Fix some broken links while we're at it. Change-Id: I161bf8eeca43b8027605acea666032da86f5ea1c Reviewed-on: https://code.wireshark.org/review/34089 Reviewed-by: Guy Harris <guy@alum.mit.edu>
2019-02-08ua3g: update of unsolicited messagenbertin1-16/+62
added new fields (vpn & encryption status and wlan status) modified display of hook status and bluetooth support added reserved bytes Change-Id: I74298a636f60c09d593288fecc16dd8c0373c65d Reviewed-on: https://code.wireshark.org/review/31892 Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2019-02-04ua3g: add support for decoding of set-appl-param (id, enable, url) messagenbertin1-0/+65
Change-Id: I57e3b3e6048c5279875c35d772eb7e6f72c2ecbf Reviewed-on: https://code.wireshark.org/review/31850 Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2019-02-03ua3g: updated vta strings (added 8008G and 8058S)nbertin1-2/+2
Change-Id: I1a0c44053f82488b2a1efe1700919dba16dae52b Reviewed-on: https://code.wireshark.org/review/31849 Reviewed-by: Jaap Keuter <jaap.keuter@xs4all.nl> Petri-Dish: Jaap Keuter <jaap.keuter@xs4all.nl> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2019-02-03ua3g: updated skins namenbertin1-3/+3
Change-Id: I63a4c7777c8b08d154fd5577c84e1123b1b890a6 Reviewed-on: https://code.wireshark.org/review/31848 Reviewed-by: Jaap Keuter <jaap.keuter@xs4all.nl> Petri-Dish: Jaap Keuter <jaap.keuter@xs4all.nl> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2019-01-24ua3g: updated freeseating message decoding to handle the "do_reset" parameterNicolas Bertin1-0/+8
Change-Id: I66bfed59301a6f149819b345d03c534993f36948 Reviewed-on: https://code.wireshark.org/review/31711 Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2018-06-08ua3g: updated freeseating message decoding to support ipv6 address in ↵Nicolas Bertin1-1/+16
maincpu0 and maincpu1 parameters. Change-Id: Ia0731906e7daaabd9e159aa95db76f54345a927a Reviewed-on: https://code.wireshark.org/review/28058 Reviewed-by: Jaap Keuter <jaap.keuter@xs4all.nl> Petri-Dish: Jaap Keuter <jaap.keuter@xs4all.nl> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2018-05-29ua3g: added setup of rtp/rtcp converstations from record-rtp messageNicolas Bertin1-2/+32
Change-Id: I167756da6cba0734cd0280528f9b5a303675d5b9 Reviewed-on: https://code.wireshark.org/review/27872 Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2018-05-28ua3g: updated decoding of set-skin-id messageNicolas Bertin1-0/+1
Change-Id: Id7c06ac8d25b4a2523604178389a2369b4f85d4c Reviewed-on: https://code.wireshark.org/review/27871 Reviewed-by: Anders Broman <a.broman58@gmail.com>
2018-05-28ua3g: fixed decoding of start-tone messageNicolas Bertin1-5/+5
Change-Id: I6ec9742b26daaa63d11c1fa47ec70fb5e6276848 Reviewed-on: https://code.wireshark.org/review/27868 Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2018-05-25ua3g: move if in the fallthrough case.Dario Lombardo1-2/+2
From gcc version 7 this causes an error: ../epan/dissectors/packet-ua3g.c:1293:28: error: this statement may fall through [-Werror=implicit-fallthrough=] if (parameter_id == 0x02) ^ ../epan/dissectors/packet-ua3g.c:1295:21: note: here case 0x03: /* Type Of Service */ ^~~~ cc1: all warnings being treated as errors Fixes: 31663c40 Change-Id: I7622614e5472cc70aac5690d4bfc327be7628ece Reviewed-on: https://code.wireshark.org/review/27800 Reviewed-by: Anders Broman <a.broman58@gmail.com>
2018-05-25ua3g: moved setup of rtp/rtcp conversations from packet-ua to packet-ua3g, ↵Nicolas BERTIN1-4/+36
where the start-rtp message is actually decoded. Change-Id: Id0dc39107f43680f082d5e6a1d8a1d4940640a13 Reviewed-on: https://code.wireshark.org/review/27792 Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
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-10ua3g: fixed VTA type display for 8018, 8028S and 8008Nicolas Bertin1-2/+3
Change-Id: I24158599831e432b62ff2ec0f36eeb1350c18a4b Reviewed-on: https://code.wireshark.org/review/23031 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: Pascal Quantin <pascal.quantin@gmail.com>
2017-05-16ua3g: fixed START_TONE message decodingNicolas Bertin1-39/+14
Change-Id: Ib0cdf793d728b9bdc39754878e878e41e09309de Reviewed-on: https://code.wireshark.org/review/21663 Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Anders Broman <a.broman58@gmail.com>
2017-05-16ua3g: fixed QOS_TICKET_RSP message decodingNicolas BERTIN1-111/+97
Change-Id: Ib0b5e0acb214769b981cb7be75cbafb8ed37403b Reviewed-on: https://code.wireshark.org/review/21623 Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Anders Broman <a.broman58@gmail.com>
2017-05-09ua3g: fix conflicting entry in its value_stringAlexis La Goutte1-2/+2
Field 'Audio Env.' (ua3g.ip.set_param_req.parameter.audio_env) has a conflicting entry in its value_string: 2 is at indices 2 (Open Space) and 3 (Meeting Room)) Field 'Audio Env.' (ua3g.ip.set_param_req.parameter.audio_env) has a conflicting entry in its value_string: 2 is at indices 2 (Open Space) and 4 (Noisy)) Field 'Audio Env.' (ua3g.ip.set_param_req.parameter.audio_env) has a conflicting entry in its value_string: 2 is at indices 3 (Meeting Room) and 4 (Noisy)) Change-Id: I495f183dd52e59ec64e6eea0df15a5b243d8b87c Reviewed-on: https://code.wireshark.org/review/21568 Reviewed-by: Nicolas BERTIN <nicolas.bertin@al-enterprise.com> Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com> 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-05-06ua3g: updated GET_PARAM_RESPONSE message decoding (freeseating and ethernet ↵Nicolas BERTIN1-7/+19
config) Change-Id: Ie67c2e61c031de5b48310d1fc8940808481b725b Reviewed-on: https://code.wireshark.org/review/21504 Petri-Dish: Michael Mann <mmann78@netscape.net> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Michael Mann <mmann78@netscape.net>
2017-05-04ua3g: added SET_SIP_PARAMETERS and FREESEATING messages decodingNicolas BERTIN1-0/+63
Change-Id: I6e00edfa09dea6f9f8c24b72769257ce428bfc61 Reviewed-on: https://code.wireshark.org/review/21481 Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Anders Broman <a.broman58@gmail.com>
2017-05-03ua3g: updated SET_PARAM_REQ message decodingNicolas BERTIN1-0/+101
Change-Id: Ic20286111b39539b5c423cc2442be0533d0b631e Reviewed-on: https://code.wireshark.org/review/21468 Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Anders Broman <a.broman58@gmail.com>
2017-05-03ua3g: updated START_RECORD_RTP message decodingNicolas BERTIN1-0/+4
Change-Id: I98d2aa5098d3019f829e75e46a0842a9d87786aa Reviewed-on: https://code.wireshark.org/review/21469 Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com> Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Anders Broman <a.broman58@gmail.com>
2017-05-02ua3g: updated RESET message decoding (appl_mode, sip_name and reset_mac ↵Nicolas Bertin1-0/+15
parameters) Change-Id: I4604b4de036f5e2d43a6e4f52e64576d63f2c8ed Reviewed-on: https://code.wireshark.org/review/21449 Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Anders Broman <a.broman58@gmail.com>
2017-05-02ua3g: updated REDIRECT message decoding (redundant maincpu parameter)Nicolas Bertin1-1/+1
Change-Id: I4c703d380010ed784946a0093307423c227d37eb Reviewed-on: https://code.wireshark.org/review/21450 Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com> 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-04-27ua3g: update decoding of START_RTP message (new compressors and new parameter)Nicolas BERTIN1-13/+10
Change-Id: Ic5bd9afc2af14a64d074d598f1e185252308008b Reviewed-on: https://code.wireshark.org/review/21357 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-04-26ua3g: fixed "GET_PARAM_RESPONSE" message decoding (firmware version) and displayNicolas Bertin1-2/+17
Change-Id: Ifb834d4895fda129878657468d21e3afc65cc5d3 Reviewed-on: https://code.wireshark.org/review/21339 Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Anders Broman <a.broman58@gmail.com>
2017-04-26ua3g: minor change in "UNSOLICITED" message decodingNicolas BERTIN1-0/+1
Display "Not Applicable" instead of "Unknown" when the "Additional VTA Type" is not set (0), which is the case with 2G and 3G terminals. Change-Id: I178248e1de18176710fe978ebf90e99486e0ec7e Reviewed-on: https://code.wireshark.org/review/21346 Reviewed-by: Anders Broman <a.broman58@gmail.com>
2017-04-25ua3g: fixed/updated "unsolicited" and "version information" messagesNicolas BERTIN1-29/+81
Change-Id: If006c36a21556a8fb690401677e9ca62a594d82b Reviewed-on: https://code.wireshark.org/review/21307 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: Anders Broman <a.broman58@gmail.com>
2017-04-22ua3g: fixed/updated init message (vta type values)Nicolas BERTIN1-5/+11
Change-Id: I8c500d6e049e9243c2c4b283731674bf3fc0a521 Reviewed-on: https://code.wireshark.org/review/21282 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-03-19Fix: UA3G read past end of tvb + NOE key descriptionserikdejong1-1/+1
packet-ua3g.c: Start tone device routing packet causes exception because it adds a subtree with length 6 where it should be 3. Changed 6 to 3. packet-noe.c: Key description is incomplete because buffer reserved is only 10 bytes long and key descriptions are up to 23 bytes long. Changed buffer length to 24. Bug: 13502 Change-Id: I8212d23c650efd341e87f766a80a887ef82f4b0b Reviewed-on: https://code.wireshark.org/review/20633 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-03-05dissectors: fix this statement may fall through ↵Alexis La Goutte1-1/+1
[-Werror=implicit-fallthrough=] found by gcc7 Change-Id: Iba6238988ded675cba328ab512232d1919d93b4a Reviewed-on: https://code.wireshark.org/review/20415 Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com> Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com> Reviewed-by: Michael Mann <mmann78@netscape.net>
2016-12-13Adjust proto_tree_add_uint_format_value calls to use unit stringMichael Mann1-1/+1
Several calls to proto_tree_add_uint_format_value could be better served using BASE_UNIT_STRING with a "unit string" in hf_ field. There also a few cases where proto_tree_add_uint_format_value could just be proto_tree_add_uint. Added a few more "common" unit string values to unit_strings.[ch] Change-Id: Iaedff82c515269c9c31ab9100dff19f5563c932d Reviewed-on: https://code.wireshark.org/review/19242 Petri-Dish: Michael Mann <mmann78@netscape.net> Reviewed-by: Michael Mann <mmann78@netscape.net>
2016-07-21Add proto_tree_add_checksum.Michael Mann1-1/+1
This is an attempt to standardize display/handling of checksum fields for all dissectors. The main target is for dissectors that do validation, but dissectors that just report the checksum were also included just to make them easier to find in the future. Bug: 10620 Bug: 12058 Ping-Bug: 8859 Change-Id: Ia8abd86e42eaf8ed50de6b173409e914b17993bf Reviewed-on: https://code.wireshark.org/review/16380 Petri-Dish: Michael Mann <mmann78@netscape.net> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Jeff Morriss <jeff.morriss.ws@gmail.com> Reviewed-by: Michael Mann <mmann78@netscape.net>
2016-01-08Fix a lot of typos and misspellingsmoshekaplan1-1/+1
Change-Id: I8512cfa1d424f82a873a0e0e1d22c7b075fdd7f3 Reviewed-on: https://code.wireshark.org/review/13069 Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com> 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>
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-08-24Eliminate proto_tree_add_text from some dissectorsMichael Mann1-91/+103
Change-Id: Id710ab10093227b27ef5f18b0d2960e31d0b95a9 Reviewed-on: https://code.wireshark.org/review/10200 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: Anders Broman <a.broman58@gmail.com>
2015-06-22Remove a bunch of deprecated tvb_length callsEvan Huus1-1/+1
Change-Id: I9362e0fdc4519ba5f3d656152966e7030f478839 Reviewed-on: https://code.wireshark.org/review/9022 Petri-Dish: Evan Huus <eapache@gmail.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Evan Huus <eapache@gmail.com>
2015-02-18Add a CF_FUNC macro for casting BASE_CUSTOM functions.Gerald Combs1-7/+7
Add a CF_FUNC macro to match VALS, TFS, etc. This should help us to avoid the following warning: warning: ISO C forbids initialization between function pointer and 'void *' [-Wpedantic] We could start adding DIAG_OFF+DIAG_ON everywhere but this seems to be more consistent with the other macros in proto.h. Update each instance of BASE_CUSTOM to use CF_FUNC. Adjust a dummy variable name generated by asn2wrs.py that was triggering an invalid error in checkhf.pl. Fix an encoding arguement in packet-elasticsearch.c found by fix-encoding-args.pl. Change-Id: Id0e75076c2d71736639d486f47b87bab84e07d22 Reviewed-on: https://code.wireshark.org/review/7150 Reviewed-by: Gerald Combs <gerald@wireshark.org>
2014-12-21Cleanup use of #includes in non-generated epan/dissector/*.cBill Meier1-3/+0
Specifically: - Set packet.h to be the first wireshark #include after config.h and "system" #includes. packet.h added as an #include in some cases when missing. - Remove some #includes included (directly/indirectly) in packet.h. E.g., glib.h. (Done only for those files including packet.h). - As needed, move "system" #includes to be after config.h and before wireshark #includes. - Rework various #include file specifications for consistency. - Misc. Change-Id: Ifaa1a14b50b69fbad38ea4838a49dfe595c54c95 Reviewed-on: https://code.wireshark.org/review/5923 Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com> Reviewed-by: Bill Meier <wmeier@newsguy.com>
2014-12-13Use ENC_BIG_ENDIAN when fetching FT_U?INT8 fields ...Bill Meier1-169/+169
(for some dissectors which fetch all other integral fields using ENC_BIG_ENDIAN). Change-Id: Ic18e3172aad76af12b12d6732c88497be22aed56 Reviewed-on: https://code.wireshark.org/review/5748 Reviewed-by: Bill Meier <wmeier@newsguy.com>
2014-10-20Get rid of unnecessary includes of ctype.h.Guy Harris1-2/+0
Change-Id: I2cf49f808558147ce77e7d086558966cfb2defca Reviewed-on: https://code.wireshark.org/review/4850 Reviewed-by: Guy Harris <guy@alum.mit.edu>
2014-09-30Add editor modelines; Adjust whitespace as needed.Bill Meier1-3/+16
Change-Id: I434da226c842298f4fb2a4335d06d51e164af2af Reviewed-on: https://code.wireshark.org/review/4394 Reviewed-by: Bill Meier <wmeier@newsguy.com>
2014-07-07convert to proto_tree_add_subtree[_format]Michael Mann1-30/+17
Change-Id: Ie10ac4f5f04a23344d183e095bbafb23c6409144 Reviewed-on: https://code.wireshark.org/review/2904 Reviewed-by: Michael Mann <mmann78@netscape.net>
2014-06-19Revert "Fixup: tvb_* -> tvb_captured"Michael Mann1-1/+1
https://www.wireshark.org/lists/wireshark-dev/201406/msg00131.html This reverts commit 246fe2ca4c67d8c98caa84e2f57694f6322e2f96. Change-Id: Ib24bae0198c13a84bd7f731bf4af921212109a8f Reviewed-on: https://code.wireshark.org/review/2430 Reviewed-by: Michael Mann <mmann78@netscape.net>