[FFmpeg-devel] [PATCH] avcodec/qsvenc_h264: fix segfault when a53 SEI is not available

2016-10-04 Thread Nablet Developer
Signed-off-by: Nablet Developer 
---
 libavcodec/qsvenc_h264.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavcodec/qsvenc_h264.c b/libavcodec/qsvenc_h264.c
index c1f6003..f5b01bb 100644
--- a/libavcodec/qsvenc_h264.c
+++ b/libavcodec/qsvenc_h264.c
@@ -53,7 +53,7 @@ static int qsv_h264_set_encode_ctrl(AVCodecContext *avctx,
 int res;
 
 res = ff_alloc_a53_sei(frame, sizeof(mfxPayload) + 2, 
(void**)&payload, &sei_size);
-if (res < 0)
+if (res < 0 || !payload)
 return res;
 
 sei_data = (mfxU8*)(payload + 1);
-- 
1.8.3.1

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


Re: [FFmpeg-devel] [PATCH] avcodec/qsvenc_h264: fix segfault when a53 SEI is not available

2016-10-05 Thread Michael Niedermayer
On Wed, Oct 05, 2016 at 09:34:25AM -0400, Nablet Developer wrote:
> Signed-off-by: Nablet Developer 
> ---
>  libavcodec/qsvenc_h264.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

applied

thx

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

It is dangerous to be right in matters on which the established authorities
are wrong. -- Voltaire


signature.asc
Description: Digital signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


Re: [FFmpeg-devel] [PATCH] avcodec/qsvenc_h264: fix segfault when a53 SEI is not available

2016-10-05 Thread Michael Niedermayer
On Wed, Oct 05, 2016 at 11:35:08PM +0200, Michael Niedermayer wrote:
> On Wed, Oct 05, 2016 at 09:34:25AM -0400, Nablet Developer wrote:
> > Signed-off-by: Nablet Developer 
> > ---
> >  libavcodec/qsvenc_h264.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> applied

actually, before applying, one question

you have set the author for your git to "Nablet Developer"
is this intended? Do you not want your name to be in the Author field ?
If you dont mind your name to be in the author field then please
resubmit the patch with correct Author

[...]

thx

-- 
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Asymptotically faster algorithms should always be preferred if you have
asymptotical amounts of data


signature.asc
Description: Digital signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


Re: [FFmpeg-devel] [PATCH] avcodec/qsvenc_h264: fix segfault when a53 SEI is not available

2016-10-06 Thread nablet developer

> 
> you have set the author for your git to "Nablet Developer"
> is this intended? Do you not want your name to be in the Author field ?
> If you dont mind your name to be in the author field then please
> resubmit the patch with correct Author
> 

yes, it's intentional, please apply as is with git Author field set to "Nablet 
Developer". thx.

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


Re: [FFmpeg-devel] [PATCH] avcodec/qsvenc_h264: fix segfault when a53 SEI is not available

2016-10-06 Thread Michael Niedermayer
On Thu, Oct 06, 2016 at 03:08:18PM +0700, nablet developer wrote:
> 
> > 
> > you have set the author for your git to "Nablet Developer"
> > is this intended? Do you not want your name to be in the Author field ?
> > If you dont mind your name to be in the author field then please
> > resubmit the patch with correct Author
> > 
> 
> yes, it's intentional, please apply as is with git Author field set to 
> "Nablet Developer". thx.

ok, applied

thanks

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

Frequently ignored answer#1 FFmpeg bugs should be sent to our bugtracker. User
questions about the command line tools should be sent to the ffmpeg-user ML.
And questions about how to use libav* should be sent to the libav-user ML.


signature.asc
Description: Digital signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel