Hello,<br clear="all"><br><div>I have been using the geos library for buffer functions and found a crash case. When I stepped into the code, I found that <span class="Apple-style-span" style="font-family: 'courier new', monospace;">RightmostEdgeFinder::findEdge()</span> crashes when no forward <span class="Apple-style-span" style="font-family: 'courier new', monospace;">DirectedEdge</span> is input into the function. The function crashes because of a null pointer reference error (<span class="Apple-style-span" style="font-family: 'courier new', monospace;">minDe=0x0</span>) later in the code. The assertion failure that catches this is on line 77:</div>
<div><br></div><blockquote class="webkit-indent-blockquote" style="margin: 0 0 0 40px; border: none; padding: 0px;"><span class="Apple-style-span" style="font-family: 'courier new', monospace;">assert(checked>0);</span><br>
</blockquote><div><br></div><div>Is this case supposed to be possible? I also found that <span class="Apple-style-span" style="font-family: 'courier new', monospace;">BufferSubgraph::create() </span>calls <span class="Apple-style-span" style="font-family: 'courier new', monospace;">findEdge()</span>, and a comment in the code says that "We are assuming that <span class="Apple-style-span" style="font-family: 'courier new', monospace;">dirEdgeList</span> contains *at least* ONE forward <span class="Apple-style-span" style="font-family: 'courier new', monospace;">DirectedEdge</span>". Again, what happens if the <span class="Apple-style-span" style="font-family: 'courier new', monospace;">DirectedEdge</span> list does not? I don't think it should crash...</div>
<div><br></div><div>Thanks,</div><div><br><div>Stephen</div></div>