Age | Commit message (Collapse) | Author | Files | Lines |
|
As "TFS" stands for "true_false_string", "UNS" stands for
"unit_name_string".
|
|
|
|
|
|
Ping #19116
|
|
|
|
The telephony menu items were originally under Statistics, but were
moved to a top level menu (5375b5f4762c2f7a6ca7aaefac77d955433efe79).
Change the name to reflect that, which also changes the Lua names.
For Lua, enable use of the older, deprecated names, because Lua
scripts are used across versions without recompiling.
Fix #17448
|
|
One increment in the process of converting manual array-length computations to
the array_length macro. This change converts about 200 instances in a small
number of files. Another (approx.) 170 instances remain, scattered throughout
a larger number of separate files.
Comparison of the built object files before and after this commit confirms that
the only net effect of this change is in the .debug_line section, so this
commit will not actually change any program behavior. This also helps assure
that all the removed "(TYPE)" expressions did indeed correctly reflect the
array type.
|
|
The C standard requires that objects with static storage duration
is initialized to zero.
|
|
|
|
A tap_id is never 0, so initializing tap variables to -1 is not
necessary.
|
|
Remove init of proto, header field, expert info and subtree variables.
This will reduces the binary size by approximate 1266320 bytes due to
using .bss to zero-initialize the fields.
The conversion is done using the tools/convert-proto-init.py script.
|
|
Manually remove init of hf and ett variable arrays because this
cannot be easily done using the convert script.
|
|
Instead of adding a TFS(&tfs_true_false) to every boolean
field, make it the default if "strings" is NULL.
This seems to match the already existing documentation:
If the Boolean field is to be displayed as "False" or "True", the
'strings' field would be set to NULL.
|
|
|
|
Use the common functions to dissect the MCC and MNC from
packet-e212.c Remove some now no longer used static functions,
arrays, and fields.
|
|
The field description and filter for type of ciphering algorithm
was copied from the previous field.
|
|
This allows flags to be passed by the registering listener
to the collection of information
|
|
Use macros from inttypes.h.
|
|
Use macros from inttypes.h with format strings.
|
|
Several improvements to dissect_description_of_velocity()
- Velocity Type is first 4 bits, do not increase offset after this
- Direction of Vertical Speed is bit 7
- Only increase curr_offset in this function
|
|
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.
|
|
Automated find/replace of wmem_packet_scope() with pinfo->pool in all
files where it didn't cause a build failure.
|
|
|
|
|
|
|
|
The init functions of the gsm statistics tables call gsm_a_stat_init, which
allocates some strings. We have to register gsm_a_stat_free_table_item to
free these table items again.
This is already done correctly for gsm_a_bssmap_stat_table. Fix it for the
other tables.
|
|
Use the new stat_tap_find_table function during init to check if our
statistics table already exists.
If so, we can safely assume that its rows have already beend initialized.
All we have to do is clear the data that was collected by the tap.
|
|
All of the stat_tap_table_ui structs use a single table at offset 0.
|
|
Improve script by ignoring common contractions, dealing with
e.g. \n within strings, and finding multiple concatenated words even
when no camelCase is used.
Also includes some actual spelling fixes.
|
|
|
|
|
|
|
|
It can be useful to track a "call" trough multiple protocols to make
that easier create a cross protocol filter, more may come.
|
|
Fix up some issues where the words were the same except for
captitalisation.
|
|
These appear to be copy/paste errors detected by running
./tools/check_typed_item_calls.py --consecutive
Quite a few issues still remain after this batch.
|
|
Includes small updates to the script and wireshark dictionary.
Probably the last spelling fixes from me for a while.
|
|
Change-Id: Ic72c679ccacd707129574d62d20719e35746a3bf
Reviewed-on: https://code.wireshark.org/review/37401
Petri-Dish: Anders Broman <a.broman58@gmail.com>
Tested-by: Petri Dish Buildbot
Reviewed-by: Anders Broman <a.broman58@gmail.com>
|
|
Fix dead store (Dead assignement/Dead increment) Warning found by Clang
Change-Id: Ic4af292bf494355a78f75df7e00d3c06bfc6eb50
Reviewed-on: https://code.wireshark.org/review/37373
Petri-Dish: Anders Broman <a.broman58@gmail.com>
Tested-by: Petri Dish Buildbot
Reviewed-by: Anders Broman <a.broman58@gmail.com>
|
|
Change-Id: I53b24f55427af53b17233897a71cf5fd70131404
Reviewed-on: https://code.wireshark.org/review/37248
Petri-Dish: Anders Broman <a.broman58@gmail.com>
Tested-by: Petri Dish Buildbot
Reviewed-by: Anders Broman <a.broman58@gmail.com>
|
|
Change-Id: I1ad8a4bd21ed590eae4d64b7a287256860e597ca
Reviewed-on: https://code.wireshark.org/review/36963
Petri-Dish: Anders Broman <a.broman58@gmail.com>
Tested-by: Petri Dish Buildbot
Reviewed-by: Anders Broman <a.broman58@gmail.com>
|
|
Change-Id: I7034cb6304bc7bab5f344b7438ffee6c5ea8844c
Reviewed-on: https://code.wireshark.org/review/35897
Petri-Dish: Anders Broman <a.broman58@gmail.com>
Tested-by: Petri Dish Buildbot
Reviewed-by: Anders Broman <a.broman58@gmail.com>
|
|
- The AVP should be an OctetString
- Orientation of major axis should not be multiplied by 2 according to
the latest standard.
Change-Id: I68532108cc36f4699c10b35ffdbcfaef0c29d9fe
Reviewed-on: https://code.wireshark.org/review/35890
Petri-Dish: Anders Broman <a.broman58@gmail.com>
Tested-by: Petri Dish Buildbot
Reviewed-by: Jaap Keuter <jaap.keuter@xs4all.nl>
Reviewed-by: Anders Broman <a.broman58@gmail.com>
|
|
Change-Id: I970e9314724c55c19d0b98578e54a5d1b598e307
Reviewed-on: https://code.wireshark.org/review/34336
Petri-Dish: Anders Broman <a.broman58@gmail.com>
Tested-by: Petri Dish Buildbot
Reviewed-by: Anders Broman <a.broman58@gmail.com>
|
|
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>
|
|
Change-Id: Id08fe25cdf118e6912fa3f201dfd07bde5ebdaf9
Reviewed-on: https://code.wireshark.org/review/33679
Petri-Dish: Anders Broman <a.broman58@gmail.com>
Tested-by: Petri Dish Buildbot
Reviewed-by: Anders Broman <a.broman58@gmail.com>
|
|
Convert our various PROTO_ITEM_ macros to inline functions and document
them.
Change-Id: I070b15d4f70d2189217a177ee8ba2740be36327c
Reviewed-on: https://code.wireshark.org/review/32706
Reviewed-by: Gerald Combs <gerald@wireshark.org>
Petri-Dish: Gerald Combs <gerald@wireshark.org>
Reviewed-by: Anders Broman <a.broman58@gmail.com>
|
|
This allows taps that can fail to report an error and fail; a failed
tap's packet routine won't be called again, so they don't have to keep
track of whether they've failed themselves.
We make the return value from the packet routine an enum.
Don't have a separate type for the per-packet routine for "follow" taps;
they're expected to act like tap packet routines, so just use the type
for tap packet routines.
One tap packet routine returned -1; that's not a valid return value, and
wasn't one before this change (the return value was a boolean), so
presume the intent was "don't redraw".
Another tap routine's early return, without doing any work, returned
TRUE; this is presumably an error (no work done, no need to redraw), so
presumably it should be "don't redraw".
Clean up some white space while we're at it.
Change-Id: Ia7d2b717b2cace4b13c2b886e699aa4d79cc82c8
Reviewed-on: https://code.wireshark.org/review/31283
Reviewed-by: Guy Harris <guy@alum.mit.edu>
|
|
Change-Id: I12d372afd8a50b9a04f92c53077e79116db6c255
Reviewed-on: https://code.wireshark.org/review/31024
Petri-Dish: Pascal Quantin <pascal.quantin@gmail.com>
Tested-by: Petri Dish Buildbot
Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
|
|
This test does not work when looking for optional elements.
This reverts commit 0cc8feb4b6bd5c2576ae5f11e947d172fe1e6cb1.
Change-Id: Ib6794c647f6e1541b7257a59c4717a6a497c3a8a
Reviewed-on: https://code.wireshark.org/review/30013
Reviewed-by: Anders Broman <a.broman58@gmail.com>
|
|
Change-Id: Iaa84d52c28fcb65cd65963f04a389f5902536eeb
Reviewed-on: https://code.wireshark.org/review/29884
Petri-Dish: Anders Broman <a.broman58@gmail.com>
Tested-by: Petri Dish Buildbot
Reviewed-by: Anders Broman <a.broman58@gmail.com>
|