<div dir="ltr">Sean-<div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span style="font-size:16px">In which case one may reasonably argue that the PLYReader is indeed broken, for failing to reject invalid files.</span></blockquote><div><br></div><div>You are opening a can of worms here. I would humbly suggest that in most all file formats in VTK if the data is invalid then bad things will happen. If you want to make a sanity check as part of the read process then do it as an optional step because fully vetting data can take forever.</div><div><br></div><div>Best,</div><div>W</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Aug 10, 2017 at 1:29 PM, Sean McBride <span dir="ltr"><<a href="mailto:sean@rogue-research.com" target="_blank">sean@rogue-research.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Thu, 10 Aug 2017 13:06:18 -0400, Bill Lorensen said:<br>
<br>
>The PLYReader is fine. The file is not a proper PLY file. The face<br>
>indices should be zero-based, but in this file they are one-based. So,<br>
>the face<br>
>3 1 2 5<br>
>is referencing an point 5 which exceeds the points array which has<br>
>tuples from 0 - 4 (5 points).<br>
<br>
</span>In which case one may reasonably argue that the PLYReader is indeed broken, for failing to reject invalid files.  Under ASan/valgrind, would the PLYReader crash with this file?<br>
<span class="HOEnZb"><font color="#888888"><br>
Sean<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
______________________________<wbr>_________________<br>
Powered by <a href="http://www.kitware.com" rel="noreferrer" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" rel="noreferrer" target="_blank">http://www.kitware.com/<wbr>opensource/opensource.html</a><br>
<br>
Search the list archives at: <a href="http://markmail.org/search/?q=vtk-developers" rel="noreferrer" target="_blank">http://markmail.org/search/?q=<wbr>vtk-developers</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://public.kitware.com/mailman/listinfo/vtk-developers" rel="noreferrer" target="_blank">http://public.kitware.com/<wbr>mailman/listinfo/vtk-<wbr>developers</a><br>
<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div>William J. Schroeder, PhD<br>Kitware, Inc. - Building the World's Technical Computing Software<br>28 Corporate Drive<br>Clifton Park, NY 12065<br><a href="mailto:will.schroeder@kitware.com" target="_blank">will.schroeder@kitware.com</a><br><a href="http://www.kitware.com" target="_blank">http://www.kitware.com</a><br>(518) 881-4902</div></div></div>
</div>