Changeset 45960 in vbox for trunk/src

Timestamp:

May 8, 2013 8:30:49 PM (12 years ago)

Author:

vboxsync

Message:

Main/VPX: crash fix if not enabled

File:

• trunk/src/VBox/Main/src-client/VideoRec.cpp (modified) (2 diffs)

trunk/src/VBox/Main/src-client/VideoRec.cpp

@@ -512 +512 @@
     pStrm->pu8YuvBuf = pStrm->VpxRawImage.planes[0];
 
-    pCtx->fEnabled = true;
+    pStrm->fEnabled = true;
     return VINF_SUCCESS;
 }
@@ -537 +537 @@
     {
         PVIDEORECSTREAM pStrm = &pCtx->Strm[uScreen];
-        if (pStrm->Ebml.file != NIL_RTFILE)
-        {
-            int rc = Ebml_WriteWebMFileFooter(&pStrm->Ebml, 0);
-            AssertRC(rc);
-            RTFileClose(pStrm->Ebml.file);
-            pStrm->Ebml.file = NIL_RTFILE;
-        }
-        if (pStrm->Ebml.cue_list)
-        {
-            RTMemFree(pStrm->Ebml.cue_list);
-            pStrm->Ebml.cue_list = NULL;
-        }
-        vpx_img_free(&pStrm->VpxRawImage);
-        vpx_codec_err_t rcv = vpx_codec_destroy(&pStrm->VpxCodec);
-        Assert(rcv == VPX_CODEC_OK);
-        RTMemFree(pStrm->pu8RgbBuf);
-        pStrm->pu8RgbBuf = NULL;
+        if (pStrm->fEnabled)
+        {
+            if (pStrm->Ebml.file != NIL_RTFILE)
+            {
+                int rc = Ebml_WriteWebMFileFooter(&pStrm->Ebml, 0);
+                AssertRC(rc);
+                RTFileClose(pStrm->Ebml.file);
+                pStrm->Ebml.file = NIL_RTFILE;
+            }
+            if (pStrm->Ebml.cue_list)
+            {
+                RTMemFree(pStrm->Ebml.cue_list);
+                pStrm->Ebml.cue_list = NULL;
+            }
+            vpx_img_free(&pStrm->VpxRawImage);
+            vpx_codec_err_t rcv = vpx_codec_destroy(&pStrm->VpxCodec);
+            Assert(rcv == VPX_CODEC_OK);
+            RTMemFree(pStrm->pu8RgbBuf);
+            pStrm->pu8RgbBuf = NULL;
+        }
     }