[ovs-dev] [bug6617] ofproto-dpif: Fix mirror configuration and deconfiguration.

Ethan Jackson ethan at nicira.com
Thu Jul 28 03:10:51 UTC 2011


Looks obviously correct.

Ethan

On Wed, Jul 27, 2011 at 17:08, Ben Pfaff <blp at nicira.com> wrote:
> Deleting or updating a mirror didn't work: mirror_set() didn't set the
> 'aux' member so mirror_lookup() couldn't find the mirror on that basis.
>
> Bug #6617.
> Reported-by: David Tsai <dtsai at nicira.com>
> ---
>  ofproto/ofproto-dpif.c |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c
> index a700d5e..1165570 100644
> --- a/ofproto/ofproto-dpif.c
> +++ b/ofproto/ofproto-dpif.c
> @@ -1267,6 +1267,7 @@ mirror_set(struct ofproto *ofproto_, void *aux,
>         mirror = ofproto->mirrors[idx] = xzalloc(sizeof *mirror);
>         mirror->ofproto = ofproto;
>         mirror->idx = idx;
> +        mirror->aux = aux;
>         mirror->out_vlan = -1;
>         mirror->name = NULL;
>     }
> --
> 1.7.4.4
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
>



More information about the dev mailing list