projects / openvswitch / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 583fce1) | patch
datapath: Avoid double-free on skb_clone failure in ODPAT_OUTPUT_GROUP.
authorBen Pfaff <blp@nicira.com>
Wed, 27 May 2009 18:37:08 +0000 (11:37 -0700)
committerBen Pfaff <blp@nicira.com>
Wed, 27 May 2009 18:37:08 +0000 (11:37 -0700)
commitd4f8946230392eaeb837f19ff5ada352ee6d192f
tree9c75ba0b2f8701dc0c4e3b27d414be787f20e7bftree | snapshot
parent583fce191c99609b61b88dd3ccb71354a8a053bfcommit | diff
datapath: Avoid double-free on skb_clone failure in ODPAT_OUTPUT_GROUP.

output_group() has no business freeing the skb passed into it, but it was
doing so in case of allocation failure.  Since execute_actions() would
also later free it, this was a serious error.

Thanks to Justin for pointing out the problem.
datapath/actions.c diff | blob | history
Unnamed repository; edit this file to name it for gitweb.