On 1/6/22 15:00, Aaron Conole wrote: > Ilya Maximets <[email protected]> writes: > >> The fuzzing target times out if the action list is too big. And we >> don't really need to fully parse all the actions just to say that they >> are too big in the end. So, check early and exit. >> >> This is a pure performance optimization, so not adding a unit test. >> >> All other code paths during the parsing are using E2BIG and not EFBIG >> for similar conditions, so using it here too. > > E2BIG makes more sense: > * Argument list too long > > EFBIG: > * File too large > >> Reported-at: https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=39670 >> Signed-off-by: Ilya Maximets <[email protected]> >> --- > > LGTM. > > Acked-by: Aaron Conole <[email protected]>
Thanks! Applied. Best regards, Ilya Maximets. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
