> 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>

Acked-by: YAMAMOTO Takashi <yamam...@valinux.co.jp>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to