Re: [dpdk-dev] [PATCH v1] lib/power: fix buffer overrun coverity issues

2019-04-22 Thread Thomas Monjalon
09/04/2019 13:18, Burakov, Anatoly: > On 09-Apr-19 10:22 AM, David Hunt wrote: > > A previous change removed the limit of 64 cores by > > moving away from 64-bit masks to char arrays. However > > this left a buffer overrun issue, where the max channels > > was defined as 64, and max cores was defin

Re: [dpdk-dev] [PATCH v1] lib/power: fix buffer overrun coverity issues

2019-04-10 Thread Ferruh Yigit
On 4/10/2019 11:55 AM, Burakov, Anatoly wrote: > On 10-Apr-19 8:52 AM, Ferruh Yigit wrote: >> On 4/9/2019 3:40 PM, Hunt, David wrote: >>> >>> On 9/4/2019 12:18 PM, Burakov, Anatoly wrote: On 09-Apr-19 10:22 AM, David Hunt wrote: > A previous change removed the limit of 64 cores by > mo

Re: [dpdk-dev] [PATCH v1] lib/power: fix buffer overrun coverity issues

2019-04-10 Thread Burakov, Anatoly
On 10-Apr-19 8:52 AM, Ferruh Yigit wrote: On 4/9/2019 3:40 PM, Hunt, David wrote: On 9/4/2019 12:18 PM, Burakov, Anatoly wrote: On 09-Apr-19 10:22 AM, David Hunt wrote: A previous change removed the limit of 64 cores by moving away from 64-bit masks to char arrays. However this left a buffer

Re: [dpdk-dev] [PATCH v1] lib/power: fix buffer overrun coverity issues

2019-04-10 Thread Ferruh Yigit
On 4/9/2019 3:40 PM, Hunt, David wrote: > > On 9/4/2019 12:18 PM, Burakov, Anatoly wrote: >> On 09-Apr-19 10:22 AM, David Hunt wrote: >>> A previous change removed the limit of 64 cores by >>> moving away from 64-bit masks to char arrays. However >>> this left a buffer overrun issue, where the max

Re: [dpdk-dev] [PATCH v1] lib/power: fix buffer overrun coverity issues

2019-04-09 Thread Hunt, David
On 9/4/2019 12:18 PM, Burakov, Anatoly wrote: On 09-Apr-19 10:22 AM, David Hunt wrote: A previous change removed the limit of 64 cores by moving away from 64-bit masks to char arrays. However this left a buffer overrun issue, where the max channels was defined as 64, and max cores was defined

Re: [dpdk-dev] [PATCH v1] lib/power: fix buffer overrun coverity issues

2019-04-09 Thread Burakov, Anatoly
On 09-Apr-19 10:22 AM, David Hunt wrote: A previous change removed the limit of 64 cores by moving away from 64-bit masks to char arrays. However this left a buffer overrun issue, where the max channels was defined as 64, and max cores was defined as 256. These should all be consistently set to R

[dpdk-dev] [PATCH v1] lib/power: fix buffer overrun coverity issues

2019-04-09 Thread David Hunt
A previous change removed the limit of 64 cores by moving away from 64-bit masks to char arrays. However this left a buffer overrun issue, where the max channels was defined as 64, and max cores was defined as 256. These should all be consistently set to RTE_MAX_LCORE. The #defines being removed a