On Mon, Jun 14, 2021 at 05:17:01PM +0200, Ilya Maximets wrote:
> We don't need to continue parsing if already oversized.  This is not
> very important, but fuzzer times out while parsing very long list of
> actions.
> 
> Reported-at: https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=29190
> Signed-off-by: Ilya Maximets <[email protected]>

Is {1..8190} portable shell syntax?  It's not in POSIX.  The "dash"
shell doesn't understand it.

I'd tend to write $(seq 8190) instead.

> +data_valid=$(  printf 'set(encap()),%.0s' {1..8190})
> +data_invalid=$(printf 'set(encap()),%.0s' {1..8191})

Thanks,

Ben.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to