Re: RFR-8012108

2013-04-17 Thread Kurchi Hazra 

Hi John,

  Does this actually fix the leak? When curr is null, you set start to 
curr, and hence start to null. So nothing is every freed

in freeAllocatedMemory is what I see. Let me know if I am missing something.

- Kurchi

On 4/17/2013 8:02 AM, John Zavgren wrote:

Greetings:
Please review the following webrev image. I fixed a memory leak that 
can occur when calloc() fails.


http://cr.openjdk.java.net/~jzavgren/8012108/webrev.01/

Thanks!
John Zavgren
john.zavg...@oracle.com


--
-Kurchi

RFR-8012108

2013-04-17 Thread John Zavgren 

Greetings: 
Please review the following webrev image. I fixed a memory leak that can occur 
when calloc() fails. 

http://cr.openjdk.java.net/~jzavgren/8012108/webrev.01/ 


Thanks! 
John Zavgren 
john.zavg...@oracle.com