[ovs-dev] [PATCH v2] expr: Properly initialize expr_constant
pkusunyifeng at gmail.com
Tue Oct 2 22:15:54 UTC 2018
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.
On Tue, Oct 2, 2018 at 2:57 PM Ben Pfaff <blp at ovn.org> 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 <pkusunyifeng at gmail.com>
> 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].
More information about the dev