Thanks, I pushed it. Thanks for fixing that typo that I checked in that broke the XenServer build, by the way.
On Wed, Jul 27, 2011 at 08:10:51PM -0700, Ethan Jackson wrote: > Looks obviously correct. > > Ethan > > On Wed, Jul 27, 2011 at 17:08, Ben Pfaff <[email protected]> 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 <[email protected]> > > --- > > ?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 > > [email protected] > > http://openvswitch.org/mailman/listinfo/dev > > _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
