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

[Neil Richards]

Now pushed to jdk7u-dev/jdk [1].

Regards,
Neil

[1] http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/f7d9ab7563b5

On Tue, 2012-10-16 at 14:40 -0700, Seán Coffey wrote:
> 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
> >>> *
> >>> *
> >>>
> >
> 


-- 
Unless stated above:
IBM email: neil_richards at uk.ibm.com
IBM United Kingdom Limited - Registered in England and Wales with number 741598.
Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 3AU