diff mbox series

[iptables] extensions: libipt_DNAT: use size of nf_nat_range2 for rev2

Message ID 20180503194052.14016-1-fw@strlen.de
State Accepted
Delegated to: Pablo Neira
Headers show
Series [iptables] extensions: libipt_DNAT: use size of nf_nat_range2 for rev2 | expand

Commit Message

Florian Westphal May 3, 2018, 7:40 p.m. UTC
DNAT tests fail on nf-next.git, kernel complains about target size
mismatch (40 vs 48), this fixes this for me.

Fixes: 36976c4b5406 ("extensions: libipt_DNAT: support shifted portmap ranges")
Signed-off-by: Florian Westphal <fw@strlen.de>
---
 extensions/libip6t_DNAT.c | 4 ++--
 extensions/libipt_DNAT.c  | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

Comments

Thierry Du Tre May 3, 2018, 8:28 p.m. UTC | #1
On 03-05-18 21:40, Florian Westphal wrote:
> DNAT tests fail on nf-next.git, kernel complains about target size
> mismatch (40 vs 48), this fixes this for me.
>
> Fixes: 36976c4b5406 ("extensions: libipt_DNAT: support shifted portmap ranges")
> Signed-off-by: Florian Westphal <fw@strlen.de>
> ---
>  extensions/libip6t_DNAT.c | 4 ++--
>  extensions/libipt_DNAT.c  | 4 ++--
>  2 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/extensions/libip6t_DNAT.c b/extensions/libip6t_DNAT.c
> index 2a7574b02444..89c5ceb15325 100644
> --- a/extensions/libip6t_DNAT.c
> +++ b/extensions/libip6t_DNAT.c
> @@ -393,8 +393,8 @@ static struct xtables_target dnat_tg_reg[] = {
>  		.version	= XTABLES_VERSION,
>  		.family		= NFPROTO_IPV6,
>  		.revision	= 2,
> -		.size		= XT_ALIGN(sizeof(struct nf_nat_range)),
> -		.userspacesize	= XT_ALIGN(sizeof(struct nf_nat_range)),
> +		.size		= XT_ALIGN(sizeof(struct nf_nat_range2)),
> +		.userspacesize	= XT_ALIGN(sizeof(struct nf_nat_range2)),
>  		.help		= DNAT_help_v2,
>  		.print		= DNAT_print_v2,
>  		.save		= DNAT_save_v2,
> diff --git a/extensions/libipt_DNAT.c b/extensions/libipt_DNAT.c
> index b89d3ca5f0d4..4907a2e83d06 100644
> --- a/extensions/libipt_DNAT.c
> +++ b/extensions/libipt_DNAT.c
> @@ -537,8 +537,8 @@ static struct xtables_target dnat_tg_reg[] = {
>  		.version	= XTABLES_VERSION,
>  		.family		= NFPROTO_IPV4,
>  		.revision	= 2,
> -		.size		= XT_ALIGN(sizeof(struct nf_nat_range)),
> -		.userspacesize	= XT_ALIGN(sizeof(struct nf_nat_range)),
> +		.size		= XT_ALIGN(sizeof(struct nf_nat_range2)),
> +		.userspacesize	= XT_ALIGN(sizeof(struct nf_nat_range2)),
>  		.help		= DNAT_help_v2,
>  		.print		= DNAT_print_v2,
>  		.save		= DNAT_save_v2,
Hi Florian,

I'm going to verify, but that looks like a logical fix indeed.

Thierry
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox series

Patch

diff --git a/extensions/libip6t_DNAT.c b/extensions/libip6t_DNAT.c
index 2a7574b02444..89c5ceb15325 100644
--- a/extensions/libip6t_DNAT.c
+++ b/extensions/libip6t_DNAT.c
@@ -393,8 +393,8 @@  static struct xtables_target dnat_tg_reg[] = {
 		.version	= XTABLES_VERSION,
 		.family		= NFPROTO_IPV6,
 		.revision	= 2,
-		.size		= XT_ALIGN(sizeof(struct nf_nat_range)),
-		.userspacesize	= XT_ALIGN(sizeof(struct nf_nat_range)),
+		.size		= XT_ALIGN(sizeof(struct nf_nat_range2)),
+		.userspacesize	= XT_ALIGN(sizeof(struct nf_nat_range2)),
 		.help		= DNAT_help_v2,
 		.print		= DNAT_print_v2,
 		.save		= DNAT_save_v2,
diff --git a/extensions/libipt_DNAT.c b/extensions/libipt_DNAT.c
index b89d3ca5f0d4..4907a2e83d06 100644
--- a/extensions/libipt_DNAT.c
+++ b/extensions/libipt_DNAT.c
@@ -537,8 +537,8 @@  static struct xtables_target dnat_tg_reg[] = {
 		.version	= XTABLES_VERSION,
 		.family		= NFPROTO_IPV4,
 		.revision	= 2,
-		.size		= XT_ALIGN(sizeof(struct nf_nat_range)),
-		.userspacesize	= XT_ALIGN(sizeof(struct nf_nat_range)),
+		.size		= XT_ALIGN(sizeof(struct nf_nat_range2)),
+		.userspacesize	= XT_ALIGN(sizeof(struct nf_nat_range2)),
 		.help		= DNAT_help_v2,
 		.print		= DNAT_print_v2,
 		.save		= DNAT_save_v2,