hypeartist / fog

Automatically exported from code.google.com/p/fog
0 stars 0 forks source link

Image_writeToStream leaks reference to stream #46

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago
1. Write image to a file stream
2. Note that the file is locked by the application until the process exits

The bug appears to be in Image.cpp Image_writeToStream. The stream is attached 
to the ImageEncoder, but never detached before the encoder is destroyed, 
leaking a reference and causing the stream to remain open.

The stream should be detached either via the destruction of the encoder, or 
explicitly before returning from Image_writeToStream.

Original issue reported on code.google.com by sonof...@gmail.com on 31 May 2012 at 9:22

GoogleCodeExporter commented 8 years ago
Hi,

I looked into this, but I see F_DELETE(encoder) line at the end of the 
function. The stream should be automatically detached in such place. So if the 
process blocks the file it means that there is a bug inside Stream 
implementation, I'm going to look at it.

Leaving as New for now

Original comment by kobalicek.petr on 3 Oct 2012 at 10:44