[ovs-dev] [PATCH v2] odp-util: Return an error on actions overflow while parsing from string.

Ilya Maximets i.maximets at ovn.org
Mon Jun 14 19:54:35 UTC 2021


On 6/14/21 8:20 PM, Ben Pfaff wrote:
> On Mon, Jun 14, 2021 at 07:36: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 <i.maximets at ovn.org>
> 
> Acked-by: Ben Pfaff <blp at ovn.org>

Applied.  Thanks!

> 
> I didn't realize that the printf program worked that way.  That's a very
> clever way to repeat a string.
> 

Internet search also suggests some interesting variants like:

yes "set(encap())," | head -8190 | tr -d '\n'

And it's even faster.  But I like the printf one better.

Best regards, Ilya Maximets.


More information about the dev mailing list