From: Pablo Neira Ayuso <pablo@netfilter.org>
To: Joas Antonio <joasantonio108@gmail.com>
Cc: netdev-bot+sinfo@kernel.org, Florian Westphal <fw@strlen.de>,
Phil Sutter <phil@nwl.cc>,
netfilter-devel@vger.kernel.org, coreteam@netfilter.org,
netdev@vger.kernel.org
Subject: Re: [PATCH nf] netfilter: nf_conntrack_sip: only honour Via: port in original direction
Date: Wed, 7 Oct 2026 21:44:03 +0200 [thread overview]
Message-ID: <asahA40Bh9chMyqP@chamomile> (raw)
In-Reply-To: <CAOCBhwE_JmVTmbuzJc8SEU1ROVyrat=7K-wqTQE=Er0gUy4fLA@mail.gmail.com>
On Wed, Oct 07, 2026 at 12:52:19PM -0300, Joas Antonio wrote:
> Hi,
>
> How the issue was discovered: manual code inspection of
> nf_conntrack_sip.c / nf_nat_sip.c, assisted by an LLM (hence the
> Assisted-by: tag), while auditing the conntrack helpers for
> direction-handling issues.
>
> I then confirmed it at runtime on 602042bf29f6 (QEMU, arm64) with
> client -> router -> server network namespaces, the router running
> nft masquerade and a "sip" ct helper on udp/5060. When the server
> sends a request with "Via: SIP/2.0/UDP <server-ip>:4444", the client
> behind the NAT receives that request and a following plain UDP payload
> on port 4444; with the patch applied both arrive on 5060.
Sorry, this cannot be nf material.
It looks fairly theoretical to me, maybe for correctness patch? Is
this assuming a rogue server and packet spoofing?
> Thanks,
> Joas
>
>
> Em qua., 7 de out. de 2026 às 12:30, <netdev-bot+sinfo@kernel.org> escreveu:
> >
> > Hi!
> >
> > This is an automated message. This series looks like a fix, but its
> > commit messages seem to be missing some information:
> >
> > - How the issue was discovered, e.g. hit in production, hit during
> > development, syzbot report, manual code inspection, LLM or static
> > analysis tool scan.
> >
> > Please do not repost the series just to address the above. Instead,
> > reply to this email with the missing information, so that reviewers
> > can take it into account. If the series needs another revision for
> > other reasons, please include the information in the commit messages
> > then.
> >
> > The evaluation is done by an LLM so it may be wrong, if you think
> > that is the case please reply and explain.
>
>
>
> --
> Joas Antonio dos Santos
> Cyber Security Specialist and Ethical Hacker
> joasantonio108@gmail.com
> LinkedIn
next prev parent reply other threads:[~2026-10-07 19:44 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-10-07 15:25 [PATCH nf] netfilter: nf_conntrack_sip: only honour Via: port in original direction Joas Antonio dos Santos
2026-10-07 15:30 ` netdev-bot+sinfo
2026-10-07 15:52 ` Joas Antonio
2026-10-07 19:44 ` Pablo Neira Ayuso [this message]
2026-10-07 19:58 ` Joas Antonio
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=asahA40Bh9chMyqP@chamomile \
--to=pablo@netfilter.org \
--cc=coreteam@netfilter.org \
--cc=fw@strlen.de \
--cc=joasantonio108@gmail.com \
--cc=netdev-bot+sinfo@kernel.org \
--cc=netdev@vger.kernel.org \
--cc=netfilter-devel@vger.kernel.org \
--cc=phil@nwl.cc \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.