Commit Graph

16462 Commits (138898f5c2b8bada9469b4276dc0636da83c66f3)
 

Author SHA1 Message Date
Jason Ish 3f8c3698db threads: helper function TmThreadsWaitForUnpause
The pattern of checking the pause flag, setting to paused then
waiting to unpause was done enough times to factor out into its own
function. This is also needed by library users who bring their own
packet acquisition threads.
2 years ago
Philippe Antoine 7d854bd97f template/detect: allow empty buffers 2 years ago
Philippe Antoine 96c8470cdd template: move detect keywords to pure rust
Ticket: 3195

Also remove unused src/tests/detect-template-buffer.c

Completes commit 4a7567b3f0
to remove references to template-rust
2 years ago
Zemeteri Kamimizu 87e6e9374f conf: init parser after check with stat()
Commit changes are made to avoid possible memory leaks. If the parser
is initialized before configuration file checking, there was no deinit
call before function return. Do check config file existance and type
before YAML parser initialization, so we don't need to deinit parser
before exiting the function.

Bug: #7302
2 years ago
Victor Julien 1591ad4408 packetpool: allow larger max-pending-packets
Original limit was due to a specific data structure.

(lifted from 96a0ffadde)
2 years ago
Jeff Lucovsky 1e0d3435db doc: add napatech plugin upgrade notes
Issue: 7165
2 years ago
Jeff Lucovsky 88451b400d configure: fail on --enable-napatech and --disable-shared
Issue: 7165

Plugins can't be build using the standard autoconf/automake
methods. We can get around this by creating our own Makefiles, but
they're often less portable.

For now, fail during ./configure instead of during compile.
2 years ago
Jeff Lucovsky d32b28c85e napatech: bring back command line argument
Re-introduce support for command line argument "--napatech"

Issue: 7165
2 years ago
Jeff Lucovsky 792f5e1d57 napatech: load plugin by default
Issue: 7165
2 years ago
Jeff Lucovsky d5feb8accc napatech: add as plugin
Issue: 7165
2 years ago
Jeff Lucovsky 330c408930 napatech: remove, to make room for plugin
Issue: 7166
2 years ago
Philippe Antoine 58556b7f8b template: remove -rust references
Ticket: 7315

Completes commit 4a7567b3f0

Allows keyword template.buffer to work properly when template
protocol is enabled
2 years ago
Jason Ish 09d604f7c3 github-ci: install prepared cbindgen on rpm distros
Currently cbindgen from system packages is broken, for now use the
cbindgen artifact we build.
2 years ago
Jason Ish a5e13d0dee github-ci: break out cbindgen installation to action 2 years ago
Jeff Lucovsky c9e4524b3c misc: Eliminate compiler warnings
Issue: 7314

Fixup macro usages to eliminate compiler warnings.
2 years ago
Philippe Antoine 76527dde79 http: fix condition check
Ticket: 7309

Do not use a constant expression in a condition
2 years ago
Alice Akaki d5dd549bcd detect/address: convert ipv4 unittests to FAIL/PASS
Ticket: OISF#6318
2 years ago
Philippe Antoine 40e97423b4 transform/base64: check for 0-sized buffer
So as to avoid undefined behavior with a 0-sized variable length
array

Ticket: #7296
2 years ago
Zemeteri Kamimizu adcac9ee0f detect: add new_de_ctx release in case of errors in initialization
Detect engine tenant reloading function hasn't got engine release call
under error label, so it is possible memory leak in case of errors in
further new detect engine initialization.

Bug: #7303
2 years ago
Jason Ish 6ae5ae701b doc/userguide: generate eve documentation
Add EVE documentation for QUIC and Pgsql to their respective sections of
the userguide.

Also add a complete EVE reference as an appendix.

Other protocols can be done, but its a manual process to document in the
schema, then add the glue to pull them into the documentation.

