Commit Graph

79435 Commits

Author SHA1 Message Date
Valerii Zapodovnikov fbccfe9996 DHCPv6: typo 2020-12-29 16:49:46 +00:00
Valerii Zapodovnikov 3f825576d7 DNS: typo in word "minimum" 2020-12-29 16:01:58 +00:00
Pascal Quantin 762bc7beba Update docbook/wsug_src/WSUG_chapter_work.adoc 2020-12-29 15:30:23 +00:00
Valerii Zapodovnikov ecf91c06de WSUG: typo in ip.src example 2020-12-29 15:24:18 +00:00
Jirka Novak b586e194bd RTP player: Size of symbols increased
Size of symbols (Wrong Timestamp, ...) in graph increased from 4pt
to size of main UI font.
2020-12-29 15:00:58 +00:00
Jirka Novak 5a7f249edb RTP player: Added pause button
Pause button added. Button is visible only when media are played.
2020-12-29 14:41:14 +01:00
Valerii Zapodovnikov 936f2a9b70 WSUG: fix incorrect tcp.completeness example 2020-12-29 10:41:49 +00:00
Gerald Combs a7c6de4c66 GitLab CI: Build merge-req:ubuntu-dpkg using Ninja.
ubuntu-dpkg is currently the slowest of the merge-req pipelines. Newer
versions of dh have a "cmake+ninja" buildsystem, so try using it
instead.
2020-12-29 10:09:36 +00:00
Basak Kalfa 9783caebdf PROFINET: According to PN spec, RSI dissections are added.
Current Profinet Spec includes additional RSI features.
As a result, new file is added for PN-RSI protocol.
DCP substitutions related to RSI are added under PN-DCP.
PDRsiInstances record is added under PN-IO.
2020-12-29 08:43:11 +00:00
Darius Davis ed8268fcc0 TFTP: Parse "windowsize" and "msftwindow" options.
Add parsing of RFC 7440 "TFTP Windowsize Option" and the Microsoft-proprietary
"msftwindow" option seen in Windows Deployment Services (WDS) PXE.  I have
found no spec or documentation for the latter, so it is reverse-engineered from
various packet captures of WDS boot.

In the RRQ, WRQ, and OACK, parse the "windowsize" and "msftwindow" options.  I
haven't seen "msftwindow" in an WRQ, only in an RRQ and corresponding OACK, but
I have implemented it symmetrically in the WRQ in this change.

In the ACK, parse the extra byte added when "msftwindow" is active (containing
the dynamic window size) and add it to the protocol tree.  Include a "chat"
expert info for changes in window size, since changes in packet queue depth
might indirectly trigger defects in a PXE implementation.
2020-12-29 12:58:39 +10:00
Jirka Novak 9443c14247 RTP: Added 'Play Streams' button to 'RTP Streams' dialog
When capture contains just RTP streams without signalizations, there
is no way how to play many streams in one player (Analyze button plays
just two streams). This patch adds Play Streams button to dialog and
allows a user to select as many streams they wish.
2020-12-28 22:18:59 +00:00
Jaap Keuter c657a6f5e7 pcapng: add missing file format interpretation of EPB options
Add the file format interpretations of Enhanced Packet Block options which
are being read by wiretap, but missing from the file format dissector.
2020-12-28 18:21:54 +00:00
Dario Lombardo 8dcff0e1f5 travis: bump macOS image version.
Fix: #17080.
2020-12-28 17:42:13 +00:00
Martin Mathieson 0f5aad4500 RLC-NR: Add reassembly of AM frames 2020-12-28 15:31:28 +00:00
Đorđije Manojlović 1c2b499e0b Improve R-GOOSE protocol tree
This enables filtering R-GOOSE packets on protocol level, too.
2020-12-28 13:37:33 +00:00
Dario Lombardo 93f6aa2c69 iLBC: fix deprecation type and cast.
Starting from libilbc 3.0.0 the type iLBC_decinst_t has been marked
deprecated.
Other fixes rely on function prototype requiring uint8_t pointer.
2020-12-28 12:45:41 +00:00
Dario Lombardo 1eff9c8982 rbm: fix compiler warnings. 2020-12-28 10:00:46 +00:00
Dario Lombardo 428ecf2013 msgpack: fix some compiler warnings.
../epan/dissectors/packet-msgpack.c: In function ‘dissect_msgpack_string’:
../epan/dissectors/packet-msgpack.c:239:9: warning: pointer targets in assignment from ‘guint8 *’ {aka ‘unsigned char *’} to ‘char *’ differ in signedness [-Wpointer-sign]
  239 |  lvalue = tvb_get_string_enc(wmem_packet_scope(), tvb, *offset + 1 + lensize, len, ENC_NA);
      |         ^
