diff options
author | Mark Thompson <sw@jkqxz.net> | 2018-03-27 00:04:32 +0100 |
---|---|---|
committer | Mark Thompson <sw@jkqxz.net> | 2018-03-28 00:19:45 +0100 |
commit | 44000b7744a0a3c425bc6d8d924b4efa866aad39 (patch) | |
tree | 908fce427ce688d71723b4ba7b45478261f0c3d3 /libavutil | |
parent | b3e9f3f5f52d2c769e0cb6ab1676e16c858e19b3 (diff) | |
download | ffmpeg-streaming-44000b7744a0a3c425bc6d8d924b4efa866aad39.zip ffmpeg-streaming-44000b7744a0a3c425bc6d8d924b4efa866aad39.tar.gz |
hwcontext_d3d11: Fix crash with valid adapter but no device
This crash was introduced by 8bbf2dacbfb4ead1535dea411035994f507f517d,
which could incorrectly overwrite the failure result from creating the
device.
Fixes ticket #7108.
Reviewed-by: wm4 <nfxjfg@googlemail.com>
Diffstat (limited to 'libavutil')
-rw-r--r-- | libavutil/hwcontext_d3d11va.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/libavutil/hwcontext_d3d11va.c b/libavutil/hwcontext_d3d11va.c index 960883c..d39fdd3 100644 --- a/libavutil/hwcontext_d3d11va.c +++ b/libavutil/hwcontext_d3d11va.c @@ -556,8 +556,6 @@ static int d3d11va_device_create(AVHWDeviceContext *ctx, const char *device, } } - hr = mD3D11CreateDevice(pAdapter, pAdapter ? D3D_DRIVER_TYPE_UNKNOWN : D3D_DRIVER_TYPE_HARDWARE, NULL, creationFlags, NULL, 0, - D3D11_SDK_VERSION, &device_hwctx->device, NULL, NULL); if (pAdapter) { DXGI_ADAPTER_DESC2 desc; hr = IDXGIAdapter2_GetDesc(pAdapter, &desc); @@ -565,8 +563,12 @@ static int d3d11va_device_create(AVHWDeviceContext *ctx, const char *device, av_log(ctx, AV_LOG_INFO, "Using device %04x:%04x (%ls).\n", desc.VendorId, desc.DeviceId, desc.Description); } - IDXGIAdapter_Release(pAdapter); } + + hr = mD3D11CreateDevice(pAdapter, pAdapter ? D3D_DRIVER_TYPE_UNKNOWN : D3D_DRIVER_TYPE_HARDWARE, NULL, creationFlags, NULL, 0, + D3D11_SDK_VERSION, &device_hwctx->device, NULL, NULL); + if (pAdapter) + IDXGIAdapter_Release(pAdapter); if (FAILED(hr)) { av_log(ctx, AV_LOG_ERROR, "Failed to create Direct3D device (%lx)\n", (long)hr); return AVERROR_UNKNOWN; |