[ovs-dev] [PATCH 1/3] dpif: Fix NULL dereference fetching non-UFID flow.
Ben Pfaff
blp at nicira.com
Thu Dec 11 22:05:05 UTC 2014
On Wed, Dec 03, 2014 at 02:24:46PM -0800, Joe Stringer wrote:
> The UFID parameter to dpif_flow_get() is optional, but the current
> implementation dereferences it to initialize part of the output flow.
> This field is filled in by the dpif implementation, so don't initialize
> it here.
>
> This could cause a NULL dereference if a dpif_flow_get() caller doesn't
> provide a UFID. Currently there are no such callers, but the next patch
> will introduce one.
>
> Signed-off-by: Joe Stringer <joestringer at nicira.com>
Acked-by: Ben Pfaff <blp at nicira.com>
The title of the patch makes this sound like a bug fix, but the
explanation later on sounds like it would only be a real bug if the
callers were changed. Is there a bug that can currently be triggered?
Thanks,
Ben.
More information about the dev
mailing list