Convex hull reconstruction segfault on qh num_facets

[kode9]

Hello,

I'm having a segfault on a convex hull reconstruction. A simple snipped :

// size is > 0

pcl::PointCloud<pcl::PointXYZ>::Ptr cloud(new pcl::PointCloud<pcl::PointXYZ>);
cloud->width    = size;
cloud->height   = 1;
cloud->is_dense = false;
cloud->points.resize(size);

//...
// Fill the point cloud
///

pcl::PointCloud<pcl::PointXYZ>::Ptr cloud_hull(new pcl::PointCloud<pcl::PointXYZ>);
pcl::ConvexHull<pcl::PointXYZ> chull;
std::vector<pcl::Vertices> polys;
chull.setInputCloud(cloud);
chull.setDimension(3);
chull.reconstruct(*cloud_hull, polys); // Segfault

The segfault appears in surface/include/pcl/surface/impl/convex_hull.hpp line 342 :

  int num_facets = qh num_facets;

Using pcl1.7.1 from Arch linux aur and qhull 2012.1-2 packaged.

I'm pretty sure I have no problem on some other computers with pcl 1.7.0 and qhul 2009.1-3

I'll try to get more info if needed

[kode9]

So, I removed all arch packages to compile and test a bit more.

Cloned qhull git [0] and downloaded pcl-1.7.1

Branch 'debian' in qhull : no problem
Branch 'master' in qhull : segfault on /surface/include/pcl/surface/impl/convex_hull.hpp line 381 : FORALLfacets

[0] https://gitorious.org/qhull/qhull/source/60d55819729d7b49391dde0271e15a56c70992b9:

[fran6co]

Qhull 2012 is not supported in PCL. The latest version of Qhull that works with PCL is 2011.1

[kode9]

Alright, thanks for the tip. Is it mentioned somewhere ? I was suprised that Arch maintainers just set qhull as a dependency, without version hint.

On PCL website (http://www.pointclouds.org/downloads/source.html), we can even see :

QHull ≥ 2011.1
pcl_surface
OPTIONAL

This is for the least misleading. Maybe a typo, and it should be QHull <= 2011.1 ?

[fran6co]

Yes, for sure it's a typo. The code is prepared for qhull 2009 and 2011 but nothing for 2012, yet.

[kode9]

Ok cheers.