On Sun, Oct 23, 2016 at 1:50 PM, Ben Pfaff <b...@ovn.org> wrote: > The order of src and dst was swapped both in assignment and reference, > which meant that the result worked OK but was really confusing to try to > extend or modify. > > Signed-off-by: Ben Pfaff <b...@ovn.org> >
Acked-by: Russell Bryant <russ...@ovn.org> > --- > tests/ovn.at | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/tests/ovn.at b/tests/ovn.at > index da0291f..1d782a3 100644 > --- a/tests/ovn.at > +++ b/tests/ovn.at > @@ -1592,8 +1592,8 @@ for i in 1 2; do > done > done > test_packet() { > - local inport=$1 src=$2 dst=$3 eth=$4; shift; shift; shift; shift > - local packet=${src}${dst}${eth} > + local inport=$1 dst=$2 src=$3 eth=$4; shift; shift; shift; shift > + local packet=${dst}${src}${eth} > hv=`vif_to_hv $inport` > vif=vif$inport > as $hv ovs-appctl netdev-dummy/receive $vif $packet > -- > 2.1.3 > > _______________________________________________ > dev mailing list > d...@openvswitch.org > http://openvswitch.org/mailman/listinfo/dev > -- Russell Bryant _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev