Menu

#246 valgrind detects error in ATL_SetAllBitsBV

Both
closed-fixed
None
5
2015-12-07
2015-03-01
No

:::C

To track down my intermittent error, I ran valgrind. It reports an error 3.11.31 in SetAllBitsBV when nmax was 8.

void ATL_SetAllBitsBV(int bv)
{
const int nbv = bv[1] ? bv[0]+1 : bv[0];
int
bp = bv+2;
int i;
for (i=0; i < nbv; i++)
bp[nbv] = -1;
}

Shouldn't that be:

void ATL_SetAllBitsBV(int bv)
{
const int nbv = bv[1] ? bv[0]+1 : bv[0];
int
bp = bv+2;
int i;
for (i=0; i < nbv; i++)
bp[ i ] = -1; // not nbv
}

Discussion

  • R. Clint Whaley

    R. Clint Whaley - 2015-03-02

    Ticket moved from /p/math-atlas/support-requests/986/

     
  • R. Clint Whaley

    R. Clint Whaley - 2015-03-02
    • status: open --> open-accepted
    • assigned_to: R. Clint Whaley
    • Group: Developer_(v3.11.x) --> Both
     
  • R. Clint Whaley

    R. Clint Whaley - 2015-03-02

    I confirm this ATLAS bug, and your proposed fix. I have made the change to the basefile for the next release.

    Thanks!
    Clint

     
  • R. Clint Whaley

    R. Clint Whaley - 2015-12-07
    • status: open-accepted --> closed-fixed
     

Log in to post a comment.