Changeset 82463 in vbox for trunk/src/VBox

Timestamp:

Dec 6, 2019 2:29:16 PM (5 years ago)

Author:

vboxsync

Message:

pdmaudioifs.h,Devices/Audio: Corrected the size alignment of PDMAUDIOSTREAMCFG to 8 bytes. bugref:9218

Location:

trunk/src/VBox/Devices/Audio

Files:

• DevIchAc97.cpp (modified) (1 diff)
• DrvAudioCommon.cpp (modified) (1 diff)
• DrvHostDSound.cpp (modified) (1 diff)
• HDAStream.h (modified) (1 diff)
• HDAStreamMap.cpp (modified) (1 diff)

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

@@ -360 +360 @@
 #endif
     /** The stream's current configuration. */
-    PDMAUDIOSTREAMCFG       Cfg; //+104
-    uint32_t                Padding2;
+    PDMAUDIOSTREAMCFG       Cfg; //+108
 #ifdef VBOX_WITH_AUDIO_AC97_ASYNC_IO
     /** Asynchronous I/O state members. */

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

@@ -951 +951 @@
     AssertPtrReturnVoid(pCfg);
 
-    RT_BZERO(pCfg, sizeof(PDMAUDIOSTREAMCFG));
+    RT_ZERO(*pCfg);
 
     pCfg->Backend.cFramesPreBuffering = UINT32_MAX; /* Explicitly set to "undefined". */

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

@@ -114 +114 @@
 {
     /** The stream's acquired configuration. */
-    PDMAUDIOSTREAMCFG  Cfg;
+    PDMAUDIOSTREAMCFG   Cfg;
     /** Buffer alignment. */
-    uint8_t            uAlign;
+    uint8_t             uAlign;
     /** Whether this stream is in an enable state on the DirectSound side. */
-    bool               fEnabled;
+    bool                fEnabled;
+    bool                afPadding[2];
+    /** Size (in bytes) of the DirectSound buffer.
+     *  @note This in *not* the size of the circular buffer above! */
+    DWORD               cbBufSize;
     /** The stream's critical section for synchronizing access. */
-    RTCRITSECT         CritSect;
+    RTCRITSECT          CritSect;
     /** The internal playback / capturing buffer. */
-    PRTCIRCBUF         pCircBuf;
-    /** Size (in bytes) of the DirectSound buffer.
-     *  Note: This in *not* the size of the circular buffer above! */
-    DWORD              cbBufSize;
+    PRTCIRCBUF          pCircBuf;
     union
     {

trunk/src/VBox/Devices/Audio/HDAStream.h

@@ -166 +166 @@
      *  Should match SDFMT. */
     PDMAUDIOSTREAMCFG       Cfg;
-    uint32_t                Padding4;
     /** Timestamp (in ns) of last stream update. */
     uint64_t                tsLastUpdateNs;

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

@@ -156 +156 @@
         PPDMAUDIOSTREAMMAP pMapLR = &pMap->paMappings[0];
 
-        pMapLR->aID[0]   = PDMAUDIOSTREAMCHANNELID_FRONT_LEFT;
-        pMapLR->aID[1]   = PDMAUDIOSTREAMCHANNELID_FRONT_RIGHT;
-        pMapLR->cbFrame  = pProps->cbSample * pProps->cChannels;
-        pMapLR->cbStep   = pProps->cbSample * 2 /* Front left + Front right channels */;
-        pMapLR->offFirst = 0;
-        pMapLR->offNext  = pMapLR->offFirst;
+        pMapLR->aenmIDs[0]  = PDMAUDIOSTREAMCHANNELID_FRONT_LEFT;
+        pMapLR->aenmIDs[1]  = PDMAUDIOSTREAMCHANNELID_FRONT_RIGHT;
+        pMapLR->cbFrame     = pProps->cbSample * pProps->cChannels;
+        pMapLR->cbStep      = pProps->cbSample * 2 /* Front left + Front right channels */;
+        pMapLR->offFirst    = 0;
+        pMapLR->offNext     = pMapLR->offFirst;
 
         rc = hdaR3StreamChannelDataInit(&pMapLR->Data, PDMAUDIOSTREAMCHANNELDATA_FLAGS_NONE);