diff options
| author | Mark Thompson <sw@jkqxz.net> | 2017-10-08 15:19:17 +0100 |
|---|---|---|
| committer | Mark Thompson <sw@jkqxz.net> | 2017-10-09 00:11:53 +0100 |
| commit | 309d660775e2b47af6723a0477c4d753bc0c54f4 (patch) | |
| tree | cc1e0ea5ec58693ba4eafe9394b7dc69bf49abea | |
| parent | f3602875b3255c533900df1c7bb4e78ef5e1ce08 (diff) | |
| download | ffmpeg-309d660775e2b47af6723a0477c4d753bc0c54f4.tar.gz | |
hwcontext: Perform usual initialisation on derived device contexts
The initialisation should be common. For libmfx, it was previously
happening in the derivation function and this moves it out. For VAAPI,
it fixes some failures when deriving from a DRM device because this
initialisation did not run.
| -rw-r--r-- | libavutil/hwcontext.c | 4 | ||||
| -rw-r--r-- | libavutil/hwcontext_qsv.c | 10 |
2 files changed, 4 insertions, 10 deletions
diff --git a/libavutil/hwcontext.c b/libavutil/hwcontext.c index 2f4ee9661e..048e82126f 100644 --- a/libavutil/hwcontext.c +++ b/libavutil/hwcontext.c @@ -650,6 +650,10 @@ int av_hwdevice_ctx_create_derived(AVBufferRef **dst_ref_ptr, goto fail; done: + ret = av_hwdevice_ctx_init(dst_ref); + if (ret < 0) + goto fail; + *dst_ref_ptr = dst_ref; return 0; diff --git a/libavutil/hwcontext_qsv.c b/libavutil/hwcontext_qsv.c index 75057f7d52..f1d16d8bf9 100644 --- a/libavutil/hwcontext_qsv.c +++ b/libavutil/hwcontext_qsv.c @@ -1037,16 +1037,6 @@ static int qsv_device_derive_from_child(AVHWDeviceContext *ctx, goto fail; } - ret = qsv_device_init(ctx); - if (ret < 0) - goto fail; - if (s->handle_type != handle_type) { - av_log(ctx, AV_LOG_ERROR, "Error in child device handle setup: " - "type mismatch (%d != %d).\n", s->handle_type, handle_type); - err = AVERROR_UNKNOWN; - goto fail; - } - return 0; fail: |