../epan/dissectors/packet-msgpack.c: In function ‘dissect_msgpack_ext’:
../epan/dissectors/packet-msgpack.c:305:9: warning: pointer targets in assignment from ‘const guint8 *’ {aka ‘const unsigned char *’} to ‘const char *’ differ in signedness [-Wpointer-sign]
  305 |   start = tvb_get_ptr(tvb, *offset, bytes);
      |         ^
../epan/dissectors/packet-msgpack.c:306:77: warning: pointer targets in passing argument 6 of ‘proto_tree_add_bytes’ differ in signedness [-Wpointer-sign]
  306 |   proto_tree_add_bytes(ext_tree, hf_msgpack_ext_bytes, tvb, *offset, bytes, start);
      |                                                                             ^~~~~
      |                                                                             |
      |                                                                             const char *
In file included from ../epan/packet.h:14,
                 from ../epan/dissectors/packet-msgpack.c:18:
../epan/proto.h:1536:1: note: expected ‘const guint8 *’ {aka ‘const unsigned char *’} but argument is of type ‘const char *’
 1536 | proto_tree_add_bytes(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start,
      | ^~~~~~~~~~~~~~~~~~~~
../epan/dissectors/packet-msgpack.c:308:37: warning: pointer targets in passing argument 2 of ‘bytes_to_hexstr’ differ in signedness [-Wpointer-sign]
  308 |    *value = bytes_to_hexstr(*value, start, bytes);
      |                                     ^~~~~
      |                                     |
      |                                     const char *
In file included from ../epan/dissectors/packet-msgpack.c:20:
../epan/to_str.h:202:21: note: expected ‘const guint8 *’ {aka ‘const unsigned char *’} but argument is of type ‘const char *’
  202 | WS_DLL_PUBLIC char *bytes_to_hexstr(char *out, const guint8 *ad, guint32 len);
2020-12-28 08:34:15 +00:00
Michael Tuexen a5822f6792 Cleanup debug output and fix indentation 2020-12-27 21:37:39 +00:00
Guy Harris c15a37db51 dumpcap: Improve the secondary error for promiscuous mode failure.
On Windows, some devices don't let promiscuous mode be enabled, and
return an error rather than silently ignoring the request to use
promiscuous mode (as UN*X devices tend to do).  Check for the error
message from that error, and suggest that the user turn off promiscuous
mode on that device.
2020-12-27 12:55:31 -08:00
Gerald Combs c15025f684 [Automatic update for 2020-12-27]
Update manuf, services enterprise numbers, translations, and other items.
2020-12-27 19:32:43 +00:00
Michael Tuexen f918c3372c Bump Qt version to match what is used on the MacOS buildbot 2020-12-27 17:04:13 +01:00
Moshe Kaplan 969c1c0271 packet-rrc: allocate correct type of object
When creating a `flowd_p`, instead of allocating
space for a `gint`, allocate space for a `guint`,
which is the actual type of object used. Also,
switch from `g_malloc` to `g_new`.
2020-12-27 14:11:37 +00:00
Huang Qiangxiong cd2d35c1d2 Protobuf: fix bugs that parsing complex syntax .proto files
Some .proto files contain complex syntax that does not be described in protobuf official site
(https://developers.google.com/protocol-buffers/docs/reference/proto3-spec).

1. Update 'epan/protobuf_lang_parser.lemon' to:
1) Support complex option names format (EBNF):
    optionName = ( ident | "(" fullIdent ")" ) { "." ( ident | "(" fullIdent ")" ) }
for example, "option (complex_opt2).(grault) = 654;".
2) Make enum body support 'reserved' section (EBNF):
    enumBody = "{" { reserved | option | enumField | emptyStatement } "}"
