Re: [ovs-dev] [PATCH] odp-util: Fix buffer overread in parsing string form of ODP flows.

2017-11-27 Thread Ben Pfaff
On Sun, Nov 26, 2017 at 09:23:08PM -0800, Joe Stringer wrote: > On 26 November 2017 at 17:41, Ben Pfaff wrote: > > scan_u128() should return 0 on an error but it actually returned an errno > > value in some cases, so a command like this: > > ovs-appctl dpctl/add-flow 'ct_label(1/55

Re: [ovs-dev] [PATCH] odp-util: Fix buffer overread in parsing string form of ODP flows.

2017-11-26 Thread Joe Stringer
On 26 November 2017 at 17:41, Ben Pfaff wrote: > scan_u128() should return 0 on an error but it actually returned an errno > value in some cases, so a command like this: > ovs-appctl dpctl/add-flow 'ct_label(1/55)' '' > could cause a buffer overread. > > This bug is not

[ovs-dev] [PATCH] odp-util: Fix buffer overread in parsing string form of ODP flows.

2017-11-26 Thread Ben Pfaff
scan_u128() should return 0 on an error but it actually returned an errno value in some cases, so a command like this: ovs-appctl dpctl/add-flow 'ct_label(1/55)' '' could cause a buffer overread. This bug is not as severe as it may sound because the string form of ODP f