Index: [Article Count Order] [Thread]

Date: Tue, 23 Jun 2009 14:46:10 +0900
From: Yukiyo Akisada <akisada@tahi.org>
Subject: [users:01230] Re: [PATCH] ipsec: fix the TCP ACK packet definition in ipsec.p2/p2_HTR_TCP_common.def
To: Wei Yongjun <yjwei@cn.fujitsu.com>
Cc: users@tahi.org
Message-Id: <20090623144610.b1c40313.akisada@tahi.org>
In-Reply-To: <4A1B55AF.1020905@cn.fujitsu.com>
References: <4A1B55AF.1020905@cn.fujitsu.com>
X-Mail-Count: 01230

Hi, Yongjun.

You are right.

I applied this fix into our source code,
even though the current IPsec_Self_Test_P2 turned off TCP test cases.

Thank you very much!


On Tue, 26 May 2009 10:36:31 +0800
Wei Yongjun <yjwei@cn.fujitsu.com> wrote:

> The definition of TCP ACK packet is the same as SYN packet, which
> is used by p2_HTR_E_TCP_tunnelPadding.seq. So the TCP connection
> can not be established. The ACK packet name is:
> 'esptun_from_sg1_net1_ack_request_from_host1_net2_to_host0_net0'
> 
> This patch fixed the packet definition.
> 
> Signed-off-by: Wei Yongjun <yjwei@cn.fujitsu.com>
> ---
>  ipsec.p2/p2_HTR_TCP_common.def |    3 ++-
>  1 files changed, 2 insertions(+), 1 deletions(-)
> 
> diff --git a/ipsec.p2/p2_HTR_TCP_common.def b/ipsec.p2/p2_HTR_TCP_common.def
> index 03f63cd..d533f08 100644
> --- a/ipsec.p2/p2_HTR_TCP_common.def
> +++ b/ipsec.p2/p2_HTR_TCP_common.def
> @@ -101,7 +101,8 @@ FEM_tcp6(
>  		_SPORT(10000);
>  		_DPORT(DESTINATION_PORT);
>  		SequenceNumber       = SEQ;
> -		SYNFlag              = 1;
> +		AcknowledgmentNumber = ACK;
> +		ACKFlag              = 1;
>  //		Window               = 16384;
>  //		option               = opt_syn;
>  	},
> -- 
> 1.5.3.8
> 
> 
> 
> 
> 


-- 
Yukiyo Akisada <akisada@tahi.org>