diff --git a/libvo/vo_direct3d.c b/libvo/vo_direct3d.c index 090f48bcf8..6e8cdaf1fe 100644 --- a/libvo/vo_direct3d.c +++ b/libvo/vo_direct3d.c @@ -674,7 +674,7 @@ static int preinit(const char *arg) priv = calloc(1, sizeof(struct global_priv)); if (!priv) { mp_msg(MSGT_VO, MSGL_ERR, "Allocating private memory failed.\n"); - return -1; + goto err_out; } /* FIXME @@ -685,26 +685,26 @@ static int preinit(const char *arg) priv->d3d9_dll = LoadLibraryA("d3d9.dll"); if (!priv->d3d9_dll) { mp_msg(MSGT_VO, MSGL_ERR, "Unable to dynamically load d3d9.dll\n"); - return -1; + goto err_out; } priv->pDirect3DCreate9 = (void *)GetProcAddress(priv->d3d9_dll, "Direct3DCreate9"); if (!priv->pDirect3DCreate9) { mp_msg(MSGT_VO, MSGL_ERR, "Unable to find entry point of Direct3DCreate9\n"); - return -1; + goto err_out; } priv->d3d_handle = priv->pDirect3DCreate9(D3D_SDK_VERSION); if (!priv->d3d_handle) { mp_msg(MSGT_VO, MSGL_ERR, "Initializing Direct3D failed.\n"); - return -1; + goto err_out; } if (FAILED(IDirect3D9_GetAdapterDisplayMode(priv->d3d_handle, D3DADAPTER_DEFAULT, &disp_mode))) { mp_msg(MSGT_VO, MSGL_ERR, "Reading display mode failed.\n"); - return -1; + goto err_out; } /* Store in priv->desktop_fmt the user desktop's colorspace. Usually XRGB. */ @@ -720,7 +720,7 @@ static int preinit(const char *arg) D3DDEVTYPE_HAL, &disp_caps))) { mp_msg(MSGT_VO, MSGL_ERR, "Reading display capabilities failed.\n"); - return -1; + goto err_out; } /* Store relevant information reguarding caps of device */ @@ -745,10 +745,14 @@ static int preinit(const char *arg) */ if (!vo_w32_init()) { mp_msg(MSGT_VO, MSGL_V, "Configuring onscreen window failed.\n"); - return -1; + goto err_out; } return 0; + +err_out: + uninit(); + return -1; } @@ -877,7 +881,9 @@ static void uninit(void) uninit_d3d(); vo_w32_uninit(); /* w32_common framework call */ - FreeLibrary(priv->d3d9_dll); + if (priv->d3d9_dll) + FreeLibrary(priv->d3d9_dll); + priv->d3d9_dll = NULL; free(priv); priv = NULL; }