Request for review (S) 7017193: Small memory leak in get_stack_bounds // os::create_stack_guard_pages

David Holmes David.Holmes at oracle.com
Mon Feb 21 15:47:19 PST 2011


Dmitry,

Dmitry Samersoff said the following on 02/21/11 22:20:
> Hi Everyone,
> 
> http://cr.openjdk.java.net/~dsamersoff/7017193
> 
> I'm uncomfortable with one malloc/free cycle per line so decided to get 
> rid of getline() call completely.

As I stated in the CR you do not have to do one malloc/free per line 
with getline. The buffer can be reused and will be realloc'd if needed 
by getline, and then can be freed at the end.

I can't evaluate your change without understanding both the detailed 
logic of the original code and your new code.

David


More information about the hotspot-runtime-dev mailing list