[DSOUND] fix double free bug in tests
Robert Reif
reif at earthlink.net
Thu Jan 5 16:37:32 CST 2006
Fix double free in tests. Only happens in interactive mode
when testing duplicated 3D buffers.
-------------- next part --------------
Index: dlls/dsound/tests/ds3d.c
===================================================================
RCS file: /home/wine/wine/dlls/dsound/tests/ds3d.c,v
retrieving revision 1.30
diff -p -u -r1.30 ds3d.c
--- dlls/dsound/tests/ds3d.c 11 Oct 2005 19:57:17 -0000 1.30
+++ dlls/dsound/tests/ds3d.c 5 Jan 2006 22:33:14 -0000
@@ -276,7 +276,7 @@ STOP:
return 0;
}
-void test_buffer(LPDIRECTSOUND dso, LPDIRECTSOUNDBUFFER dsbo,
+void test_buffer(LPDIRECTSOUND dso, LPDIRECTSOUNDBUFFER *dsbo,
BOOL is_primary, BOOL set_volume, LONG volume,
BOOL set_pan, LONG pan, BOOL play, double duration,
BOOL buffer3d, LPDIRECTSOUND3DLISTENER listener,
@@ -290,7 +290,7 @@ void test_buffer(LPDIRECTSOUND dso, LPDI
int ref;
if (set_frequency) {
- rc=IDirectSoundBuffer_SetFrequency(dsbo,frequency);
+ rc=IDirectSoundBuffer_SetFrequency(*dsbo,frequency);
ok(rc==DS_OK||rc==DSERR_CONTROLUNAVAIL,
"IDirectSoundBuffer_SetFrequency() failed to set frequency "
"%s\n",DXGetErrorString8(rc));
@@ -299,19 +299,19 @@ void test_buffer(LPDIRECTSOUND dso, LPDI
}
/* DSOUND: Error: Invalid caps pointer */
- rc=IDirectSoundBuffer_GetCaps(dsbo,0);
+ rc=IDirectSoundBuffer_GetCaps(*dsbo,0);
ok(rc==DSERR_INVALIDPARAM,"IDirectSoundBuffer_GetCaps() should have "
"returned DSERR_INVALIDPARAM, returned: %s\n",DXGetErrorString8(rc));
ZeroMemory(&dsbcaps, sizeof(dsbcaps));
/* DSOUND: Error: Invalid caps pointer */
- rc=IDirectSoundBuffer_GetCaps(dsbo,&dsbcaps);
+ rc=IDirectSoundBuffer_GetCaps(*dsbo,&dsbcaps);
ok(rc==DSERR_INVALIDPARAM,"IDirectSoundBuffer_GetCaps() should have "
"returned DSERR_INVALIDPARAM, returned: %s\n",DXGetErrorString8(rc));
dsbcaps.dwSize=sizeof(dsbcaps);
- rc=IDirectSoundBuffer_GetCaps(dsbo,&dsbcaps);
+ rc=IDirectSoundBuffer_GetCaps(*dsbo,&dsbcaps);
ok(rc==DS_OK,"IDirectSoundBuffer_GetCaps() failed: %s\n",
DXGetErrorString8(rc));
if (rc==DS_OK && winetest_debug > 1) {
@@ -321,11 +321,11 @@ void test_buffer(LPDIRECTSOUND dso, LPDI
/* Query the format size. Note that it may not match sizeof(wfx) */
size=0;
- rc=IDirectSoundBuffer_GetFormat(dsbo,NULL,0,&size);
+ rc=IDirectSoundBuffer_GetFormat(*dsbo,NULL,0,&size);
ok(rc==DS_OK && size!=0,"IDirectSoundBuffer_GetFormat() should have "
"returned the needed size: rc=%s size=%ld\n",DXGetErrorString8(rc),size);
- rc=IDirectSoundBuffer_GetFormat(dsbo,&wfx,sizeof(wfx),NULL);
+ rc=IDirectSoundBuffer_GetFormat(*dsbo,&wfx,sizeof(wfx),NULL);
ok(rc==DS_OK,"IDirectSoundBuffer_GetFormat() failed: %s\n",
DXGetErrorString8(rc));
if (rc==DS_OK && winetest_debug > 1) {
@@ -336,12 +336,12 @@ void test_buffer(LPDIRECTSOUND dso, LPDI
}
/* DSOUND: Error: Invalid frequency buffer */
- rc=IDirectSoundBuffer_GetFrequency(dsbo,0);
+ rc=IDirectSoundBuffer_GetFrequency(*dsbo,0);
ok(rc==DSERR_INVALIDPARAM,"IDirectSoundBuffer_GetFrequency() should have "
"returned DSERR_INVALIDPARAM, returned: %s\n",DXGetErrorString8(rc));
/* DSOUND: Error: Primary buffers don't support CTRLFREQUENCY */
- rc=IDirectSoundBuffer_GetFrequency(dsbo,&freq);
+ rc=IDirectSoundBuffer_GetFrequency(*dsbo,&freq);
ok((rc==DS_OK && !is_primary) || (rc==DSERR_CONTROLUNAVAIL&&is_primary) ||
(rc==DSERR_CONTROLUNAVAIL&&!(dsbcaps.dwFlags&DSBCAPS_CTRLFREQUENCY)),
"IDirectSoundBuffer_GetFrequency() failed: %s\n",DXGetErrorString8(rc));
@@ -352,11 +352,11 @@ void test_buffer(LPDIRECTSOUND dso, LPDI
}
/* DSOUND: Error: Invalid status pointer */
- rc=IDirectSoundBuffer_GetStatus(dsbo,0);
+ rc=IDirectSoundBuffer_GetStatus(*dsbo,0);
ok(rc==DSERR_INVALIDPARAM,"IDirectSoundBuffer_GetStatus() should have "
"returned DSERR_INVALIDPARAM, returned: %s\n",DXGetErrorString8(rc));
- rc=IDirectSoundBuffer_GetStatus(dsbo,&status);
+ rc=IDirectSoundBuffer_GetStatus(*dsbo,&status);
ok(rc==DS_OK,"IDirectSoundBuffer_GetStatus() failed: %s\n",
DXGetErrorString8(rc));
ok(status==0,"status=0x%lx instead of 0\n",status);
@@ -371,12 +371,12 @@ void test_buffer(LPDIRECTSOUND dso, LPDI
return;
/* DSOUND: Error: Invalid format pointer */
- rc=IDirectSoundBuffer_SetFormat(dsbo,0);
+ rc=IDirectSoundBuffer_SetFormat(*dsbo,0);
ok(rc==DSERR_INVALIDPARAM,"IDirectSoundBuffer_SetFormat() should have "
"returned DSERR_INVALIDPARAM, returned: %s\n",DXGetErrorString8(rc));
init_format(&wfx2,WAVE_FORMAT_PCM,11025,16,2);
- rc=IDirectSoundBuffer_SetFormat(dsbo,&wfx2);
+ rc=IDirectSoundBuffer_SetFormat(*dsbo,&wfx2);
ok(rc==DS_OK,"IDirectSoundBuffer_SetFormat() failed: %s\n",
DXGetErrorString8(rc));
@@ -384,7 +384,7 @@ void test_buffer(LPDIRECTSOUND dso, LPDI
* format to what we asked for. It depends on what the soundcard
* supports. So we must re-query the format.
*/
- rc=IDirectSoundBuffer_GetFormat(dsbo,&wfx,sizeof(wfx),NULL);
+ rc=IDirectSoundBuffer_GetFormat(*dsbo,&wfx,sizeof(wfx),NULL);
ok(rc==DS_OK,"IDirectSoundBuffer_GetFormat() failed: %s\n",
DXGetErrorString8(rc));
if (rc==DS_OK &&
@@ -439,7 +439,7 @@ void test_buffer(LPDIRECTSOUND dso, LPDI
if (buffer3d) {
LPDIRECTSOUNDBUFFER temp_buffer;
- rc=IDirectSoundBuffer_QueryInterface(dsbo,&IID_IDirectSound3DBuffer,
+ rc=IDirectSoundBuffer_QueryInterface(*dsbo,&IID_IDirectSound3DBuffer,
(LPVOID *)&buffer);
ok(rc==DS_OK,"IDirectSoundBuffer_QueryInterface() failed: %s\n",
DXGetErrorString8(rc));
@@ -447,38 +447,38 @@ void test_buffer(LPDIRECTSOUND dso, LPDI
return;
/* check the COM interface */
- rc=IDirectSoundBuffer_QueryInterface(dsbo, &IID_IDirectSoundBuffer,
+ rc=IDirectSoundBuffer_QueryInterface(*dsbo, &IID_IDirectSoundBuffer,
(LPVOID *)&temp_buffer);
ok(rc==DS_OK && temp_buffer!=NULL,
"IDirectSoundBuffer_QueryInterface() failed: %s\n",
DXGetErrorString8(rc));
- ok(temp_buffer==dsbo,"COM interface broken: %p != %p\n",
- temp_buffer,dsbo);
+ ok(temp_buffer==*dsbo,"COM interface broken: %p != %p\n",
+ temp_buffer,*dsbo);
ref=IDirectSoundBuffer_Release(temp_buffer);
ok(ref==1,"IDirectSoundBuffer_Release() has %d references, "
"should have 1\n",ref);
temp_buffer=NULL;
- rc=IDirectSound3DBuffer_QueryInterface(dsbo,
+ rc=IDirectSound3DBuffer_QueryInterface(*dsbo,
&IID_IDirectSoundBuffer,
(LPVOID *)&temp_buffer);
ok(rc==DS_OK && temp_buffer!=NULL,
"IDirectSound3DBuffer_QueryInterface() failed: %s\n",
DXGetErrorString8(rc));
- ok(temp_buffer==dsbo,"COM interface broken: %p != %p\n",
- temp_buffer,dsbo);
+ ok(temp_buffer==*dsbo,"COM interface broken: %p != %p\n",
+ temp_buffer,*dsbo);
ref=IDirectSoundBuffer_Release(temp_buffer);
ok(ref==1,"IDirectSoundBuffer_Release() has %d references, "
"should have 1\n",ref);
- ref=IDirectSoundBuffer_Release(dsbo);
+ ref=IDirectSoundBuffer_Release(*dsbo);
ok(ref==0,"IDirectSoundBuffer_Release() has %d references, "
"should have 0\n",ref);
rc=IDirectSound3DBuffer_QueryInterface(buffer,
&IID_IDirectSoundBuffer,
- (LPVOID *)&dsbo);
- ok(rc==DS_OK && dsbo!=NULL,"IDirectSound3DBuffer_QueryInterface() "
+ (LPVOID *)dsbo);
+ ok(rc==DS_OK && *dsbo!=NULL,"IDirectSound3DBuffer_QueryInterface() "
"failed: %s\n",DXGetErrorString8(rc));
/* DSOUND: Error: Invalid buffer */
@@ -501,16 +501,16 @@ void test_buffer(LPDIRECTSOUND dso, LPDI
if (set_volume) {
if (dsbcaps.dwFlags & DSBCAPS_CTRLVOLUME) {
LONG val;
- rc=IDirectSoundBuffer_GetVolume(dsbo,&val);
+ rc=IDirectSoundBuffer_GetVolume(*dsbo,&val);
ok(rc==DS_OK,"IDirectSoundBuffer_GetVolume() failed: %s\n",
DXGetErrorString8(rc));
- rc=IDirectSoundBuffer_SetVolume(dsbo,volume);
+ rc=IDirectSoundBuffer_SetVolume(*dsbo,volume);
ok(rc==DS_OK,"IDirectSoundBuffer_SetVolume() failed: %s\n",
DXGetErrorString8(rc));
} else {
/* DSOUND: Error: Buffer does not have CTRLVOLUME */
- rc=IDirectSoundBuffer_GetVolume(dsbo,&volume);
+ rc=IDirectSoundBuffer_GetVolume(*dsbo,&volume);
ok(rc==DSERR_CONTROLUNAVAIL,"IDirectSoundBuffer_GetVolume() "
"should have returned DSERR_CONTROLUNAVAIL, returned: %s\n",
DXGetErrorString8(rc));
@@ -520,16 +520,16 @@ void test_buffer(LPDIRECTSOUND dso, LPDI
if (set_pan) {
if (dsbcaps.dwFlags & DSBCAPS_CTRLPAN) {
LONG val;
- rc=IDirectSoundBuffer_GetPan(dsbo,&val);
+ rc=IDirectSoundBuffer_GetPan(*dsbo,&val);
ok(rc==DS_OK,"IDirectSoundBuffer_GetPan() failed: %s\n",
DXGetErrorString8(rc));
- rc=IDirectSoundBuffer_SetPan(dsbo,pan);
+ rc=IDirectSoundBuffer_SetPan(*dsbo,pan);
ok(rc==DS_OK,"IDirectSoundBuffer_SetPan() failed: %s\n",
DXGetErrorString8(rc));
} else {
/* DSOUND: Error: Buffer does not have CTRLPAN */
- rc=IDirectSoundBuffer_GetPan(dsbo,&pan);
+ rc=IDirectSoundBuffer_GetPan(*dsbo,&pan);
ok(rc==DSERR_CONTROLUNAVAIL,"IDirectSoundBuffer_GetPan() "
"should have returned DSERR_CONTROLUNAVAIL, returned: %s\n",
DXGetErrorString8(rc));
@@ -541,17 +541,17 @@ void test_buffer(LPDIRECTSOUND dso, LPDI
else
state.wave=wave_generate_la(&wfx,duration,&state.wave_len);
- state.dsbo=dsbo;
+ state.dsbo=*dsbo;
state.wfx=&wfx;
state.buffer_size=dsbcaps.dwBufferBytes;
state.played=state.written=state.offset=0;
buffer_refill(&state,state.buffer_size);
- rc=IDirectSoundBuffer_Play(dsbo,0,0,DSBPLAY_LOOPING);
+ rc=IDirectSoundBuffer_Play(*dsbo,0,0,DSBPLAY_LOOPING);
ok(rc==DS_OK,"IDirectSoundBuffer_Play() failed: %s\n",
DXGetErrorString8(rc));
- rc=IDirectSoundBuffer_GetStatus(dsbo,&status);
+ rc=IDirectSoundBuffer_GetStatus(*dsbo,&status);
ok(rc==DS_OK,"IDirectSoundBuffer_GetStatus() failed: %s\n",
DXGetErrorString8(rc));
ok(status==(DSBSTATUS_PLAYING|DSBSTATUS_LOOPING),
@@ -855,7 +855,7 @@ static HRESULT test_secondary(LPGUID lpG
if (rc==DS_OK && secondary!=NULL) {
double duration;
duration=(move_listener || move_sound?4.0:1.0);
- test_buffer(dso,secondary,0,FALSE,0,FALSE,0,
+ test_buffer(dso,&secondary,0,FALSE,0,FALSE,0,
winetest_interactive,duration,has_3dbuffer,
listener,move_listener,move_sound,FALSE,0);
ref=IDirectSoundBuffer_Release(secondary);
@@ -956,7 +956,7 @@ static HRESULT test_primary(LPGUID lpGui
if (rc==DSERR_CONTROLUNAVAIL)
trace(" No Primary\n");
else if (rc==DS_OK && primary!=NULL) {
- test_buffer(dso,primary,1,TRUE,0,TRUE,0,winetest_interactive &&
+ test_buffer(dso,&primary,1,TRUE,0,TRUE,0,winetest_interactive &&
!(dscaps.dwFlags & DSCAPS_EMULDRIVER),1.0,0,NULL,0,0,
FALSE,0);
if (winetest_interactive) {
@@ -964,7 +964,7 @@ static HRESULT test_primary(LPGUID lpGui
volume = DSBVOLUME_MAX;
for (i = 0; i < 6; i++) {
- test_buffer(dso,primary,1,TRUE,volume,TRUE,0,
+ test_buffer(dso,&primary,1,TRUE,volume,TRUE,0,
winetest_interactive &&
!(dscaps.dwFlags & DSCAPS_EMULDRIVER),
1.0,0,NULL,0,0,FALSE,0);
@@ -973,7 +973,7 @@ static HRESULT test_primary(LPGUID lpGui
pan = DSBPAN_LEFT;
for (i = 0; i < 7; i++) {
- test_buffer(dso,primary,1,TRUE,0,TRUE,pan,
+ test_buffer(dso,&primary,1,TRUE,0,TRUE,pan,
winetest_interactive &&
!(dscaps.dwFlags & DSCAPS_EMULDRIVER),1.0,0,0,0,0,FALSE,0);
pan += ((DSBPAN_RIGHT-DSBPAN_LEFT) / 6);
@@ -1050,7 +1050,7 @@ static HRESULT test_primary_3d(LPGUID lp
ok(rc==DS_OK && primary!=NULL,"IDirectSound_CreateSoundBuffer() "
"failed to create a 3D primary buffer: %s\n",DXGetErrorString8(rc));
if (rc==DS_OK && primary!=NULL) {
- test_buffer(dso,primary,1,FALSE,0,FALSE,0,winetest_interactive &&
+ test_buffer(dso,&primary,1,FALSE,0,FALSE,0,winetest_interactive &&
!(dscaps.dwFlags & DSCAPS_EMULDRIVER),1.0,0,0,0,0,
FALSE,0);
ref=IDirectSoundBuffer_Release(primary);
@@ -1148,7 +1148,7 @@ static HRESULT test_primary_3d_with_list
"should have 1\n",ref);
/* Testing the buffer */
- test_buffer(dso,primary,1,FALSE,0,FALSE,0,
+ test_buffer(dso,&primary,1,FALSE,0,FALSE,0,
winetest_interactive &&
!(dscaps.dwFlags & DSCAPS_EMULDRIVER),1.0,0,
listener,0,0,FALSE,0);
Index: dlls/dsound/tests/ds3d8.c
===================================================================
RCS file: /home/wine/wine/dlls/dsound/tests/ds3d8.c,v
retrieving revision 1.22
diff -p -u -r1.22 ds3d8.c
--- dlls/dsound/tests/ds3d8.c 11 Oct 2005 19:57:17 -0000 1.22
+++ dlls/dsound/tests/ds3d8.c 5 Jan 2006 22:33:15 -0000
@@ -179,7 +179,7 @@ STOP:
return 0;
}
-void test_buffer8(LPDIRECTSOUND8 dso, LPDIRECTSOUNDBUFFER dsbo,
+void test_buffer8(LPDIRECTSOUND8 dso, LPDIRECTSOUNDBUFFER * dsbo,
BOOL is_primary, BOOL set_volume, LONG volume,
BOOL set_pan, LONG pan, BOOL play, double duration,
BOOL buffer3d, LPDIRECTSOUND3DLISTENER listener,
@@ -192,19 +192,19 @@ void test_buffer8(LPDIRECTSOUND8 dso, LP
int ref;
/* DSOUND: Error: Invalid caps pointer */
- rc=IDirectSoundBuffer_GetCaps(dsbo,0);
+ rc=IDirectSoundBuffer_GetCaps(*dsbo,0);
ok(rc==DSERR_INVALIDPARAM,"IDirectSoundBuffer_GetCaps() should have "
"returned DSERR_INVALIDPARAM, returned: %s\n",DXGetErrorString8(rc));
ZeroMemory(&dsbcaps, sizeof(dsbcaps));
/* DSOUND: Error: Invalid caps pointer */
- rc=IDirectSoundBuffer_GetCaps(dsbo,&dsbcaps);
+ rc=IDirectSoundBuffer_GetCaps(*dsbo,&dsbcaps);
ok(rc==DSERR_INVALIDPARAM,"IDirectSoundBuffer_GetCaps() should have "
"returned DSERR_INVALIDPARAM, returned: %s\n",DXGetErrorString8(rc));
dsbcaps.dwSize=sizeof(dsbcaps);
- rc=IDirectSoundBuffer_GetCaps(dsbo,&dsbcaps);
+ rc=IDirectSoundBuffer_GetCaps(*dsbo,&dsbcaps);
ok(rc==DS_OK,"IDirectSoundBuffer_GetCaps() failed: %s\n",
DXGetErrorString8(rc));
if (rc==DS_OK && winetest_debug > 1) {
@@ -214,11 +214,11 @@ void test_buffer8(LPDIRECTSOUND8 dso, LP
/* Query the format size. Note that it may not match sizeof(wfx) */
size=0;
- rc=IDirectSoundBuffer_GetFormat(dsbo,NULL,0,&size);
+ rc=IDirectSoundBuffer_GetFormat(*dsbo,NULL,0,&size);
ok(rc==DS_OK && size!=0,"IDirectSoundBuffer_GetFormat() should have "
"returned the needed size: rc=%s size=%ld\n",DXGetErrorString8(rc),size);
- rc=IDirectSoundBuffer_GetFormat(dsbo,&wfx,sizeof(wfx),NULL);
+ rc=IDirectSoundBuffer_GetFormat(*dsbo,&wfx,sizeof(wfx),NULL);
ok(rc==DS_OK,"IDirectSoundBuffer_GetFormat() failed: %s\n",
DXGetErrorString8(rc));
if (rc==DS_OK && winetest_debug > 1) {
@@ -229,12 +229,12 @@ void test_buffer8(LPDIRECTSOUND8 dso, LP
}
/* DSOUND: Error: Invalid frequency buffer */
- rc=IDirectSoundBuffer_GetFrequency(dsbo,0);
+ rc=IDirectSoundBuffer_GetFrequency(*dsbo,0);
ok(rc==DSERR_INVALIDPARAM,"IDirectSoundBuffer_GetFrequency() should have "
"returned DSERR_INVALIDPARAM, returned: %s\n",DXGetErrorString8(rc));
/* DSOUND: Error: Primary buffers don't support CTRLFREQUENCY */
- rc=IDirectSoundBuffer_GetFrequency(dsbo,&freq);
+ rc=IDirectSoundBuffer_GetFrequency(*dsbo,&freq);
ok((rc==DS_OK && !is_primary) || (rc==DSERR_CONTROLUNAVAIL&&is_primary) ||
(rc==DSERR_CONTROLUNAVAIL&&!(dsbcaps.dwFlags&DSBCAPS_CTRLFREQUENCY)),
"IDirectSoundBuffer_GetFrequency() failed: %s\n",DXGetErrorString8(rc));
@@ -244,11 +244,11 @@ void test_buffer8(LPDIRECTSOUND8 dso, LP
}
/* DSOUND: Error: Invalid status pointer */
- rc=IDirectSoundBuffer_GetStatus(dsbo,0);
+ rc=IDirectSoundBuffer_GetStatus(*dsbo,0);
ok(rc==DSERR_INVALIDPARAM,"IDirectSoundBuffer_GetStatus() should have "
"returned DSERR_INVALIDPARAM, returned: %s\n",DXGetErrorString8(rc));
- rc=IDirectSoundBuffer_GetStatus(dsbo,&status);
+ rc=IDirectSoundBuffer_GetStatus(*dsbo,&status);
ok(rc==DS_OK,"IDirectSoundBuffer_GetStatus() failed: %s\n",
DXGetErrorString8(rc));
ok(status==0,"status=0x%lx instead of 0\n",status);
@@ -263,12 +263,12 @@ void test_buffer8(LPDIRECTSOUND8 dso, LP
return;
/* DSOUND: Error: Invalid format pointer */
- rc=IDirectSoundBuffer_SetFormat(dsbo,0);
+ rc=IDirectSoundBuffer_SetFormat(*dsbo,0);
ok(rc==DSERR_INVALIDPARAM,"IDirectSoundBuffer_SetFormat() should have "
"returned DSERR_INVALIDPARAM, returned: %s\n",DXGetErrorString8(rc));
init_format(&wfx2,WAVE_FORMAT_PCM,11025,16,2);
- rc=IDirectSoundBuffer_SetFormat(dsbo,&wfx2);
+ rc=IDirectSoundBuffer_SetFormat(*dsbo,&wfx2);
ok(rc==DS_OK,"IDirectSoundBuffer_SetFormat() failed: %s\n",
DXGetErrorString8(rc));
@@ -276,7 +276,7 @@ void test_buffer8(LPDIRECTSOUND8 dso, LP
* format to what we asked for. It depends on what the soundcard
* supports. So we must re-query the format.
*/
- rc=IDirectSoundBuffer_GetFormat(dsbo,&wfx,sizeof(wfx),NULL);
+ rc=IDirectSoundBuffer_GetFormat(*dsbo,&wfx,sizeof(wfx),NULL);
ok(rc==DS_OK,"IDirectSoundBuffer_GetFormat() failed: %s\n",
DXGetErrorString8(rc));
if (rc==DS_OK &&
@@ -326,7 +326,7 @@ void test_buffer8(LPDIRECTSOUND8 dso, LP
if (buffer3d) {
LPDIRECTSOUNDBUFFER temp_buffer;
- rc=IDirectSoundBuffer_QueryInterface(dsbo,&IID_IDirectSound3DBuffer,
+ rc=IDirectSoundBuffer_QueryInterface(*dsbo,&IID_IDirectSound3DBuffer,
(LPVOID *)&buffer);
ok(rc==DS_OK,"IDirectSoundBuffer_QueryInterface() failed: %s\n",
DXGetErrorString8(rc));
@@ -334,37 +334,37 @@ void test_buffer8(LPDIRECTSOUND8 dso, LP
return;
/* check the COM interface */
- rc=IDirectSoundBuffer_QueryInterface(dsbo, &IID_IDirectSoundBuffer,
+ rc=IDirectSoundBuffer_QueryInterface(*dsbo, &IID_IDirectSoundBuffer,
(LPVOID *)&temp_buffer);
ok(rc==DS_OK && temp_buffer!=NULL,
"IDirectSoundBuffer_QueryInterface() failed: %s\n",
DXGetErrorString8(rc));
- ok(temp_buffer==dsbo,"COM interface broken: %p != %p\n",
- temp_buffer,dsbo);
+ ok(temp_buffer==*dsbo,"COM interface broken: %p != %p\n",
+ temp_buffer,*dsbo);
ref=IDirectSoundBuffer_Release(temp_buffer);
ok(ref==1,"IDirectSoundBuffer_Release() has %d references, "
"should have 1\n",ref);
temp_buffer=NULL;
- rc=IDirectSound3DBuffer_QueryInterface(dsbo, &IID_IDirectSoundBuffer,
+ rc=IDirectSound3DBuffer_QueryInterface(*dsbo, &IID_IDirectSoundBuffer,
(LPVOID *)&temp_buffer);
ok(rc==DS_OK && temp_buffer!=NULL,
"IDirectSound3DBuffer_QueryInterface() failed: %s\n",
DXGetErrorString8(rc));
- ok(temp_buffer==dsbo,"COM interface broken: %p != %p\n",
- temp_buffer,dsbo);
+ ok(temp_buffer==*dsbo,"COM interface broken: %p != %p\n",
+ temp_buffer,*dsbo);
ref=IDirectSoundBuffer_Release(temp_buffer);
ok(ref==1,"IDirectSoundBuffer_Release() has %d references, "
"should have 1\n",ref);
- ref=IDirectSoundBuffer_Release(dsbo);
+ ref=IDirectSoundBuffer_Release(*dsbo);
ok(ref==0,"IDirectSoundBuffer_Release() has %d references, "
"should have 0\n",ref);
rc=IDirectSound3DBuffer_QueryInterface(buffer,
&IID_IDirectSoundBuffer,
- (LPVOID *)&dsbo);
- ok(rc==DS_OK && dsbo!=NULL,"IDirectSound3DBuffer_QueryInterface() "
+ (LPVOID *)dsbo);
+ ok(rc==DS_OK && *dsbo!=NULL,"IDirectSound3DBuffer_QueryInterface() "
"failed: %s\n",DXGetErrorString8(rc));
/* DSOUND: Error: Invalid buffer */
@@ -387,16 +387,16 @@ void test_buffer8(LPDIRECTSOUND8 dso, LP
if (set_volume) {
if (dsbcaps.dwFlags & DSBCAPS_CTRLVOLUME) {
LONG val;
- rc=IDirectSoundBuffer_GetVolume(dsbo,&val);
+ rc=IDirectSoundBuffer_GetVolume(*dsbo,&val);
ok(rc==DS_OK,"IDirectSoundBuffer_GetVolume() failed: %s\n",
DXGetErrorString8(rc));
- rc=IDirectSoundBuffer_SetVolume(dsbo,volume);
+ rc=IDirectSoundBuffer_SetVolume(*dsbo,volume);
ok(rc==DS_OK,"IDirectSoundBuffer_SetVolume() failed: %s\n",
DXGetErrorString8(rc));
} else {
/* DSOUND: Error: Buffer does not have CTRLVOLUME */
- rc=IDirectSoundBuffer_GetVolume(dsbo,&volume);
+ rc=IDirectSoundBuffer_GetVolume(*dsbo,&volume);
ok(rc==DSERR_CONTROLUNAVAIL,"IDirectSoundBuffer_GetVolume() "
"should have returned DSERR_CONTROLUNAVAIL, returned: %s\n",
DXGetErrorString8(rc));
@@ -406,16 +406,16 @@ void test_buffer8(LPDIRECTSOUND8 dso, LP
if (set_pan) {
if (dsbcaps.dwFlags & DSBCAPS_CTRLPAN) {
LONG val;
- rc=IDirectSoundBuffer_GetPan(dsbo,&val);
+ rc=IDirectSoundBuffer_GetPan(*dsbo,&val);
ok(rc==DS_OK,"IDirectSoundBuffer_GetPan() failed: %s\n",
DXGetErrorString8(rc));
- rc=IDirectSoundBuffer_SetPan(dsbo,pan);
+ rc=IDirectSoundBuffer_SetPan(*dsbo,pan);
ok(rc==DS_OK,"IDirectSoundBuffer_SetPan() failed: %s\n",
DXGetErrorString8(rc));
} else {
/* DSOUND: Error: Buffer does not have CTRLPAN */
- rc=IDirectSoundBuffer_GetPan(dsbo,&pan);
+ rc=IDirectSoundBuffer_GetPan(*dsbo,&pan);
ok(rc==DSERR_CONTROLUNAVAIL,"IDirectSoundBuffer_GetPan() "
"should have returned DSERR_CONTROLUNAVAIL, returned: %s\n",
DXGetErrorString8(rc));
@@ -424,17 +424,17 @@ void test_buffer8(LPDIRECTSOUND8 dso, LP
state.wave=wave_generate_la(&wfx,duration,&state.wave_len);
- state.dsbo=dsbo;
+ state.dsbo=*dsbo;
state.wfx=&wfx;
state.buffer_size=dsbcaps.dwBufferBytes;
state.played=state.written=state.offset=0;
buffer_refill8(&state,state.buffer_size);
- rc=IDirectSoundBuffer_Play(dsbo,0,0,DSBPLAY_LOOPING);
+ rc=IDirectSoundBuffer_Play(*dsbo,0,0,DSBPLAY_LOOPING);
ok(rc==DS_OK,"IDirectSoundBuffer_Play() failed: %s\n",
DXGetErrorString8(rc));
- rc=IDirectSoundBuffer_GetStatus(dsbo,&status);
+ rc=IDirectSoundBuffer_GetStatus(*dsbo,&status);
ok(rc==DS_OK,"IDirectSoundBuffer_GetStatus() failed: %s\n",
DXGetErrorString8(rc));
ok(status==(DSBSTATUS_PLAYING|DSBSTATUS_LOOPING),
@@ -749,7 +749,7 @@ static HRESULT test_secondary8(LPGUID lp
if (rc==DS_OK && secondary!=NULL) {
double duration;
duration=(move_listener || move_sound?4.0:1.0);
- test_buffer8(dso,secondary,0,FALSE,0,FALSE,0,
+ test_buffer8(dso,&secondary,0,FALSE,0,FALSE,0,
winetest_interactive,duration,has_3dbuffer,
listener,move_listener,move_sound);
ref=IDirectSoundBuffer_Release(secondary);
@@ -850,14 +850,14 @@ static HRESULT test_primary8(LPGUID lpGu
if (rc == DSERR_CONTROLUNAVAIL)
trace(" No Primary\n");
else if (rc==DS_OK && primary!=NULL) {
- test_buffer8(dso,primary,1,TRUE,0,TRUE,0,winetest_interactive &&
+ test_buffer8(dso,&primary,1,TRUE,0,TRUE,0,winetest_interactive &&
!(dscaps.dwFlags & DSCAPS_EMULDRIVER),1.0,0,NULL,0,0);
if (winetest_interactive) {
LONG volume,pan;
volume = DSBVOLUME_MAX;
for (i = 0; i < 6; i++) {
- test_buffer8(dso,primary,1,TRUE,volume,TRUE,0,
+ test_buffer8(dso,&primary,1,TRUE,volume,TRUE,0,
winetest_interactive &&
!(dscaps.dwFlags & DSCAPS_EMULDRIVER),
1.0,0,NULL,0,0);
@@ -866,7 +866,7 @@ static HRESULT test_primary8(LPGUID lpGu
pan = DSBPAN_LEFT;
for (i = 0; i < 7; i++) {
- test_buffer8(dso,primary,1,TRUE,0,TRUE,pan,
+ test_buffer8(dso,&primary,1,TRUE,0,TRUE,pan,
winetest_interactive &&
!(dscaps.dwFlags & DSCAPS_EMULDRIVER),1.0,0,0,0,0);
pan += ((DSBPAN_RIGHT-DSBPAN_LEFT) / 6);
@@ -943,7 +943,7 @@ static HRESULT test_primary_3d8(LPGUID l
ok(rc==DS_OK && primary!=NULL,"IDirectSound8_CreateSoundBuffer() "
"failed to create a 3D primary buffer: %s\n",DXGetErrorString8(rc));
if (rc==DS_OK && primary!=NULL) {
- test_buffer8(dso,primary,1,FALSE,0,FALSE,0,
+ test_buffer8(dso,&primary,1,FALSE,0,FALSE,0,
winetest_interactive &&
!(dscaps.dwFlags & DSCAPS_EMULDRIVER),1.0,0,0,0,0);
ref=IDirectSoundBuffer_Release(primary);
@@ -1039,7 +1039,7 @@ static HRESULT test_primary_3d_with_list
"should have 1\n",ref);
/* Testing the buffer */
- test_buffer8(dso,primary,1,FALSE,0,FALSE,0,
+ test_buffer8(dso,&primary,1,FALSE,0,FALSE,0,
winetest_interactive &&
!(dscaps.dwFlags & DSCAPS_EMULDRIVER),
1.0,0,listener,0,0);
Index: dlls/dsound/tests/dsound.c
===================================================================
RCS file: /home/wine/wine/dlls/dsound/tests/dsound.c,v
retrieving revision 1.56
diff -p -u -r1.56 dsound.c
--- dlls/dsound/tests/dsound.c 12 Sep 2005 10:30:06 -0000 1.56
+++ dlls/dsound/tests/dsound.c 5 Jan 2006 22:33:16 -0000
@@ -466,7 +466,7 @@ static HRESULT test_primary(LPGUID lpGui
trace("All subsequent tones should be identical to this one.\n");
trace("Listen for stutter, changes in pitch, volume, etc.\n");
}
- test_buffer(dso,primary,1,FALSE,0,FALSE,0,winetest_interactive &&
+ test_buffer(dso,&primary,1,FALSE,0,FALSE,0,winetest_interactive &&
!(dscaps.dwFlags & DSCAPS_EMULDRIVER),5.0,0,0,0,0,FALSE,0);
ref=IDirectSoundBuffer_Release(primary);
@@ -601,7 +601,7 @@ static HRESULT test_primary_secondary(LP
"buffer %s\n",DXGetErrorString8(rc));
if (rc==DS_OK && secondary!=NULL) {
- test_buffer(dso,secondary,0,FALSE,0,FALSE,0,
+ test_buffer(dso,&secondary,0,FALSE,0,FALSE,0,
winetest_interactive,1.0,0,NULL,0,0,FALSE,0);
ref=IDirectSoundBuffer_Release(secondary);
@@ -714,7 +714,7 @@ static HRESULT test_secondary(LPGUID lpG
"buffer %s\n",DXGetErrorString8(rc));
if (rc==DS_OK && secondary!=NULL) {
- test_buffer(dso,secondary,0,FALSE,0,FALSE,0,
+ test_buffer(dso,&secondary,0,FALSE,0,FALSE,0,
winetest_interactive,1.0,0,NULL,0,0,FALSE,0);
ref=IDirectSoundBuffer_Release(secondary);
@@ -874,7 +874,7 @@ static HRESULT test_frequency(LPGUID lpG
"buffer %s\n",DXGetErrorString8(rc));
if (rc==DS_OK && secondary!=NULL) {
- test_buffer(dso,secondary,0,FALSE,0,FALSE,0,
+ test_buffer(dso,&secondary,0,FALSE,0,FALSE,0,
winetest_interactive,1.0,0,NULL,0,0,TRUE,rates[r]);
ref=IDirectSoundBuffer_Release(secondary);
Index: dlls/dsound/tests/dsound8.c
===================================================================
RCS file: /home/wine/wine/dlls/dsound/tests/dsound8.c,v
retrieving revision 1.27
diff -p -u -r1.27 dsound8.c
--- dlls/dsound/tests/dsound8.c 10 Oct 2005 10:27:50 -0000 1.27
+++ dlls/dsound/tests/dsound8.c 5 Jan 2006 22:33:17 -0000
@@ -497,7 +497,7 @@ static HRESULT test_primary8(LPGUID lpGu
trace("All subsequent tones should be identical to this one.\n");
trace("Listen for stutter, changes in pitch, volume, etc.\n");
}
- test_buffer8(dso,primary,1,FALSE,0,FALSE,0,winetest_interactive &&
+ test_buffer8(dso,&primary,1,FALSE,0,FALSE,0,winetest_interactive &&
!(dscaps.dwFlags & DSCAPS_EMULDRIVER),5.0,0,0,0,0);
ref=IDirectSoundBuffer_Release(primary);
@@ -633,7 +633,7 @@ static HRESULT test_primary_secondary8(L
"buffer %s\n",DXGetErrorString8(rc));
if (rc==DS_OK && secondary!=NULL) {
- test_buffer8(dso,secondary,0,FALSE,0,FALSE,0,
+ test_buffer8(dso,&secondary,0,FALSE,0,FALSE,0,
winetest_interactive,1.0,0,NULL,0,0);
ref=IDirectSoundBuffer_Release(secondary);
@@ -746,7 +746,7 @@ static HRESULT test_secondary8(LPGUID lp
"buffer: %s\n",DXGetErrorString8(rc));
if (rc==DS_OK && secondary!=NULL) {
- test_buffer8(dso,secondary,0,FALSE,0,FALSE,0,
+ test_buffer8(dso,&secondary,0,FALSE,0,FALSE,0,
winetest_interactive,1.0,0,NULL,0,0);
ref=IDirectSoundBuffer_Release(secondary);
Index: dlls/dsound/tests/dsound_test.h
===================================================================
RCS file: /home/wine/wine/dlls/dsound/tests/dsound_test.h,v
retrieving revision 1.10
diff -p -u -r1.10 dsound_test.h
--- dlls/dsound/tests/dsound_test.h 18 Nov 2005 12:04:36 -0000 1.10
+++ dlls/dsound/tests/dsound_test.h 5 Jan 2006 22:33:17 -0000
@@ -53,10 +53,10 @@ static const unsigned int formats[][4]={
extern char* wave_generate_la(WAVEFORMATEX*,double,DWORD*);
extern HWND get_hwnd(void);
extern void init_format(WAVEFORMATEX*,int,int,int,int);
-extern void test_buffer(LPDIRECTSOUND,LPDIRECTSOUNDBUFFER,
+extern void test_buffer(LPDIRECTSOUND,LPDIRECTSOUNDBUFFER*,
BOOL,BOOL,LONG,BOOL,LONG,BOOL,double,BOOL,
LPDIRECTSOUND3DLISTENER,BOOL,BOOL,BOOL,DWORD);
-extern void test_buffer8(LPDIRECTSOUND8,LPDIRECTSOUNDBUFFER,
+extern void test_buffer8(LPDIRECTSOUND8,LPDIRECTSOUNDBUFFER*,
BOOL,BOOL,LONG,BOOL,LONG,BOOL,double,BOOL,
LPDIRECTSOUND3DLISTENER,BOOL,BOOL);
extern const char * getDSBCAPS(DWORD xmask);
More information about the wine-patches
mailing list