Looks good to me.<br><br>-- ramki<br><br><div class="gmail_quote">On Thu, Oct 25, 2012 at 8:18 AM, Jon Masamitsu <span dir="ltr"><<a href="mailto:jon.masamitsu@oracle.com" target="_blank">jon.masamitsu@oracle.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">8001584: NPG: Incorrect assertion in BinaryTreeDictionary::get_<u></u>chunk()<br>
<br>
The assertion<br>
<br>
     assert(dither != FreeBlockDictionary<Chunk_t>::<u></u>exactly ||<br>
           res->size() == size, "Not correct size");<br>
<br>
does not take into account that res may be NULL (as the assertion<br>
above it does).<br>
<br>
<a href="http://cr.openjdk.java.net/%7Ejmasa/8001584/webrev.00/" target="_blank">http://cr.openjdk.java.net/~<u></u>jmasa/8001584/webrev.00/</a><br>
<br>
or the diff<br>
<br>
--- a/src/share/vm/memory/<u></u>binaryTreeDictionary.hpp<br>
<br>
+++ b/src/share/vm/memory/<u></u>binaryTreeDictionary.hpp<br>
<br>
@@ -259,7 +259,7 @@<br>
<br>
     assert(res == NULL || res->is_free(),<br>
<br>
            "Should be returning a free chunk");<br>
<br>
     assert(dither != FreeBlockDictionary<Chunk_t>::<u></u>exactly ||<br>
<br>
-           res->size() == size, "Not correct size");<br>
<br>
+           res == NULL || res->size() == size, "Not correct size");<br>
<br>
     return res;<br>
<br>
   }<br>
<br>
Thanks.<span class="HOEnZb"><font color="#888888"><br>
<br>
Jon<br>
</font></span></blockquote></div><br>