Changeset 26004 in vbox for trunk/src/VBox/Main/DisplayImpl.cpp

Timestamp:

Jan 25, 2010 2:36:17 PM (15 years ago)

Author:

vboxsync

Message:

png_write_data_fn: another (uint32_t) cast to shut up MSC, added limit+overflow assertion.

File:

• trunk/src/VBox/Main/DisplayImpl.cpp (modified) (2 diffs)

trunk/src/VBox/Main/DisplayImpl.cpp

@@ -217 +217 @@
         {
             uint32_t cbNew = pCtx->cbPNG + (uint32_t)cb;
+            AssertReturnVoidStmt(cbNew > pCtx->cbPNG && cbNew <= _1G, pCtx->rc = VERR_TOO_MUCH_DATA);
             cbNew = RT_ALIGN_32(cbNew, 4096) + 4096;
 
@@ -231 +232 @@
 
         memcpy(pCtx->pu8PNG + pCtx->cbPNG, p, cb);
-        pCtx->cbPNG += cb;
+        pCtx->cbPNG += (uint32_t)cb;
     }
 }