[ovs-dev] [dp-cleanups 2/4] datapath: Avoid freeing wild pointer in corner case.

Jesse Gross jesse at nicira.com
Fri Apr 29 17:44:50 UTC 2011


On Fri, Apr 29, 2011 at 10:07 AM, Ben Pfaff <blp at nicira.com> wrote:
> 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 at nicira.com>

Acked-by: Jesse Gross <jesse at nicira.com>



More information about the dev mailing list