From: Pablo Neira Ayuso <pablo@netfilter.org>
To: Subham Pal <subhampal789@gmail.com>
Cc: fw@strlen.de, phil@nwl.cc, kaber@trash.net, davem@davemloft.net,
edumazet@google.com, kuba@kernel.org, pabeni@redhat.com,
horms@kernel.org, netfilter-devel@vger.kernel.org,
netdev@vger.kernel.org, stable@vger.kernel.org
Subject: Re: [PATCH net 0/2] netfilter: nf_conntrack_h323: fixes for NAT bypass and ASN.1 decode
Date: Wed, 23 Sep 2026 10:46:01 +0200 [thread overview]
Message-ID: <arORyXjxvZUoEAIw@chamomile> (raw)
In-Reply-To: <CAHeizqHDqx3ue1=0QL92ef+6C63Ur+v1Z=d93smgdD5zTj_-Pw@mail.gmail.com>
Hi,
On Wed, Sep 23, 2026 at 09:50:22AM +0530, Subham Pal wrote:
> This series resolves two separate issues in the Netfilter H.323 helper:
>
> 1. Overly restrictive tuple comparisons across expectation helpers
> (expect_h245, expect_rtp_rtcp, expect_t120, expect_callforwarding)
> that bypass NAT payload mangling under DNAT, hairpin routing, and
> reverse/reply traffic directions.
> 2. An ASN.1 PER decoding bug in decode_seq() where absent extension fields
> are processed before verifying their presence in the extension bitmap,
> leading to packet bitstream desynchronization or premature aborts.
I understand these are correctness fixes.
Do you have a reproducer/PoC? Do you tests for this?
> Subham Pal (2):
> netfilter: nf_conntrack_h323: do not bypass NAT helpers when tuple
> source matches reply destination
> netfilter: nf_conntrack_h323: check extension bitmap before decoding
> components
>
> net/netfilter/nf_conntrack_h323_asn1.c | 6 +++---
> net/netfilter/nf_conntrack_h323_main.c | 24 ++++++------------------
> 2 files changed, 9 insertions(+), 21 deletions(-)
next prev parent reply other threads:[~2026-09-23 8:46 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-09-23 4:20 [PATCH net 0/2] netfilter: nf_conntrack_h323: fixes for NAT bypass and ASN.1 decode Subham Pal
2026-09-23 4:23 ` [PATCH net 1/2] netfilter: nf_conntrack_h323: do not bypass NAT helpers when tuple source matches reply destination Subham Pal
2026-09-23 4:25 ` [PATCH net 2/2] netfilter: nf_conntrack_h323: check extension bitmap before decoding components Subham Pal
2026-09-23 8:46 ` Pablo Neira Ayuso [this message]
2026-09-23 10:46 ` [PATCH net 0/2] netfilter: nf_conntrack_h323: fixes for NAT bypass and ASN.1 decode Subham Pal
2026-10-01 4:39 ` Subham Pal
2026-10-01 8:45 ` Pablo Neira Ayuso
2026-10-01 8:44 ` Pablo Neira Ayuso
2026-10-01 15:58 ` Pablo Neira Ayuso
2026-10-06 3:57 ` Subham Pal
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=arORyXjxvZUoEAIw@chamomile \
--to=pablo@netfilter.org \
--cc=davem@davemloft.net \
--cc=edumazet@google.com \
--cc=fw@strlen.de \
--cc=horms@kernel.org \
--cc=kaber@trash.net \
--cc=kuba@kernel.org \
--cc=netdev@vger.kernel.org \
--cc=netfilter-devel@vger.kernel.org \
--cc=pabeni@redhat.com \
--cc=phil@nwl.cc \
--cc=stable@vger.kernel.org \
--cc=subhampal789@gmail.com \
/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.