Re: [FFmpeg-devel] [PATCH] lavc/options: don't copy priv context when it is not AVOption object

2014-12-05 Thread Lukasz Marek
On 05.12.2014 00:15, Michael Niedermayer wrote: On Thu, Dec 04, 2014 at 12:50:06AM +0100, Lukasz Marek wrote: Subject: [PATCH] lavc/options: don't copy priv context when it is not AVOption object This prevents potential crash when opt API is used without a class. LGTM Pushed _

Re: [FFmpeg-devel] [PATCH] lavc/options: don't copy priv context when it is not AVOption object

2014-12-04 Thread Michael Niedermayer
On Thu, Dec 04, 2014 at 12:50:06AM +0100, Lukasz Marek wrote: > On 03.12.2014 01:28, Michael Niedermayer wrote: > >On Wed, Dec 03, 2014 at 01:24:55AM +0100, Lukasz Marek wrote: > >>On 03.12.2014 01:15, Michael Niedermayer wrote: > >>>On Wed, Dec 03, 2014 at 12:06:47AM +0100, Lukasz Marek wrote: > >

Re: [FFmpeg-devel] [PATCH] lavc/options: don't copy priv context when it is not AVOption object

2014-12-03 Thread Lukasz Marek
On 03.12.2014 01:28, Michael Niedermayer wrote: On Wed, Dec 03, 2014 at 01:24:55AM +0100, Lukasz Marek wrote: On 03.12.2014 01:15, Michael Niedermayer wrote: On Wed, Dec 03, 2014 at 12:06:47AM +0100, Lukasz Marek wrote: This prevents potential crash when opt API is used without a class. Signe

Re: [FFmpeg-devel] [PATCH] lavc/options: don't copy priv context when it is not AVOption object

2014-12-02 Thread Michael Niedermayer
On Wed, Dec 03, 2014 at 01:24:55AM +0100, Lukasz Marek wrote: > On 03.12.2014 01:15, Michael Niedermayer wrote: > >On Wed, Dec 03, 2014 at 12:06:47AM +0100, Lukasz Marek wrote: > >>This prevents potential crash when opt API is used without a class. > >> > >>Signed-off-by: Lukasz Marek > >>--- > >>

Re: [FFmpeg-devel] [PATCH] lavc/options: don't copy priv context when it is not AVOption object

2014-12-02 Thread Lukasz Marek
On 03.12.2014 01:15, Michael Niedermayer wrote: On Wed, Dec 03, 2014 at 12:06:47AM +0100, Lukasz Marek wrote: This prevents potential crash when opt API is used without a class. Signed-off-by: Lukasz Marek --- libavcodec/options.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

Re: [FFmpeg-devel] [PATCH] lavc/options: don't copy priv context when it is not AVOption object

2014-12-02 Thread Michael Niedermayer
On Wed, Dec 03, 2014 at 12:06:47AM +0100, Lukasz Marek wrote: > This prevents potential crash when opt API is used without a class. > > Signed-off-by: Lukasz Marek > --- > libavcodec/options.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/options.c b/libavc

[FFmpeg-devel] [PATCH] lavc/options: don't copy priv context when it is not AVOption object

2014-12-02 Thread Lukasz Marek
This prevents potential crash when opt API is used without a class. Signed-off-by: Lukasz Marek --- libavcodec/options.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/options.c b/libavcodec/options.c index 7f9fb07..42ef754 100644 --- a/libavcodec/options.c +++ b/