[vtkusers] [Paraview] 64 bit bug in vtkImageData

[David Gobbi]

Hi Berk,

I started working on a fix before I saw your email, there are several
other places in vtkImageData.cxx that have a similar problem.

Nearly everywhere in the code where "dims" was used (excluding
PrintSelf), there was a potential for overflow. This is partly my
fault, because when I did a batch of 64-bit fixes to vtkImageData a
couple years ago, I searched through the code for "increments" and
"extents" but forgot about "dims".  I'm innocent of the extent problem
that Mike just found, but I've made a fix for that one, too, and will
do the commit today.

   David


On Fri, Mar 21, 2008 at 2:06 PM, Berk Geveci <berk.geveci at kitware.com> wrote:
> I will commit a fix next week. After this fix, is everything else working well?
>
>  -berk
>
>
>  On 3/21/08, Mike Jackson <imikejackson at gmail.com> wrote:
>  > While working with a rather large data set (5GB) I cam across a bug in
>  >  the vtkImageData.cxx file.
>  >
>  >  In the void vtkImageData::AllocateScalars(); method the following
>  >  should be used:
>  >
>  >   vtkIdType extent[6] ={ this->Extent[0], this->Extent[1], this->Extent[2],
>  >                          this->Extent[3], this->Extent[4], this->Extent[5]};
>  >
>  >  instead of
>  >   const int* extent = this->Extent;
>  >
>  >  The second version will lead to over flow errors if the extents are
>  >  large enough. (1800x1800x1800 in our case).
>  >
>  >  Should I file a bug or can someone with cvs access get this reviewed
>  >  and committed to cvs.
>  >  --
>  >  Mike Jackson
>  >  imikejackson _at_ gee-mail dot com
>  >
>  >  PS: Sorry for the cross post, but I wasn't sure which list to post to.
>  >  I was using the vtk from the ParaView CVS Head when i found the bug.
>  >  _______________________________________________
>  >  ParaView mailing list
>  >  ParaView at paraview.org
>  >  http://www.paraview.org/mailman/listinfo/paraview
>
>
> >
>  _______________________________________________
>  This is the private VTK discussion list.
>  Please keep messages on-topic. Check the FAQ at: http://www.vtk.org/Wiki/VTK_FAQ
>  Follow this link to subscribe/unsubscribe:
>  http://www.vtk.org/mailman/listinfo/vtkusers
>