[PATCH 3/3] covoar: Fix overflow of high PC address

Alex White alex.white at oarcorp.com
Tue Mar 30 15:17:26 UTC 2021


On Tue, Mar 30, 2021 at 10:09 AM Gedare Bloom <gedare at rtems.org> wrote:
>
> looks ok. what happens if the high_pc is 1? Just wondering.

Good question. :)

I believe a zero-sized AddressRange is constructed and added to that symbol's CoverageMap.

It is possibly wasteful, but should not cause a crash or anything.

Alex

>
> On Tue, Mar 30, 2021 at 8:45 AM Alex White <alex.white at oarcorp.com> wrote:
> >
> > ping
> >
> > On Fri, Mar 12, 2021 at 11:14 AM Alex White <alex.white at oarcorp.com> wrote:
> > >
> > > This fixes an integer overflow that would occur if a function's high PC
> > > address were zero in the DWARF info.
> > > ---
> > >  tester/covoar/ExecutableInfo.cc | 5 +++++
> > >  1 file changed, 5 insertions(+)
> > >
> > > diff --git a/tester/covoar/ExecutableInfo.cc b/tester/covoar/ExecutableInfo.cc
> > > index 30828a6..7f33f20 100644
> > > --- a/tester/covoar/ExecutableInfo.cc
> > > +++ b/tester/covoar/ExecutableInfo.cc
> > > @@ -71,6 +71,11 @@ namespace Coverage {
> > >              }
> > >            }
> > >
> > > +          // We can't process a zero size function.
> > > +          if (func.pc_high() == 0) {
> > > +            continue;
> > > +          }
> > > +
> > >            createCoverageMap (cu.name(), func.name(),
> > >                                func.pc_low(), func.pc_high() - 1);
> > >          }
> > > --
> > > 2.27.0
> > >
> > > _______________________________________________
> > > devel mailing list
> > > devel at rtems.org
> > > http://lists.rtems.org/mailman/listinfo/devel
> > _______________________________________________
> > devel mailing list
> > devel at rtems.org
> > http://lists.rtems.org/mailman/listinfo/devel
> _______________________________________________
> devel mailing list
> devel at rtems.org
> http://lists.rtems.org/mailman/listinfo/devel


More information about the devel mailing list