Mercurial > SDL_sound_CoreAudio

changeset 172:705dcbf94639

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Sound_DecodeAll() is now more robust.
author Ryan C. Gordon <icculus@icculus.org>
date Sat, 01 Dec 2001 01:12:58 +0000
parents 697fabe1453e
children 7e5151f8dd16
files SDL_sound.c
diffstat 1 files changed, 7 insertions(+), 4 deletions(-) [+]
line wrap: on
line diff
--- a/SDL_sound.c	Sat Dec 01 01:12:40 2001 +0000
+++ b/SDL_sound.c	Sat Dec 01 01:12:58 2001 +0000
@@ -638,17 +638,19 @@
 {
     Sound_SampleInternal *internal = NULL;
     void *buf = NULL;
-    void *ptr;
     Uint32 newBufSize = 0;
 
     BAIL_IF_MACRO(!initialized, ERR_NOT_INITIALIZED, 0);
+    BAIL_IF_MACRO(sample->flags & SOUND_SAMPLEFLAG_EOF, ERR_PREV_EOF, 0);
+    BAIL_IF_MACRO(sample->flags & SOUND_SAMPLEFLAG_ERROR, ERR_PREV_ERROR, 0);
 
     internal = (Sound_SampleInternal *) sample->opaque;
 
     while ( ((sample->flags & SOUND_SAMPLEFLAG_EOF) == 0) &&
             ((sample->flags & SOUND_SAMPLEFLAG_ERROR) == 0) )
     {
-        ptr = realloc(buf, newBufSize + sample->buffer_size);
+        Uint32 br = Sound_Decode(sample);
+        void *ptr = realloc(buf, newBufSize + br);
         if (ptr == NULL)
         {
             sample->flags |= SOUND_SAMPLEFLAG_ERROR;
@@ -656,14 +658,15 @@
         } /* if */
         else
         {
-            Uint32 br;
             buf = ptr;
-            br = Sound_Decode(sample);
             memcpy( ((char *) buf) + newBufSize, sample->buffer, br );
             newBufSize += br;
         } /* else */
     } /* while */
 
+    if (buf == NULL)  /* ...in case first call to realloc() fails... */
+        return(sample->buffer_size);
+
     if (internal->buffer != sample->buffer)
         free(internal->buffer);