It probably needs to be converted into a scoped_array like in
JPEGCodec::Decode. Can you file a bug for this on http://crbug.com/ ?

On Thu, Jul 2, 2009 at 5:59 AM, runtime<[email protected]> wrote:
>
> Hi
> There is memory leak in the module jpeg_codec.cc, member function
>
> bool JPEGCodec::Encode(const unsigned char* input, ColorFormat format,
>                       int w, int h, int row_byte_width,
>                       int quality, std::vector<unsigned char>*
> output);
>
> code
>
>    // output row after converting
>    unsigned char* row = new unsigned char[w * 3];
>
>    while (cinfo.next_scanline < cinfo.image_height) {
>      converter(&input[cinfo.next_scanline * row_byte_width], w, row);
>      jpeg_write_scanlines(&cinfo, &row, 1);
>    }
>    delete[] row;
>
> The allocated in row pointer memory will not be released if error
> happens in libjpeg.
>
> >
>

--~--~---------~--~----~------------~-------~--~----~
Chromium Developers mailing list: [email protected] 
View archives, change email options, or unsubscribe: 
    http://groups.google.com/group/chromium-dev
-~----------~----~----~----~------~----~------~--~---

Reply via email to