Changeset 29542 in vbox for trunk/src/VBox/Frontends/VirtualBox

Timestamp:

May 17, 2010 1:41:20 PM (15 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

61682

Message:

Main: better handle null shapes

Location:

trunk/src/VBox/Frontends/VirtualBox/src

Files:

• VBoxConsoleView.cpp (modified) (2 diffs)
• runtime/UISession.cpp (modified) (2 diffs)

trunk/src/VBox/Frontends/VirtualBox/src/VBoxConsoleView.cpp

@@ -216 +216 @@
         com::SafeArray <BYTE> aShape(ComSafeArrayInArg (pShape));
         size_t cbShapeSize = aShape.size();
-        shape.resize(cbShapeSize);
-        ::memcpy(shape.raw(), aShape.raw(), cbShapeSize);
+        if (cbShapeSize > 0)
+        {
+            shape.resize(cbShapeSize);
+            ::memcpy(shape.raw(), aShape.raw(), cbShapeSize);
+        }
     }
     ~MousePointerChangeEvent()
@@ -228 +231 @@
     uint width() const { return w; }
     uint height() const { return h; }
-    const uchar *shapeData() const { return shape.raw(); }
+    const uchar *shapeData() const { return shape.size() > 0 ? shape.raw() : NULL; }
 private:
     bool vis, alph;

trunk/src/VBox/Frontends/VirtualBox/src/runtime/UISession.cpp

@@ -60 +60 @@
     {
         com::SafeArray <BYTE> aShape(ComSafeArrayInArg (pShape));
-        size_t cbShapeSize = aShape.size();
-        m_shape.resize(cbShapeSize);
-        ::memcpy(m_shape.raw(), aShape.raw(), cbShapeSize);
+        size_t cbShapeSize = aShape.size();        
+        if (cbShapeSize > 0)
+        {
+            m_shape.resize(cbShapeSize);
+            ::memcpy(m_shape.raw(), aShape.raw(), cbShapeSize);
+        }
     }
 
@@ -75 +78 @@
     uint width() const { return m_uWidth; }
     uint height() const { return m_uHeight; }
-    const uchar *shapeData() const { return m_shape.raw(); }
+    const uchar *shapeData() const { return m_shape.size() > 0 ? m_shape.raw() : NULL; }
 
 private: