Yes, I saw that members of expr_constant are accessed in many places without checking its type. I will work out another version.
For the subject, it is weird that I can't see [ovs-dev] added. Thanks for your review. Yifeng On Tue, Oct 2, 2018 at 2:57 PM Ben Pfaff <[email protected]> wrote: > On Tue, Oct 02, 2018 at 11:37:02AM -0700, Yifeng Sun wrote: > > expr_constant.masked may be uninitialized when its type is EXPR_C_STRING. > > This patch fixes this issue. > > > > Reported-at: https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=10731 > > Signed-off-by: Yifeng Sun <[email protected]> > > Thanks for working to fix the bugs reported by the fuzzer. > > I don't understand this fix. expr_constant is a union. If an instance > has type EXPR_C_STRING, then its 'masked' member must not be accessed, > because that member is only relevant when it has type EXPR_C_INTEGER. > > > v1->v2: Fix email subject by adding [ovs-dev] > > I don't understand this either. The mailing list itself adds [ovs-dev]. > > Thanks, > > Ben. > _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
