[FFmpeg-devel] [PATCH 2/3] lavc/libopenjpegenc: add layerrates parameter to allow different compression rates per layer

[Michael Niedermayer]

On Mon, Feb 02, 2015 at 05:31:07PM +0100, Jean First wrote:
> On Mon Feb 02 2015 17:20:44 GMT+0100 (CET), Michael Niedermayer wrote:
> > On Mon, Feb 02, 2015 at 01:18:35PM +0100, Jean First wrote:
> >>  libopenjpegenc.c |   42 ++++++++++++++++++++----------------------
> >>  1 file changed, 20 insertions(+), 22 deletions(-)
> >> 6bed682adf441fa060b9fef84df173cd758320ed  0001-lavc-libopenjpegenc-move-opj_create_compress-opj_cio.patch
> >> From fe0afff9c7b8c6f78dcad0720965c6f6a50e7813 Mon Sep 17 00:00:00 2001
> >> From: Jean First <jeanfirst at gmail.com>
> >> Date: Mon, 2 Feb 2015 12:57:03 +0100
> >> Subject: [PATCH] lavc/libopenjpegenc: move opj_create_compress, opj_cio_open
> >>  and opj_set_event_mgr to libopenjpeg_encode_frame
> >>
> >>  libopenjpegenc crashes with "pointer being freed was not allocated" when threading
> >>  is enabled with:
> >>  ffmpeg -i tests/vsynth1/01.pgm -vcodec libopenjpeg file.j2k
> > cannot reproduce
> > do i assume correctly that this is a bug in libopenjpeg and unlreated
> > to this patch ?
> >
> 
> Yes, that's my guess. Any other implementation I saw using libopenjpeg
> did initialise and encode the date in the same thread.

ok, patch applied with slightly updated commit message to clarify
this
also this should be reported to libopenjpeg if its intended to
work or we should disable multiple threads or both

Thanks

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I do not agree with what you have to say, but I'll defend to the death your
right to say it. -- Voltaire
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150202/d4c97ccd/attachment.asc>