Changeset 76986 in vbox

Timestamp:

Jan 25, 2019 12:23:12 PM (6 years ago)

Author:

vboxsync

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

128397

Message:

Audio/DrvAudioCommon: Some more division by zero checks for the DrvAudioHlpBytesToXXX() functions.

File:

• trunk/src/VBox/Devices/Audio/DrvAudioCommon.cpp (modified) (3 diffs)

trunk/src/VBox/Devices/Audio/DrvAudioCommon.cpp

@@ -1245 +1245 @@
     AssertPtrReturn(pProps, 0);
 
+    if (!pProps->uHz) /* Prevent division by zero. */
+        return 0;
+
     const unsigned cbFrame = PDMAUDIOPCMPROPS_F2B(pProps, 1 /* Frame */);
+
+    if (!cbFrame) /* Prevent division by zero. */
+        return 0;
 
     uint64_t uTimeMs = ((cbBytes + cbFrame - 1) / cbFrame) * RT_MS_1SEC;
@@ -1265 +1271 @@
     AssertPtrReturn(pProps, 0);
 
+    if (!pProps->uHz) /* Prevent division by zero. */
+        return 0;
+
     const unsigned cbFrame = PDMAUDIOPCMPROPS_F2B(pProps, 1 /* Frame */);
+
+    if (!cbFrame) /* Prevent division by zero. */
+        return 0;
 
     uint64_t uTimeUs = ((cbBytes + cbFrame - 1) / cbFrame) * RT_US_1SEC;
@@ -1285 +1297 @@
     AssertPtrReturn(pProps, 0);
 
+    if (!pProps->uHz) /* Prevent division by zero. */
+        return 0;
+
     const unsigned cbFrame = PDMAUDIOPCMPROPS_F2B(pProps, 1 /* Frame */);
+
+    if (!cbFrame) /* Prevent division by zero. */
+        return 0;
 
     uint64_t uTimeNs = ((cbBytes + cbFrame - 1) / cbFrame) * RT_NS_1SEC;