MantisBT - VTK
View Issue Details
0005341VTK(No Category)public2007-07-13 13:582007-09-18 16:56
Andreas Gerndt 
Brad King 
normalminoralways
closedfixed 
 
 
0005341: BUG in 64bit vtkDataArrayTemplate
I am working on a very huge terrain dataset consisting of multiple tiles. I
hit the 32bit limit of point numbers so that I recompiled VTK with the
VTK_USE_64BITS_IDS flag set. Nevertheless, appending data still crashed. The
reason was the usage of an "int" in vtkDataArrayTemplate.txx. Changing it to
vtkIdType fixed this problem.

//----------------------------------------------------------------------------
// Allocate memory for this array. Delete old storage only if necessary.
template <class T>
int vtkDataArrayTemplate<T>::Allocate(vtkIdType sz, vtkIdType)
{
...
    //=========================================
    int newSize = (sz > 0 ? sz : 1);
    //=========================================
    this->Array = (T*)malloc(newSize * sizeof(T));
    if(!this->Array)
      {
      vtkErrorMacro("Unable to allocate " << newSize
                    << " elements of size " << sizeof(T)
                    << " bytes. ");
      return 0;
      }
    this->Size = newSize;
    }

  return 1;
}
//----------------------------------------------------------------------------
No tags attached.
Issue History
2007-09-18 16:55David ColeStatusclosed => @80@
2007-09-18 16:55David ColeNote Added: 0009107
2007-09-18 16:56David ColeStatus@80@ => closed
2007-09-18 16:56David ColeNote Added: 0009108
2011-06-16 13:11Zack GalbreathCategory => (No Category)

Notes
(0008324)
Brad King   
2007-08-01 11:32   
Fixed in CVS:

/cvsroot/VTK/VTK/Common/vtkDataArrayTemplate.txx,v <-- vtkDataArrayTemplate.txx
new revision: 1.26; previous revision: 1.25

I'll report this for inclusion in the 5.0 branch.
(0009107)
David Cole   
2007-09-18 16:55   
Merged this fix into the VTK-5-0 branch today:

  /cvsroot/VTK/VTK/Common/vtkDataArrayTemplate.txx,v <-- vtkDataArrayTemplate.txx
  new revision: 1.16.4.3; previous revision: 1.16.4.2
(0009108)
David Cole   
2007-09-18 16:56   
Re-closing after adding note about merging into VTK-5-0