hwcontext: Perform usual initialisation on derived device contexts
authorMark Thompson <sw@jkqxz.net>
Sun, 8 Oct 2017 14:19:17 +0000 (15:19 +0100)
committerMark Thompson <sw@jkqxz.net>
Sun, 11 Feb 2018 22:11:10 +0000 (22:11 +0000)
The initialisation should be common.  For libmfx, it was previously
happening in the derivation function and this moves it out.

libavutil/hwcontext.c
libavutil/hwcontext_qsv.c

index 2630087..cccbd62 100644 (file)
@@ -645,6 +645,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;
 
index 9270b22..9fa603c 100644 (file)
@@ -989,7 +989,6 @@ static int qsv_device_derive_from_child(AVHWDeviceContext *ctx,
                                         int flags)
 {
     AVQSVDeviceContext *hwctx = ctx->hwctx;
-    QSVDeviceContext       *s = ctx->internal->priv;
 
     mfxVersion    ver = { { 3, 1 } };
     mfxHDL        handle;
@@ -1058,16 +1057,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: