ryujinx/SDL
1
0
Fork 0
mirror of https://github.com/Ryujinx/SDL.git synced 2024-12-25 08:35:31 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fixed bug 2467 - bad memcpy in SDL_OpenAudio/open_audio_device/prepare_audiospec chain

Browse source 
Rainer Deyke

If 'SDL_OpenAudio' is called with 'obtained == NULL', 'prepare_audiospec' performs a bad 'memcpy' with the destination and source pointing to the same block of memory.  The problem appears to be on in 'SDL_OpenAudio', which calls open_audio_device with 'obtained = desired' when 'obtained == NULL'.  'open_audio_device' cannot deal with 'desired' and 'obtained' pointing to the same block of memory but can deal with 'obtained == NULL'
This commit is contained in:
Sam Lantinga 2014-06-24 01:38:21 -07:00
parent 3344db40ff
commit cbafb158b0
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/audio/SDL_audio.c
View file

@ -1094,7 +1094,7 @@ SDL_OpenAudio(SDL_AudioSpec * desired, SDL_AudioSpec * obtained)
id = open_audio_device(NULL, 0, desired, obtained, id = open_audio_device(NULL, 0, desired, obtained,
SDL_AUDIO_ALLOW_ANY_CHANGE, 1); SDL_AUDIO_ALLOW_ANY_CHANGE, 1);
} else { } else {
id = open_audio_device(NULL, 0, desired, desired, 0, 1); id = open_audio_device(NULL, 0, desired, NULL, 0, 1);
} }
SDL_assert((id == 0) || (id == 1)); SDL_assert((id == 0) || (id == 1));