datapath: Use local variable for freeing on flow put error.
authorJesse Gross <jesse@nicira.com>
Fri, 24 Dec 2010 05:50:49 +0000 (21:50 -0800)
committerJesse Gross <jesse@nicira.com>
Wed, 29 Dec 2010 18:43:08 +0000 (10:43 -0800)
commit3d82583c40055f6cc257fc4663c2ac14498c7464
tree995ceab66cff4e53128c6f0a0d33d70e97853bb0
parent1326754973a4ecb53ede31e8d3e38d472ca50091
datapath: Use local variable for freeing on flow put error.

If inserting a flow failed and we need to free the actions, we
currently directly free them from the flow struct.  This is fine
but it makes sparse complain about directly accessing an RCU
protected field.  We could insert some casts to avoid this but
it's cleaner to just free the data from the local variable
instead.

Found with sparse.

Signed-off-by: Jesse Gross <jesse@nicira.com>
Acked-by: Ben Pfaff <blp@nicira.com>
datapath/datapath.c