Re: [PATCH FOR REVIEW] OPENJDK6-3: Fix small memory leak in get_stack_bounds

2013-03-21 Thread Omair Majid
On 03/21/2013 09:49 AM, Andrew Hughes wrote: > > remote: > OPENJDK6-3: Fix memory leak in get_stack_bounds > remote: > Summary: Memory is allocated by getline but not freed if it fails > remote: > Reviewed-by: aph > remote: > remote: Incomplete comment: Missing bugid line > remote: Incomplete com

Re: [PATCH FOR REVIEW] OPENJDK6-3: Fix small memory leak in get_stack_bounds

2013-03-21 Thread Andrew Hughes
- Original Message - > On 03/20/2013 10:45 PM, Andrew Hughes wrote: > > A small memory leak was found in get_stack_bounds when getline > > returned with an error code, > > but it was fixed in OpenJDK 7 & 8 in a way that introduced worse > > issues: > > > > http://mail.openjdk.java.net/pi

Re: [PATCH FOR REVIEW] OPENJDK6-3: Fix small memory leak in get_stack_bounds

2013-03-21 Thread Andrew Haley
On 03/20/2013 10:45 PM, Andrew Hughes wrote: > A small memory leak was found in get_stack_bounds when getline returned with > an error code, > but it was fixed in OpenJDK 7 & 8 in a way that introduced worse issues: > > http://mail.openjdk.java.net/pipermail/hotspot-dev/2013-February/008695.html

[PATCH FOR REVIEW] OPENJDK6-3: Fix small memory leak in get_stack_bounds

2013-03-20 Thread Andrew Hughes
A small memory leak was found in get_stack_bounds when getline returned with an error code, but it was fixed in OpenJDK 7 & 8 in a way that introduced worse issues: http://mail.openjdk.java.net/pipermail/hotspot-dev/2013-February/008695.html This webrev adds a simpler fix, just adding the necess