github-actions: bump actions/upload-artifact from 4 to 5

Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 4 to 5.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](https://github.com/actions/upload-artifact/compare/v4...v5)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-version: '5'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

doc/devguide: document eve callback

Document the callback for adding additional data to EVE.

Ticket: #4708

doc/devguide: document eve file types

Ticket: #4708

output/jsonbuilder: helper function SCJbSetPrintAsciiString

To replace C PrintStringsToBuffer and avoid a stack alloc
+ copy

Ticket: 8004

requirements.txt: use suricata-update master

Master makes sense for Suricata main branch as we are on a ways out on a
release.

lua: remove luajit pushlstring workaround

81ee6f5aadeb ("lua: push correct length back through ScFlowvarGet, work around valgrind warning")
added a workaround for valgrind warnings in pushing a string buffer
into the lua state. This is no longer needed as tested with both
address sanitizer and valgrind.

rust/sip: delete redundant computing codes

Delete call of is_alphanumeric where is_alphanumeric and
is_token_char are called together. Four places are modified.

Ticket: 8003

doc/userguide: document reject-both expection policy

Ticket: #5974.

exception-policy: add 'reject-both' option

Allow rejecting both sides of a connection. Has the same support
as regular reject (which is essentially rejectsrc).

Ticket: #5974.

output/eve: fix typos

To accompany documentation work done in

Task #4708

detect/base64_data: Convert unittests to FAIL/PASS API

Ticket: #6320

rust/ike: convert to nom 8

Ticket: #8050

rust/mime: convert to nom 8

Ticket: #8048

rust/mqtt: convert to nom 8

Ticket: #8047

rust/websocket: convert to nom 8

Ticket: #8046

rust/bt-dht: convert to nom 8

Ticket: #8045

rust/rdp: convert to nom 8

Ticket: #8044

rust/telnet: convert to nom 8

Ticket: #8043

rust/ssh: convert to nom 8

Ticket: #8042

rust/rfb: convert to nom 8

Ticket: #8041

rust/pgsql: convert to nom 8

Ticket: #8039

rust/enip: convert to nom 8

Ticket: #8038

rust/tftp: convert to nom 8

Ticket: #8037

rust/dhcp: convert to nom 8

Ticket: #8036

util/var: add NULL check in VarNameStoreRegister

And check return value in entropy keyword setup

detect: fix null deref with entropy keyword

Ticket: 7959

Usage of entropy with base64_data led to NULL dereference

detect/http2: call correct free function on errors

Fix cases where the wrong free function was being called in error
handlers.

DetectHTTP2sizeUpdateSetup was calling DetectHTTP2settingsFree instead
of DetectHTTP2sizeUpdateFree in error case.

Moving http2.priority and http2.window to multi-integers, instead
of basic integers only modified the Free callback, but the
Setup function was still using the direct call to old obsolete free
function.
Using the callback Free abstration in Setup, allows to be
consistent and have less code to change.

flow-timeout: Use yaml config value for CAPTURE_BYPASSED flow

Instead of non configurable constant FLOW_BYPASSED_TIMEOUT

Ticket: #8014

rust: reduce visibility of detect_parse_uint_notending

It is meant as an internal function

Also document the function

detect: urilen keyword fails on trailing junk

Fails especailly on 1<>2 which looks like a range, but is invalid
as too small, and ended up being accepted as =1

Ticket: 8028

plugins: update SC_API_VERSION to 9

So that plugins built for 8 will fail to load with 9 as there
were already breaking changes in the structures and functions
prototypes

detect/filestore: Convert unittest to new FAIL/PASS API

Ticket: #6317

rust/applayertemplate: convert to nom 8

Ticket: #8027

rust/sip+sdp: update to nom 8

Done together as there is a dependence of SDP by SIP.

Ticket: #8025

rust/ftp: update to nom 8

Ticket: #8026

rust/dns: convert to nom 8.0

Ticket: #8024

rust: add nom8 as a dependency

For the update from nom 7 to nom 8.

detect/ip.src: Allow use with transforms

This commit registers ip.src/ip.dst properly so they can be used with
transforms.

Issue: 8015

detect/analyzer: added more details for the ttl keyword

Ticket: #6310

detect-urilen: convert unittests to FAIL/PASS APIs

Ticket: #6325

rule/vars: convert unittests to new FAIL/PASS API

Ticket: #6333

threads: convert unittests for FAIL/PASS APIs

Ticket: #6327

detect: address intermittent UT crash on OpenBSD

Add missing flow memset in test SCSigOrderingTest12 to avoid the crash.

Test SCSigOrderingTest12                                          : Info: unittest-helper: Sid 1 matched 1 times, as expected [UTHCheckPacketMatchResults:util-unittest-helper.c:638]
Info: unittest-helper: Sid 2 matched 1 times, as expected [UTHCheckPacketMatchResults:util-unittest-helper.c:638]
bash: line 144: 88567 Segmentation fault      (core dumped) ./src/suricata -u -l /tmp/

Bug: #8008.

ippair-storage: convert unittests to FAIL/PASS APIs

Ticket: #6315

rust: pin indexmap

Newer indexmap, pulled in by serde_json requires Rust 1.82, so pin
pack to 2.11.4 which still works with Rust 1.75.

detect/dbv: Assert MT lock held

Add a DBV assert to validate that the master->lock is held.

Suppress missing master->lock warning

Issue: 7819

engine/mt: Ensure master lock held for reload

Issue: 7819

DetectEngineReload must hold the `master->lock`; recent changes changed
the locking usages to avoid deadlock when registering/handling tenants.
These changes added the presumption that the master lock is held at a
higher level. Coverity highlighted that the lock is not held
consistently.

doc: minor verbiage tweaks and reST fix

rust: bindgen more http range functions used in http2

Ticket: 7667

rust: bindgen SCHttpRangeFreeBlock

Ticket: 7667

rust: bindgen SCHTTP2MimicHttp1Request

Ticket: 7667

detect: tcp.flags rejects non-sensical values

ignored flags are only meaningful for equal mode

detect: tcp.flags unit tests improvements

Tests better the ignored flags functionality.
This functionality is only used at detection for default mode,
it is not used for any, plus or not mode.

So, have unit tests about igored flags with default mode with
both matching and non matching cases

detect/tcp: make tcp.flags a generic integer with bitflags

Ticket: 6724

Allows to use numerical values for example

Also fixes some unit tests that were returning 1 after goto error
FlagsTestParse05 especially took this path as
de->ignored_flags != (TH_SYN|TH_RST) was false
we had de->ignored_flags == 0xff ^ (TH_SYN|TH_RST)
And then, we had a match, instead of what the not-run code
was supposing.

detect/ipv4: make fragbits a generic uint16 bitflags keyword

Ticket: 6724

Allows to use numerical values

detect/integers: generalize support for bitflags modifier

Ticket: 6724

Allows sugar syntax for bitflags keywords.
While the expressivity does not increase, because we could already
use numerial values with all generic integer modes, this modifier
prefix is used with the strings, and follows the syntax
that is already used for fragbits and tcp.flags keyword.

ippair-bit: Convert unittests to FAIL/PASS APIs

Ticket: #6323

applayer/tls: do not free SAN for decoding error

SSL connp maintains all the state and certificate data that was
parsed/decoded successfully and it must retain that for later usage.
There should be just one place to free this object which is SSLStateFree
for both the directions. By freeing the connp data during parsing error,
there is room for memory errors.
This works so far because the field parsed after this cannot error out so
if there's an error parsing this, it anyway does not exist. However, this
is incorrect and leaves scope for mistakes.

Remove this extra free and treat SAN like all other TLS keywords.

Bug 7996

pcap-log: fix bpf-filter not set for multi mode

Bug: #8002.

libhtp: fix newer clippy lints with newer MSRV

quic/crypto: remove use of GenericArray

Its not needed, and gets rid of the deprecation warnings.

rust: update deps with cargo-update

Suppress deprecation warnings in quick/crypto.rs about GenericArray.

rust: update aes-gcm to 0.10.3

rust: update aes to 0.8.4

detect/integers: count argument for multi-integers

Ticket: 7211

Allows to count the number of elements, without matching on
individual elements

doc: fix enip_command name in json schema

enip.command is not a keyword nor an alias

unittests: move http.cookie tests to FAIL/PASS API

unittests: move http.stat_code tests to FAIL/PASS API

unittests: convert http_raw_header tests to FAIL/PASS api

unittests: move http.stat_msg tests to FAIL/PASS API

unittests: move http.server_body tests to FAIL/PASS API

unittests: clean up memory for host storage tests

unittests: clean up memory for ippair storage tests

output: fix long logline test

Fixes: 023a2fe9ab1f ("unittests: fix format-truncation warning")

github-actions: check for leaks in unittests

unittests: clean up memory for icmpv4 tests

unittests: clean up memory for nsh tests

unittests: clean up memory for ethernet tests

unittests: clean up memory for ipv6 tests

unittests: clean up memory for ipv4 tests

unittests: clean up memory for vlan tests

unittests: clean up memory for esp tests

unittests: clean up memory for raw tests

Convert to FAIL/PASS api

unittests: clean up memory for chdlc tests

unittests: clean up memory for pppoe tests

unittests: clean up memory for gre tests

unittests: clean up memory for ppp tests

unittests: clean up memory for dnp3 tests

unittests: clean up memory for defrag tests

unittests: clean up memory for detect stream_size tests

Convert tests to FAIL/PASS api

unittests: clean up memory for detect dce payload tests

unittests: clean up memory for detect sameip tests

unittests: clean up memory for detect reference tests

unittests: clean up memory for detect rpc tests

unittests: clean up memory for detect replace tests

unittests: clean up memory for detect quic tests

unittests: clean up memory for detect pcre tests

unittests: clean up memory for detect msg tests

unittests: clean up memory for detect l3_proto tests

unittests: clean up memory for detect iprep tests