[ovs-dev] [PATCH v2] Fix incorrect handling of return value.

2018-01-24 Thread huanglili
From: Lili Huang The value cookie_offset should be 'size_t' type. Signed-off-by: Lili Huang --- ofproto/ofproto-dpif-xlate.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/ofproto/ofproto-dpif-xlate.c b/ofproto/ofproto-dpif-xlate.c index 40c04cc..e7e8321 100644 --- a/ofp

[ovs-dev] [PATCH v1] ofproto-dpif-xlate: Fix incorrect handling of return value.

2018-01-23 Thread huanglili
From: Lili Huang The value cookie_offset should be 'size_t' type. Signed-off-by: Lili Huang --- ofproto/ofproto-dpif-xlate.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ofproto/ofproto-dpif-xlate.c b/ofproto/ofproto-dpif-xlate.c index 40c04cc..a54ade1 100644 --- a/ofpro

[ovs-dev] [PATCH] ofproto-dpif-xlate: Fix incorrect handling of return values.

2018-01-23 Thread huanglili
From: Lili Huang The value cookie_offset should be 'size_t' type. Change-Id: Id68b83d26b1235048cbcfd21429ad6915a7a2b36 Signed-off-by: Lili Huang --- ofproto/ofproto-dpif-xlate.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ofproto/ofproto-dpif-xlate.c b/ofproto/ofproto-d

[ovs-dev] [PATCH] ovs-ofctl:Fix memory leak when parse colors command.

2017-11-28 Thread huanglili
From: Lili Huang OVS_COLORS variable color_str is parsed by using xstrdup and strsep, we should free original address of the string, not used after strsep. Change-Id: I8ea735b10149f657137937bbd9c883d22bf85d4a Signed-off-by: Lili Huang --- lib/colors.c | 3 ++- 1 file changed, 2 insertions(+),

[ovs-dev] [PATCH v2] conntrack: Fix ct-clean thread crash bug

2017-08-24 Thread huanglili
From: Lili Huang Conn should be removed from the list before freed. This crash will be triggered when a established flow do ct(nat) again, like "ip,actions=ct(table=1) table=1,in_port=1,ip,actions=ct(commit,nat(dst=5.5.5.5)),2 table=1,in_port=2,ip,ct_state=+est,actions=1 table=1,in_port=1,ip,

Re: [ovs-dev] [PATCH] conntrack: Fix ct-clean thread crash bug

2017-08-24 Thread Huanglili (lee)
et back to you today. Darrell On 8/24/17, 3:36 AM, "ovs-dev-boun...@openvswitch.org on behalf of huanglili" wrote: From: Lili Huang Conn should be removed from the list before freed. This crash will be triggered when a establ

[ovs-dev] [PATCH] conntrack: Fix ct-clean thread crash bug

2017-08-24 Thread huanglili
From: Lili Huang Conn should be removed from the list before freed. This crash will be triggered when a established flow do ct(nat) again, like "ip,actions=ct(table=1) table=1,in_port=1,ip,actions=ct(commit,nat(dst=5.5.5.5)),2 table=1,in_port=2,ip,ct_state=+est,actions=1 table=1,in_port=1,ip,