On Fri, Dec 05, 2014 at 09:41:44AM -0800, Pravin Shelar wrote:
> On Thu, Dec 4, 2014 at 2:33 PM, Ben Pfaff <b...@nicira.com> wrote:
> > Commit 7eb4b1f1d70345f ("ofp-actions: Support OF1.5 (draft) masked
> > Set-Field, merge with reg_load.") introduced a bug in that a set_field
> > action that set an entire field would be translated incorrectly to
> > reg_load, if the field being set only occupied a portion of the bytes that
> > it contains. For example, an MPLS label is 20 bits but has a 4-byte field,
> > which meant that a set_field would get translated into a reg_load that
> > wrote all 32 bits; in turn, the receiver of that reg_load would reject it
> > because it was attempting to set invalid bits (the top 12 bits).
> >
> > This commit fixes the problem by omitting invalid bits when encoding a
> > reg_load action.
> >
> > Reported-by: Pravin Shelar <pshe...@nicira.com>
> > Signed-off-by: Ben Pfaff <b...@nicira.com>
> This patch fixed the bug.
Thanks for testing, applied to master.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
