git://git.onelab.eu / sliver-openvswitch.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: fcbd99e) | patch
datapath: Avoid freeing wild pointer in corner case.
authorBen Pfaff <blp@nicira.com>
Thu, 28 Apr 2011 23:34:56 +0000 (16:34 -0700)
committerBen Pfaff <blp@nicira.com>
Fri, 29 Apr 2011 21:32:01 +0000 (14:32 -0700)
commit3594990b070f8cd6df9d7693cacb8ff4d91429fe
tree6cabab38c4c9e876c2e4e03e9f834985a4784e83tree | snapshot
parentfcbd99e69cb74ba1775ab5dc2f62988ba5194e5ccommit | diff
datapath: Avoid freeing wild pointer in corner case.

In odp_flow_cmd_new_or_set(), if flow_actions_alloc() fails in the "new
flow" case, then flow_put() will kfree() the new flow's 'sf_acts' pointer,
but nothing has initialized that pointer.  Initialize the pointer to NULL
to avoid the problem.

Found by inspection.

Signed-off-by: Ben Pfaff <blp@nicira.com>
Acked-by: Jesse Gross <jesse@nicira.com>
datapath/flow.c diff | blob | history
sliver-openvswitch