git://git.onelab.eu / sliver-openvswitch.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 077257b) | patch
ofproto-dpif: Avoid bad pointer dereference in execute_odp_actions().
authorBen Pfaff <blp@nicira.com>
Thu, 13 Oct 2011 17:16:59 +0000 (10:16 -0700)
committerBen Pfaff <blp@nicira.com>
Thu, 13 Oct 2011 19:56:38 +0000 (12:56 -0700)
commitb6bff4e9abaf2d9a53136854a733b7c4c5d4384e
tree6b9338d3e4e5d4452741987b9333eef128970a5ctree | snapshot
parent077257b83c68a36ea86f2d21c8395f60df710c21commit | diff
ofproto-dpif: Avoid bad pointer dereference in execute_odp_actions().

execute_odp_actions() can be passed a zero-length set of actions, in which
case it may not dereference its 'odp_actions' parameter at all, but in fact
it did do so.  In at least one corner case, odp_actions can be NULL, so
that this caused a segfault.

Introduced in commit 98403001ec "datapath: Move Netlink PID for userspace
actions from flows to actions."

Reported-by: Pravin Shelar <pshelar@nicira.com>
ofproto/ofproto-dpif.c diff | blob | history
sliver-openvswitch