Do not perform validation in learn_parse();
authorSimon Horman <horms@verge.net.au>
Wed, 8 May 2013 01:50:15 +0000 (10:50 +0900)
committerBen Pfaff <blp@nicira.com>
Wed, 8 May 2013 17:46:25 +0000 (10:46 -0700)
commitdd43a558597bbf99a62541bc77e85f86f63e2f12
tree14ce765f66c844a01a266bacde742b55820af00e
parentb019d34da6e6c185bb093984906adcb36c3f6686
Do not perform validation in learn_parse();

I believe this is consistent with the handling of all other action
parsing called from parse_named_action().

Verification of all actions, including learn actions, occurs separately
in ofpact_check__(). It also occurs via in a call to ofpacts_check()
in parse_ofp_str(),

This patch is larger than might otherwise be expected as the flow argument
of learn_parse() is now unused and thus removed.  This propagates up the
call-chain some way.

This implementation was suggested by Jesse Gross in response to an
enhancement I made to the validation performed during parsing learn actions
to allow it to correctly account for changes to the dl_type due to MPLS
push and pop actions.

Tests have also been updated to check for the less specific messages
generated by the call to ofpacts_check() in parse_ofp_str() which at the
suggestion of Ben Pfaff was added by a prior patch for this purpose.

Cc: Jesse Gross <jesse@nicira.com>
Cc: Ben Pfaff <blp@nicira.com>
Signed-off-by: Simon Horman <horms@verge.net.au>
Signed-off-by: Ben Pfaff <blp@nicira.com>
lib/learn.c
lib/learn.h
lib/ofp-parse.c
tests/learn.at