Skip to Content.
Sympa Menu

cgal-discuss - Re: [cgal-discuss] Integer overflow for large images:

Subject: CGAL users discussion list

List archive

Re: [cgal-discuss] Integer overflow for large images:


Chronological Thread 
  • From: "Sebastien Loriot (GeometryFactory)" <>
  • To:
  • Subject: Re: [cgal-discuss] Integer overflow for large images:
  • Date: Fri, 13 Feb 2015 07:23:26 +0100
  • Organization: GeometryFactory

On 02/03/2015 01:49 PM, Dominik Szczerba wrote:
Hi,

Has the official version fixing this bug already been released?


It will be integration in the 4.6 release of CGAL which beta should
be release this month.

Sebastien.

Thanks
Dominik

2014-12-08 17:47 GMT+01:00 Jane Tournois
<>:
Dear Bryn,

we have fixed the bug you pointed out. The fix will be available in the next
public release of CGAL.

Thanks again.
Best regards,

Jane.

Le 20/10/2014 08:57, Bryn Lloyd a écrit :


Dear Jane

Thanks, Bryn



On 10/17/2014 8:55 AM, Jane Tournois wrote:

Dear Bryn,

thanks for the patch you suggested.

I will take care of this and keep you informed of its integration to
CGAL.

Best regards,
Jane.

Le 14/10/2014 09:38, Bryn Lloyd a écrit :

Dear CGAL developers

I would like to submit a patch for a problem which occurs when
generating a 3D mesh from very large images. The detailed information is
below:

OS, compiler:
- Windows 7 & 8, Visual Studio 2012

Build options:
- Configured using cmake for VS 2012, without GMP, without ZLIB

Description of the problem:
- Generating meshes from very large image (approx. 4e9 voxels) was not
working as expected. Sometimes one half of the image was meshed correctly,
but strange things were happening to the other half.

Kernel (3D Mesh Generation from images/label fields):
- I was using an old version of CGAL (3.8), however, looking at the
newest version I can see that issue has not been fixed.

Solution:
- After debugging a bit, I found that the function
'Image_3::labellized_trilinear_interpolation' was using 'unsigned int'
instead of e.g. 'std::uint64_t' or size_t. This causes an integer overflow,
which breaks mesh generation.
Changing the code in Image_3.h and Image_to_labeled_function_wrapper.h
to use size_t fixed the issue.

I attached a patch.

Best regards
Bryn











--
Jane Tournois, PhD
R&D Engineer at GeometryFactory
http://www.geometryfactory.com/


--
You are currently subscribed to cgal-discuss.
To unsubscribe or access the archives, go to
https://sympa.inria.fr/sympa/info/cgal-discuss







Archive powered by MHonArc 2.6.18.

Top of Page