Backporting 8020983: OutOfMemoryError caused by non garbage collected JPEGImageWriter Instances

[Andrew Hughes]

----- Original Message -----
> Hi Andrew,
> 
> On 08/27/2013 11:22 AM, Andrew Hughes wrote:
> > 0.  What problem does this solve?
> 
> The current code has memory leaks. With some usage patterns, Java will
> run heap space. For more details, please see the bug:
> http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8020983
> 

"This image writer object will never be garbage collected. This is reproducible with java 6u45. It works fine for us until version 6_u43."

It sounds like a recent regression.  Have you been able to reproduce the issue
on OpenJDK 6?

> > 1.  Is this in jdk7u?
> 
> Yes: http://hg.openjdk.java.net/jdk7u/jdk7u40/jdk/rev/46b203b37e06
> 

Ok, but not in any release as yet. I'm just trying to ascertain how well-tested
this fix is to be going in a stable release.

> > 2.  Is this in IcedTea6?
> 
> As far as I can see, no.
> 

Ok, it should probably also go in 1.11.x and 1.12.x if the issue is present there.

> > 3.  Where has the bug ID information gone from the backported changeset?
> 
> webrevs can be generated against patches as well as changesets. This one
> is not generated against a changeset (yet) since a simple hg
> export/import did not work.

I'm well aware of that, but I can't approve it unless it's in its final form.
I want to make sure the original information isn't being lost.

> 
> Thanks,
> Omair
> 
> --
> PGP Key: 66484681 (http://pgp.mit.edu/)
> Fingerprint = F072 555B 0A17 3957 4E95  0056 F286 F14F 6648 4681
> 

-- 
Andrew :)

Free Java Software Engineer
Red Hat, Inc. (http://www.redhat.com)

PGP Key: 248BDC07 (https://keys.indymedia.org/)
Fingerprint = EC5A 1F5E C0AD 1D15 8F1F  8F91 3B96 A578 248B DC07