On Mon, Jul 23, 2012 at 12:46 AM, Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote: > If there is no OVS_SAMPLE_ATTR_ACTIONS set then "acts_list" is NULL and > it leads to a NULL dereference when we call nla_len(acts_list). This > is a static checker fix, not something I have seen in testing. > > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> This can never happen in practice because the action list is validated at the time that userspace installs the flow. There are plenty of things in this category that would appear to be unsafe because we'd much rather do sanity checking on a per-flow basis rather than per-packet. -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
