diff mbox series

[ovs-dev] ovn-controller: pending_ct_zones should be destroy

Message ID OF58BD5DD4.5E287C57-ON482581A1.00202C00-482581A1.0020B824@zte.com.cn
State Superseded
Delegated to: Russell Bryant
Headers show
Series [ovs-dev] ovn-controller: pending_ct_zones should be destroy | expand

Commit Message

xurong00037997 Sept. 20, 2017, 5:57 a.m. UTC
pending_ct_zones in ovn-controller main should be destroy when exit.

Signed-off-by: xu rong <xu.rong@zte.com.cn>

---
 ovn/controller/ovn-controller.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Miguel Angel Ajo Sept. 21, 2017, 11:14 a.m. UTC | #1
Acked-by: Miguel Angel Ajo <majopela@redhat.com>

On Wed, Sep 20, 2017 at 7:57 AM, <xu.rong@zte.com.cn> wrote:

> pending_ct_zones in ovn-controller main should be destroy when exit.
>
> Signed-off-by: xu rong <xu.rong@zte.com.cn>
>
> ---
>  ovn/controller/ovn-controller.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/ovn/controller/ovn-controller.c
> b/ovn/controller/ovn-controller.c
> index a935a79..32cdc9f 100644
> --- a/ovn/controller/ovn-controller.c
> +++ b/ovn/controller/ovn-controller.c
> @@ -843,6 +843,7 @@ main(int argc, char *argv[])
>      pinctrl_destroy();
>
>      simap_destroy(&ct_zones);
> +    shash_destroy(&pending_ct_zones);
>
>      bitmap_free(group_table.group_ids);
>      hmap_destroy(&group_table.desired_groups);
> --
> 1.8.3.1
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>
Miguel Angel Ajo Sept. 21, 2017, 11:15 a.m. UTC | #2
Only fix the commit message s/destroy/destroyed/g

On Thu, Sep 21, 2017 at 1:14 PM, Miguel Angel Ajo Pelayo <
majopela@redhat.com> wrote:

> Acked-by: Miguel Angel Ajo <majopela@redhat.com>
>
> On Wed, Sep 20, 2017 at 7:57 AM, <xu.rong@zte.com.cn> wrote:
>
>> pending_ct_zones in ovn-controller main should be destroy when exit.
>>
>> Signed-off-by: xu rong <xu.rong@zte.com.cn>
>>
>> ---
>>  ovn/controller/ovn-controller.c | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/ovn/controller/ovn-controller.c
>> b/ovn/controller/ovn-controller.c
>> index a935a79..32cdc9f 100644
>> --- a/ovn/controller/ovn-controller.c
>> +++ b/ovn/controller/ovn-controller.c
>> @@ -843,6 +843,7 @@ main(int argc, char *argv[])
>>      pinctrl_destroy();
>>
>>      simap_destroy(&ct_zones);
>> +    shash_destroy(&pending_ct_zones);
>>
>>      bitmap_free(group_table.group_ids);
>>      hmap_destroy(&group_table.desired_groups);
>> --
>> 1.8.3.1
>> _______________________________________________
>> dev mailing list
>> dev@openvswitch.org
>> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>>
>
>
diff mbox series

Patch

diff --git a/ovn/controller/ovn-controller.c 
b/ovn/controller/ovn-controller.c
index a935a79..32cdc9f 100644
--- a/ovn/controller/ovn-controller.c
+++ b/ovn/controller/ovn-controller.c
@@ -843,6 +843,7 @@  main(int argc, char *argv[])
     pinctrl_destroy();
 
     simap_destroy(&ct_zones);
+    shash_destroy(&pending_ct_zones);
 
     bitmap_free(group_table.group_ids);
     hmap_destroy(&group_table.desired_groups);