The documentation is generated during "make dist", or if it doesn't
exist, "conf.py" will attempt to generate the eve documentation for
building on Readthedocs.
2 years ago
Jason Ish 2626895a93 evedoc.py: script to generate rst doc from eve schema
Also supports a "--flat" command line option to produce a "dot"
separated version of all the fields in the EVE schema.
2 years ago
Jason Ish 2b16369071 github-ci: run macos python jobs in virtualenv
With the latest brew changes, a virtualenv is required to install
pyyaml.
2 years ago
Jason Ish d8f73c9215 tm-modules: minor code cleanups
- includes
- loops
2 years ago
Jason Ish 415c5786de threading: remove TM_FLAG_LOGAPI_TM as its not used
This flag is never set, remove.
2 years ago
Philippe Antoine 3f0512ebeb rust: unpin serde
As oss-fuzz uses a newer compatible rustc version
2 years ago
Jason Ish 75b760b450 github-ci: add rpm build job
Build RPMs for Fedora 40 and EPEL 9 (using AlmaLinux).
2 years ago
Jason Ish c9369641f8 ebpf: include llvm_bpfload.h in distribution
Otherwise we fail to build ebpf from a release archive.
2 years ago
Jason Ish 9816a795dd github-ci: cache deb packages on Ubuntu dist builder
Should speed up setup a little.
2 years ago
Shivani Bhardwaj 501f79c0ed detect/engine: deduplicate fn definitions 2 years ago
dependabot[bot] 06dd6bacb0 github-actions: bump actions/checkout from 4.1.7 to 4.2.0
Bumps [actions/checkout](https://github.com/actions/checkout) from 4.1.7 to 4.2.0.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](692973e3d9...d632683dd7)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
2 years ago
dependabot[bot] 0ffd33ab92 github-actions: bump codecov/codecov-action from 4.5.0 to 4.6.0
Bumps [codecov/codecov-action](https://github.com/codecov/codecov-action) from 4.5.0 to 4.6.0.
- [Release notes](https://github.com/codecov/codecov-action/releases)
- [Changelog](https://github.com/codecov/codecov-action/blob/main/CHANGELOG.md)
- [Commits](e28ff129e5...b9fd7d16f6)

---
updated-dependencies:
- dependency-name: codecov/codecov-action
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
2 years ago
dependabot[bot] 908ff2faa3 github-actions: bump github/codeql-action from 3.26.6 to 3.26.10
Bumps [github/codeql-action](https://github.com/github/codeql-action) from 3.26.6 to 3.26.10.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Commits](https://github.com/github/codeql-action/compare/v3.26.6...v3.26.10)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
2 years ago
Alexey Simakov c72404e554 yaml: Add check of allocation for node object
Fix potential dereference of nullptr in case
of unsuccessful allocation of memory for
list node

Bug: #7270
2 years ago
Liam Wilson 592d860434 configure: clear cached header value
Unset ac_cv_header_hs_h before checking for header. This allows
hyperscan to be included in compilation when switching from a non-hs
build to a hs build.
2 years ago
Jason Ish 3b6f709331 threads: don't drop capabilities for packet threads
Remove the call to SCDropCaps for packet processing threads. This
logic in this function is required to setup packet processing even
when the thread is provided by a library user, in which case Suricata
should not be touching is capabilities.

As SCDropCaps is currently a no-op its clear this feature needs to
be (re)designed properly, taking into consideration library users as
well.

Related ticket: https://redmine.openinfosecfoundation.org/issues/2375
2 years ago
Ilya Bakhtin 976dec7f33 detect: pseudo-packets inherit inspect flags from parent packet
Instead of inheriting from flow, because encrypted protocols like TLS
and SSH may have just set the flow flags to indicate rest of stream is
encrypted and does not need to run stream inspection. But inspection
still needs to be run detection on this last flushing packet.

Ticket: #7235.
2 years ago
Philippe Antoine 26da953f6d util/hash: use randomized hash algorithm
For datasets and http ranges

Ticket: 7209

Prevents abusive hash collisions from known djb2 algorithm
2 years ago
Philippe Antoine bb714c9178 http: have a headers limit
Ticket: 7191

So as to avoid quadratic complexity in libhtp.
Make the limit configurable from suricata.yaml,
and have an event when network traffic goes over the limit.
2 years ago
Philippe Antoine 1e152d1f10 ja4: handles non alphanumeric alpn
Ticket: 7267

Follows more closely the specification :
https://github.com/FoxIO-LLC/ja4/blob/main/technical_details/JA4.md#alpn-extension-value

Also fixes the case with a single-char alpn.
2 years ago
Philippe Antoine 9203656496 defrag: fix off by one
Ticket: 7067

This off by one could lead to an empty fragment being inserted
in the rb tree, which led to integer underflow
2 years ago
Victor Julien 45eb7e4881 stream: improve 3whs completed by ACK with data
If the ACK packet completing the 3whs is received, the stream engine will
transition to "established". However, the packet itself will not be tagged
as "established". This will only happen for the next packet after the 3whs,
so that `flow:established` only matches after the 3whs.

It is possible that the ACK packet completing the 3whs was lost. Since the
ACK packets themselves are not acknowledged, there will be no retransmission
of them. Instead, the next packet can have the expected ACK flag as well as
data.

This case was mishandled in a subtle way. The stream engine state transition
was done correctly, as well as the data handling and app-layer updates.
However, the packet itself was not tagged as "established", which meant
that `flow:established` would not yet match.

This patch detects this case and tags the packet as established if ACK
with data is received that completes the 3whs.

Bug: #7264.
2 years ago
Philippe Antoine 521928e2a3 sip: use pure rust function
For internal function that is not part of C FFI
2 years ago
Philippe Antoine c55c7d6c27 detect/dataset: abort only in debug mode
Ticket: 7195
2 years ago
Philippe Antoine e47598110a detect/datasets: implement unset command
Ticket: 7195

Otherwise, Suricata aborted on such a rule
2 years ago
Philippe Antoine 1352ed68c7 datasets: restrict scope of macro/enum 2 years ago
Philippe Antoine 1d32f11745 ssl/ja3: better check for ja3 being enabled
Ticket: 6634

Completes commit 84735251b5

Avoids error log in Ja3BufferAddValue about NULL buffer
2 years ago
Noah Liu 951bcff970 stream/reassembly: optimize GetBlock
Current GetBlock degrees the sbb search from rb tree to
line, which costs much cpu time, and could be replaced by
SBB_RB_FIND_INCLUSIVE. It reduces time complexity from
O(nlogn) to O(logn).

Ticket: 7208.
2 years ago
Juliana Fajardini 18e0d23ed3 docs: remove mentions to Suricata-6
Task #7262
2 years ago
Juliana Fajardini d1d1c8cdac doc/conf/yaml: replace underscore with dashes
Use sed + regex to replace all occurrences of suricata.yaml terms that
used underscore for their up-to-date dash version.

Also search for such terms in the eve-log.yaml partials file, as that
is referenced in the configuration section.

commands used:

sed -i 's/\(^ *[a-z]*\)_\([a-z]*:\)/\1-\2/g'
sed -i 's/\(^ *[a-z]*\)_\([a-z]*\)_\([a-z]*:\)/\1-\2-\3/g'

Some other instances were found manually.

Task #7260
2 years ago
Shivani Bhardwaj 6ff0f72f4d detect/base64: minor cleanups
1. decode_len can be u32 as it stores min of two u32s.
2. Add defensive check for payload_len calculation underflow.
2 years ago