git://git.onelab.eu / sliver-openvswitch.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: a154533) | patch
datapath: Check IS_ERR() in do_execute().
authorJesse Gross <jesse@nicira.com>
Wed, 15 Sep 2010 23:52:48 +0000 (16:52 -0700)
committerJesse Gross <jesse@nicira.com>
Thu, 16 Sep 2010 00:43:46 +0000 (17:43 -0700)
commit8ba1fd2fb9eb616ec028027e303c1664185c88e7
tree17bbef59d16836c1f9c134f610e5f7cd11d2eaddtree | snapshot
parenta154533795474bd8d13a2a935c4b6719215d6907commit | diff
datapath: Check IS_ERR() in do_execute().

flow_actions_alloc() returns an error code in the form of a pointer
but we checked that the pointer was not NULL, which is always true.
This caused oopses on allocation errors when we would write into
an invalid pointer.

NIC-234

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