git://git.onelab.eu / sliver-openvswitch.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 51a66e1) | patch
datapath: Check IS_ERR() in do_execute().
authorJesse Gross <jesse@nicira.com>
Wed, 15 Sep 2010 23:52:48 +0000 (16:52 -0700)
committerJustin Pettit <jpettit@nicira.com>
Tue, 5 Oct 2010 23:41:39 +0000 (16:41 -0700)
commita842e7b093786019292998fa192d4ed7d228d686
treef9b11c0ea2595564b8b8887d9487496e9b2e0189tree | snapshot
parent51a66e1bc935c50afe955da947cf253adf115751commit | 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