[FFmpeg-cvslog] hwcontext_d3d11: Fix crash with valid adapter but no device
Mark Thompson
git at videolan.org
Wed Mar 28 02:20:55 EEST 2018
ffmpeg | branch: master | Mark Thompson <sw at jkqxz.net> | Tue Mar 27 00:04:32 2018 +0100| [44000b7744a0a3c425bc6d8d924b4efa866aad39] | committer: Mark Thompson
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 at googlemail.com>
> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=44000b7744a0a3c425bc6d8d924b4efa866aad39
---
libavutil/hwcontext_d3d11va.c | 8 +++++---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/libavutil/hwcontext_d3d11va.c b/libavutil/hwcontext_d3d11va.c
index 960883c9d8..d39fdd3fc8 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;
More information about the ffmpeg-cvslog
mailing list