[ovs-dev] [bugfixes 3/6] flow: Fix memory leak in minimask_move().

Ethan Jackson ethan at nicira.com
Wed Sep 4 19:53:22 UTC 2013


Acked-by: Ethan Jackson <ethan at nicira.com>


On Wed, Sep 4, 2013 at 12:39 PM, Ben Pfaff <blp at nicira.com> wrote:
> Found by valgrind.
>
> Signed-off-by: Ben Pfaff <blp at nicira.com>
> ---
>  lib/flow.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/lib/flow.c b/lib/flow.c
> index 8263672..0e7c493 100644
> --- a/lib/flow.c
> +++ b/lib/flow.c
> @@ -1373,7 +1373,7 @@ minimask_clone(struct minimask *dst, const struct minimask *src)
>  void
>  minimask_move(struct minimask *dst, struct minimask *src)
>  {
> -    miniflow_clone(&dst->masks, &src->masks);
> +    miniflow_move(&dst->masks, &src->masks);
>  }
>
>  /* Initializes 'dst_' as the bit-wise "and" of 'a_' and 'b_'.
> --
> 1.7.10.4
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev



More information about the dev mailing list