On Wed, 2018-10-03 at 07:56 +0200, Luca Barbato wrote:
> On 30/09/2018 11:59, Zhong Li wrote:
> > Variable 'ret' hasn't been initialized，thus introducing a random
> > hwupload failure regression due to qsv session uninitialized.
> >
> > Signed-off-by: Zhong Li <zhong...@intel.com>
> > ---
> > libavutil/hwcontext_qsv.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/libavutil/hwcontext_qsv.c b/libavutil/hwcontext_qsv.c
> > index 1887936..f550bac 100644
> > --- a/libavutil/hwcontext_qsv.c
> > +++ b/libavutil/hwcontext_qsv.c
> > @@ -833,7 +833,7 @@ static int
> > qsv_transfer_data_to(AVHWFramesContext *ctx, AVFrame *dst,
> >
> > mfxSyncPoint sync = NULL;
> > mfxStatus err;
> > - int ret;
> > + int ret = 0;
> >
> > while (!s->session_upload_init && !s->session_upload && !ret)
> > {
> > #if HAVE_PTHREADS
> >
>
> Sounds correct, thank you for spotting it!
>
> lu
> _______________________________________________
> libav-devel mailing list
> libav-devel@libav.org
> https://lists.libav.org/mailman/listinfo/libav-devel
Works on my side and fixes an issue with hwupload which I saw. Can,
please, this be merged?
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel
