I sent a bug report http://code.google.com/p/chromium/issues/detail?id=15990 I don't know I did it correct or not.
Lei Zhang wrote: > 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 -~----------~----~----~----~------~----~------~--~---
