Hi Paul,

thanks for providing the fix. Do you mind pushing that patch to gerrit and
adding Utkarsh and I as a reviewer?

Thanks,

Seb

PS: More information can be found here on how to push a patch to gerrit.
http://www.vtk.org/Wiki/VTK/Git/Develop


On Mon, Jan 6, 2014 at 12:35 PM, Paul Edwards <paul.m.edwa...@gmail.com>wrote:

> Below is the fix:
>
> diff --git a/Filters/Core/vtkAppendFilter.cxx
> b/Filters/Core/vtkAppendFilter.cxx
> index 674135c..1f8878f 100644
> --- a/Filters/Core/vtkAppendFilter.cxx
> +++ b/Filters/Core/vtkAppendFilter.cxx
> @@ -302,6 +302,7 @@ int vtkAppendFilter::RequestData(
>          if (ug && ds->GetCellType(cellId) == VTK_POLYHEDRON )
>            {
>            vtkIdType nfaces, *facePtIds;
> +          newPtIds->Reset ();
>            ug->GetFaceStream(cellId,nfaces,facePtIds);
>            for(vtkIdType id=0; id < nfaces; ++id)
>              {
>
>
>
> On 6 January 2014 16:04, Paul Edwards <paul.m.edwa...@gmail.com> wrote:
>
>> I'm having problems when using data with polyhedron cells.  I get a
>> segfault when I run append data on the attached datasets.  I've got a very
>> simple example - two datasets, one with 3 cells and other with 2 cells.
>>  The steps to reproduce:
>>
>> Load polyhedron1.vtu
>> Load polyhedron2.vtu
>> Select both, choose "Append datasets" and apply
>>
>> There seems to be some memory corruption somewhere but it's not obvious
>> to me...  Any hints would be appeciated if anyone has the time to look.
>>
>> Thanks,
>> Paul
>>
>>
>>
>
> _______________________________________________
> Powered by www.kitware.com
>
> Visit other Kitware open-source projects at
> http://www.kitware.com/opensource/opensource.html
>
> Please keep messages on-topic and check the ParaView Wiki at:
> http://paraview.org/Wiki/ParaView
>
> Follow this link to subscribe/unsubscribe:
> http://www.paraview.org/mailman/listinfo/paraview
>
>
_______________________________________________
Powered by www.kitware.com

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Please keep messages on-topic and check the ParaView Wiki at: 
http://paraview.org/Wiki/ParaView

Follow this link to subscribe/unsubscribe:
http://www.paraview.org/mailman/listinfo/paraview

Reply via email to