From mboxrd@z Thu Jan  1 00:00:00 1970
From: Michael Tremer <michael.tremer@ipfire.org>
To: development@lists.ipfire.org
Subject: Re: [PATCH] suricata: Disable sid 2210059.
Date: Thu, 09 Dec 2021 16:39:56 +0000
Message-ID: <983629B3-3B46-4692-8431-09C41C042398@ipfire.org>
In-Reply-To: <20211208171805.309048-1-stefan.schantl@ipfire.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============4723926190682362288=="
List-Id: <development.lists.ipfire.org>

--===============4723926190682362288==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable

Reviewed-by: Michael Tremer <michael.tremer(a)ipfire.org>

> On 8 Dec 2021, at 17:18, Stefan Schantl <stefan.schantl(a)ipfire.org> wrote:
>=20
> This rule emits a massive logspam and temporary will be disabled until
> a better solution is found.
>=20
> Fixes #12738.
>=20
> Signed-off-by: Stefan Schantl <stefan.schantl(a)ipfire.org>
> ---
> lfs/suricata                                   |  1 +
> src/patches/suricata-disable-sid-2210059.patch | 12 ++++++++++++
> 2 files changed, 13 insertions(+)
> create mode 100644 src/patches/suricata-disable-sid-2210059.patch
>=20
> diff --git a/lfs/suricata b/lfs/suricata
> index 96c2b33fe..6a24a02ab 100644
> --- a/lfs/suricata
> +++ b/lfs/suricata
> @@ -71,6 +71,7 @@ $(TARGET) : $(patsubst %,$(DIR_DL)/%,$(objects))
> 	@$(PREBUILD)
> 	@rm -rf $(DIR_APP) && cd $(DIR_SRC) && tar zxf $(DIR_DL)/$(DL_FILE)
> 	cd $(DIR_APP) && patch -Np1 < $(DIR_SRC)/src/patches/suricata-5.0-stream-t=
cp-Handle-retransmitted-SYN-with-TSval.patch
> +	cd $(DIR_APP) && patch -Np1 < $(DIR_SRC)/src/patches/suricata-disable-sid=
-2210059.patch
> 	cd $(DIR_APP) && LDFLAGS=3D"$(LDFLAGS)" ./configure \
> 		--prefix=3D/usr \
> 		--sysconfdir=3D/etc \
> diff --git a/src/patches/suricata-disable-sid-2210059.patch b/src/patches/s=
uricata-disable-sid-2210059.patch
> new file mode 100644
> index 000000000..54747dfd2
> --- /dev/null
> +++ b/src/patches/suricata-disable-sid-2210059.patch
> @@ -0,0 +1,12 @@
> +diff -Nur a/rules/stream-events.rules b/rules/stream-events.rules
> +--- a/rules/stream-events.rules	2021-11-17 16:55:12.000000000 +0100
> ++++ b/rules/stream-events.rules	2021-12-08 18:12:39.850189502 +0100
> +@@ -89,7 +89,7 @@
> + # rule to alert if a stream has excessive retransmissions
> + alert tcp any any -> any any (msg:"SURICATA STREAM excessive retransmissi=
ons"; flowbits:isnotset,tcp.retransmission.alerted; flowint:tcp.retransmissio=
n.count,>=3D,10; flowbits:set,tcp.retransmission.alerted; classtype:protocol-=
command-decode; sid:2210054; rev:1;)
> + # Packet on wrong thread. Fires at most once per flow.
> +-alert tcp any any -> any any (msg:"SURICATA STREAM pkt seen on wrong thre=
ad"; stream-event:wrong_thread; sid:2210059; rev:1;)
> ++#alert tcp any any -> any any (msg:"SURICATA STREAM pkt seen on wrong thr=
ead"; stream-event:wrong_thread; sid:2210059; rev:1;)
> +=20
> + # Packet with FIN+SYN set
> + alert tcp any any -> any any (msg:"SURICATA STREAM FIN SYN reuse"; stream=
-event:fin_syn; classtype:protocol-command-decode; sid:2210060; rev:1;)
> --=20
> 2.30.2
>=20


--===============4723926190682362288==--