[ovs-dev] [PATCH 1/7] bridge: Fix possible null pointer dereference reported by clang.

William Tu u9012063 at gmail.com
Tue Jul 5 13:54:41 UTC 2016


Hi Ben,

Thanks, I tested your patch below and clang no longer reports no error.

Regards,
William

> A port is guaranteed to have at least one interface, so 'iface' can't be
> null here.  Even if it could, this patch wouldn't help because a few
> lines down there's an unconditional dereference.
>
> I'd happily add an assertion to document this, e.g.:
>         http://openvswitch.org/pipermail/dev/2016-July/074352.html



More information about the dev mailing list