[viogpu] Fix resource leak when cursor init fails

Similar to the framebuffer fix, add cleanup for GPU resource and ID
when VioGpuObj::Init() or GpuObjectAttach() fails in CreateCursor().

Signed-off-by: xz-dev <xiangzhedev@gmail.com>

Author: xz-dev

viogpu/viogpudo/viogpudo.cpp

@@ -3710,6 +3710,7 @@ BOOLEAN VioGpuAdapter::CreateCursor(_In_ CONST DXGKARG_SETPOINTERSHAPE *pSetPoin
 {
     UINT resid, format, size;
     VioGpuObj *obj;
+    BOOLEAN status = TRUE;
     PAGED_CODE();
     DbgPrint(TRACE_LEVEL_INFORMATION,
              ("---> %s - %d: (%d x %d - %d) (%d + %d)\n",
@@ -3730,20 +3731,26 @@ BOOLEAN VioGpuAdapter::CreateCursor(_In_ CONST DXGKARG_SETPOINTERSHAPE *pSetPoin
     obj = new (NonPagedPoolNx) VioGpuObj();
     if (!obj->Init(size, &m_CursorSegment))
     {
-        VioGpuDbgBreak();
         DbgPrint(TRACE_LEVEL_FATAL, ("<--- %s Failed to init obj size = %d\n", __FUNCTION__, size));
-        delete obj;
-        return FALSE;
+        status = FALSE;
     }
-    if (!GpuObjectAttach(resid, obj))
+    else if (!GpuObjectAttach(resid, obj))
     {
-        VioGpuDbgBreak();
         DbgPrint(TRACE_LEVEL_FATAL, ("<--- %s Failed to attach gpu object\n", __FUNCTION__));
+        status = FALSE;
+    }
+    if (!status)
+    {
+        VioGpuDbgBreak();
+        m_CtrlQueue.DestroyResource(resid);
+        m_Idr.PutId(resid);
         delete obj;
-        return FALSE;
     }
-    m_pCursorBuf = obj;
-    return TRUE;
+    else
+    {
+        m_pCursorBuf = obj;
+    }
+    return status;
 }
 
 BOOLEAN VioGpuAdapter::UpdateCursor(_In_ CONST DXGKARG_SETPOINTERSHAPE *pSetPointerShape,