re: [PATCH (try 2)] dsound: Copy SetFormat format more directly if we're not in PRIMARYWRITE mode

[Maarten Lankhorst]

NAK.

+		HeapFree(GetProcessHeap(), 0, device->primary_pwfx);
+		device->primary_pwfx = DSOUND_CopyFormat(passed_fmt);
Copy to a tmp buffer first and return E_OUTOFMEMORY instead of reassigning primary_pwfx to NULL?

~Maarten

[PATCH (try 2)] dsound: Copy SetFormat format more directly if we're not in PRIMARYWRITE mode

[Andrew Eikum]

On Wed, May 29, 2013 at 01:10:56PM +0200, Maarten Lankhorst wrote:
> NAK.
> -
> HeapFree(GetProcessHeap(), 0, device->primary_pwfx);
> -
> device->primary_pwfx = DSOUND_CopyFormat(passed_fmt);
>
> Copy to a tmp buffer first and return E_OUTOFMEMORY instead of reassigning primary_pwfx to NULL?

CopyFormat will only return NULL if HeapAlloc fails (actually,
CopyFormat will probably crash). I've stopped caring about out of
memory conditions since they add way too much error handling code and
we can't reasonably recover from them anyway.

Andrew