3) Allow the value of field or enumValue option to be "{ ... }" other than constant:
    enumValueOption = optionName  "=" ( constant | customOptionValue ) ";"
    fieldOption = optionName  "=" ( constant | customOptionValue ) ";"
4) Allow 'group' section missing 'label' (for example, in 'oneof' section).
5) Make 'oneof' section support 'option' and 'group' sections (BNF):
    oneof = "oneof" oneofName "{" { oneofField | option | group | emptyStatement } "}"
6) Ignore unused 'extend' section.
7) Fix the bug of one string being splitted into multi-lines.

2. Update 'epan/protobuf_lang_tree.c' to:
8) Fix the bug of parsing repeated option.

3. Update 'test/suite_dissection.py' to add test case for parsing complex syntax .proto files:
  test/protobuf_lang_files/complex_proto_files/unittest_custom_options.proto
  test/protobuf_lang_files/complex_proto_files/complex_syntax.proto
and dependency files:
  test/protobuf_lang_files/well_know_types/google/protobuf/any.proto
  test/protobuf_lang_files/well_know_types/google/protobuf/descriptor.proto

Refer to issue #17046
2020-12-27 11:32:10 +00:00
Moshe Kaplan 5778b2403e wiretap/k12text.l: replace g_malloc with g_new
Replace `g_malloc` with `g_new` to improve
source code readability.
2020-12-27 09:58:01 +00:00
Moshe Kaplan fe6b774358 mate: replace g_malloc with g_new
Replace `g_malloc` with `g_new` to improve
source code readability.
2020-12-27 08:45:54 +00:00
Martin Mathieson b98754dc9e PDCP LTE: Add refs to message formats for security 2020-12-26 18:55:48 +00:00
Jaap Keuter 2104b79479 WSDG: update protocol stats section to match current API
As proposed by Olivier Aveline
2020-12-24 11:27:45 +00:00
Guy Harris f75e2e5956 editcap: one more integer -> Boolean.
selected() returns a Boolean value indicating whether the packet with a
given number was selected by the packet-range arguments or not.
2020-12-23 23:09:19 -08:00
Dr. Lars Völker e8ebebe860 Adding dissector for PDU-Transport
This patch adds a new dissector for PDU-Transport. This is a very
simple protocol to transport CAN, FlexRay, LIN, and other PDUs.
It is typically used to transport legacy messages/PDUs over Ethernet.
The format is compatible to FDN and AUTOSAR.

Enabled AUTOSAR NM on top of the PDU Transport and fixed a few bugs
in AUTOSAR NM as well (length parsing).

Solves #17095
2020-12-24 06:05:27 +00:00
Guy Harris 0d0071a302 editcap: make verbose a Boolean as well.
And, to catch people habituated to -r and -v toggling the flag, fail if
it's already been set, to dishabituate them.

Not having them toggle makes it easier to document their semantics.
2020-12-23 17:55:20 -08:00
Guy Harris d98106876c editcap: treat the "keep packets" flag as a Boolean.
Give it the type gboolean, initialize it to FALSE, and have -r set it to
TRUE.

