Actually, this should be a memcpy().
authorRyan C. Gordon <icculus@icculus.org>
Tue, 20 Aug 2013 23:20:32 -0400
changeset 7676013483acb374
parent 7675 230c1a5e1af5
child 7677 871d43c6968a
Actually, this should be a memcpy().

We already know the exact length we just allocated, and we plan to append
our own null terminator to the end of the copy, so this makes more sense.
src/filesystem/beos/SDL_sysfilesystem.cc
     1.1 --- a/src/filesystem/beos/SDL_sysfilesystem.cc	Tue Aug 20 21:29:40 2013 -0400
     1.2 +++ b/src/filesystem/beos/SDL_sysfilesystem.cc	Tue Aug 20 23:20:32 2013 -0400
     1.3 @@ -63,7 +63,7 @@
     1.4          return NULL;
     1.5      }
     1.6  
     1.7 -    strcpy(retval, str);
     1.8 +    SDL_memcpy(retval, str, len);
     1.9      retval[len] = '/';
    1.10      retval[len+1] = '\0';
    1.11      return retval;