On Tue, Oct 22, 2013 at 05:20:39PM -0700, Jarno Rajahalme wrote:
> Make union ofp_action more generic by allowing it to be used for
> all actions.  OFP_ACTION_ALIGN (which is 8, as the sizeof(union
> ofp_action) used to be) is now used where sizeof union ofp_action
> was used before.  On the same note, variable name 'n_actions' is
> changed to 'max_actions' as the number of actions cannot be simply
> divided when actions have various sizes.
> 
> Signed-off-by: Jarno Rajahalme <[email protected]>

I think that this pointer arithmetic in ofpacts_from_openflow() is now
wrong:
            log_bad_action(in, n_in, a - in, error);
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to