This makes it a little clearer what it's doing.
2020-12-23 15:49:07 -08:00
Pascal Quantin a9ce232c37 NAS 5GS: upgrade dissector to v16.7.0 2020-12-23 19:12:14 +00:00
Pascal Quantin ca971264a0 NAS EPS: upgrade dissector to v16.7.0 2020-12-23 19:46:52 +01:00
Moshe Kaplan 6bce7b859a Detect and replace bad allocation patterns (more)
Extension of !1413, to improve regex, detect
additional opportunities to replace
`g_malloc` with `g_new`, and fix them.
2020-12-23 14:07:18 +00:00
Moshe Kaplan 180b5e5dc0 Replace more g_malloc with g_new
Replace g_malloc with g_new to improve
source code readability.
2020-12-23 11:47:17 +00:00
Moshe Kaplan 1c7edf49f8 mate_setup: allocate correct amount of memory
When creating a _mate_cfg_gog, instead of
allocating space for a `_mate_cfg_gop`,
allocate space for a _mate_cfg_gog. Also, use
`g_new` instead of `gmalloc`.
2020-12-22 22:53:26 +00:00
Jaap Keuter 38ab00a094 Let the CLI programs tell about how to show their version
Add to the help text the option to show the version of the CLI program.
2020-12-22 21:18:21 +00:00
Gerald Combs b3352bd639 macOS: Force dark mode support.
We initially disabled dark mode support in Info.plist when we didn't
support it very well, and later passively enabled it depending on our
SDK version. Go ahead and force it on since we officially support dark
mode. Closes #17098.
2020-12-22 19:37:22 +00:00
Dario Lombardo e017fd5a04 cmake: fix comment in rpm-build target. 2020-12-22 18:34:52 +00:00
Gerald Combs d45eab1f2d GitLab CI: Don't run locale-gen.
We do so at container build time as of wireshark/wireshark-containers!25.
2020-12-22 18:16:39 +00:00
Dario Lombardo 55bcf95d52 gitlab-ci: replace old notation with extends. 2020-12-22 17:42:15 +00:00
Dario Lombardo 33ab986589 gitlab-ci: fix build:ubuntu-dist. 2020-12-22 17:24:27 +00:00
Dario Lombardo 29b331bf0c make-version: use 0 as number of commits if they can't be obtained.
When running on gitlab-ci, the checkout of the project doesn't give
a complete repo. Then git describe fails at giving the number of commits.
In this case just use 0 instead of NULL.
2020-12-22 17:06:28 +00:00
Moshe Kaplan 5227f385b8 Replace g_malloc calls with g_new
Replace calls of `g_malloc(sizeof(foo_t))` with
`g_new(foo_t, 1)`, to improve readability.
2020-12-22 15:48:48 +00:00
Moshe Kaplan e16166a74c Detect and replace bad allocation patterns
Adds a pre-commit hook for detecting and replacing
occurrences of `g_malloc()` and `wmem_alloc()` with
`g_new()` and `wmem_new()`, to improve the
readability of Wireshark's code, and
occurrences of
`g_malloc(sizeof(struct myobj) * foo)`
with
`g_new(struct myobj, foo)`
to prevent integer overflows

Also fixes all existing occurrences across
the codebase.
2020-12-22 14:56:38 +00:00
Gerald Combs 7b27b444cb GitLab CI: Set CCACHE_DIR globally. 2020-12-22 02:55:43 +00:00
Gerald Combs fc124f45bb GitLab CI: Use shallow clones for merge requests.
We currently disable shallow clones in our general pipeline settings
because jobs that create packages need a usable `git describe`. None of
our merge request jobs create packages, so clone shallowly in order to
speed things up.
2020-12-21 17:19:04 -08:00
Gerald Combs b468b1fb01 GitLab CI: Don't set our ccache size in ubuntu-dist.
We no longer build anything, so there's nothing to cache.
2020-12-21 23:17:13 +00:00
Dario Lombardo a2bc5b5357 idn: prevent heap buffer overflow.
Check index before using it.
Fix: #17102.
2020-12-21 22:57:26 +00:00