> On Thu, Dec 21, 2017 at 04:30:05PM -0800, Justin Pettit wrote: >> >> >> > On Dec 4, 2017, at 11:51 AM, Ben Pfaff <blp at ovn.org> wrote: >> > >> > The 'mask' buffer in parse_odp_action() is supposed to always be big >> > enough: >> > /* 'mask' is big enough to hold any key. */ >> > >> > Geneve options can be really big and the comment was wrong. In addition, >> > the user might supply more options than can really fit in any case, so >> > we might as well just use a stub. >> > >> > Found by libfuzzer. >> > >> > Reported-by: Bhargava Shastry <bshastry at sec.t-labs.tu-berlin.de> >> > Signed-off-by: Ben Pfaff <blp at ovn.org> >> >> Acked-by: Justin Pettit <jpettit at ovn.org> > > Thanks, applied to master and backported as far as 2.4.
Hi Ben and Simon. This patch breaks the build on branch-2.4, because there is no such thing as OFPBUF_STUB_INITIALIZER on this branch: lib/odp-util.c:1063:33: error: undefined identifier 'OFPBUF_STUB_INITIALIZER' lib/odp-util.c:1063:56: error: invalid initializer We need to revert it or fix in other way. BTW, seems nobody uses this branch. It was broken for more than a year. Maybe we could stop backporting stuff to it. Best regards, Ilya Maximets. _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev