RE: [PATCH] GcovData.cc: Fix out-of-bounds access errors

2021-08-12 Thread Ryan Long
PATCH] GcovData.cc: Fix out-of-bounds access errors On Thu, Aug 12, 2021 at 7:54 AM Ryan Long wrote: > > Would you need to check if length < sizeof(gcov_preamble) since length is > assigned that value? > No, but my question is about 'preamble'. If there's a

Re: [PATCH] GcovData.cc: Fix out-of-bounds access errors

2021-08-12 Thread Gedare Bloom
eamble', then that should be checked. If they should be the same, that should be asserted, probably. > -Original Message- > From: Gedare Bloom > Sent: Wednesday, August 11, 2021 11:13 AM > To: Ryan Long > Cc: devel@rtems.org > Subject: Re: [PATCH] GcovData.cc: Fix

RE: [PATCH] GcovData.cc: Fix out-of-bounds access errors

2021-08-12 Thread Ryan Long
Would you need to check if length < sizeof(gcov_preamble) since length is assigned that value? -Original Message- From: Gedare Bloom Sent: Wednesday, August 11, 2021 11:13 AM To: Ryan Long Cc: devel@rtems.org Subject: Re: [PATCH] GcovData.cc: Fix out-of-bounds access errors On

Re: [PATCH] GcovData.cc: Fix out-of-bounds access errors

2021-08-11 Thread Gedare Bloom
On Wed, Aug 11, 2021 at 8:06 AM Ryan Long wrote: > > Adjusted number of bytes to be read > > CID 1506208: Out-of-bounds access > CID 1506209: Out-of-bounds access > > Closes #4485 > --- > tester/covoar/GcovData.cc | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/test

[PATCH] GcovData.cc: Fix out-of-bounds access errors

2021-08-11 Thread Ryan Long
Adjusted number of bytes to be read CID 1506208: Out-of-bounds access CID 1506209: Out-of-bounds access Closes #4485 --- tester/covoar/GcovData.cc | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tester/covoar/GcovData.cc b/tester/covoar/GcovData.cc index 02e7489..da0cc2