Thanks for the review. How about returning a -EINVAL in this case?
On Thu, Oct 11, 2018 at 1:39 PM Ben Pfaff <[email protected]> wrote: > On Tue, Oct 09, 2018 at 03:39:18PM -0700, Yifeng Sun wrote: > > When parse_odp_key_mask_attr runs into ufid, it returns length of ufid > > without appending data into ofpbufs. This commit adds additional > > checking for this case. > > > > Found this bug when debugging > > https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=10850, > > but not certain it is related. > > > > Signed-off-by: Yifeng Sun <[email protected]> > > Thanks for the fix. > > It might be better to come up with a way to disallow a ufid here, since > it doesn't make sense to put a ufid into a "set" action. > _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
