Request for approval:7151427: Fix the potential memory leak in error handling code in X11SurfaceData.c

[Seán Coffey]

Neil,

I don't see a problem with this being ported to jdk7u. I've checked the 
bug report again and don't see any associated issues with this fix. 
Approved.

regards,
Sean.

On 16/10/2012 10:36, Neil Richards wrote:
> Hi Sean,
> Am I now good to proceed to act on this approval, or is more discussion
> still needed to reach consensus on the merits of this backport ?
>
> Regards,
> Neil
>
> On Tue, 2012-09-18 at 18:39 +0100, Seán Coffey wrote:
>> Approved for jdk7u-dev.  Note that this most likely means the fix will
>> end up in 7u12. If there's a strong justification for 7u10 inclusion,
>> let me know and one of the jdk7u maintainers can help you work through a
>> phase2 request[1] for 7u10.
>>
>> I'll create a bug record to track this fix in jdk7u.
>>
>> Regards,
>> Sean.
>>
>> [1] http://openjdk.java.net/projects/jdk7u/phase2/phase2-process.html
>>
>> On 14/09/2012 09:55, Sean Chou wrote:
>>> Hi all,
>>>
>>> Here's a request of porting fix 7151427 back to JDK7, could you please help
>>> to review?
>>>
>>> Bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7151427
>>>
>>> Change set: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/b1af41b86f9f
>>>
>>> Thread where it was reviewed:
>>> http://mail.openjdk.java.net/pipermail/2d-dev/2012-March/002403.html
>>> *
>>> *
>>>
>