[physfs] Replace malloc/strcpy by strdup
Christoph Nelles
evilazrael at evilazrael.de
Wed Sep 29 11:33:54 EDT 2010
Hi,
yes, looks like this is a problem:
http://stackoverflow.com/questions/482375/strdup-function
And you are forced to use free(). allocator.malloc/free may use other
functions.
So why worrying about saving one LOC.
Am 29.09.2010 17:25, schrieb Tim Cas:
> Correct me if I'm wrong, but strdup doesn't seem to be a part of the
C90 or C99 standard, but rather POSIX (which is a problem in Windows).
>
> Compare:
> http://www.opengroup.org/onlinepubs/009695399/basedefs/string.h.html
(POSIX implementation)
> http://www.cplusplus.com/reference/clibrary/cstring/ (C99 and/or C++
implementation)
>
> On 29 September 2010 15:27, Tolga Dalman <tolga.dalman at googlemail.com
<mailto:tolga.dalman at googlemail.com>> wrote:
>
> Hi,
>
> the attached patch replaces a malloc and strdup sequence by a plain
> strdup. Also, I fixed a trivial copy & paste error in the error
> checking path.
>
> Thanks
> Tolga Dalman
_______________________________________________
physfs mailing list
physfs at icculus.org <mailto:physfs at icculus.org>
http://icculus.org/mailman/listinfo/physfs
> _______________________________________________
> physfs mailing list
> physfs at icculus.org
> http://icculus.org/mailman/listinfo/physfs
--
Christoph Nelles
E-Mail : evilazrael at evilazrael.de
Jabber : eazrael at evilazrael.net ICQ : 78819723
PGP-Key : ID 0x424FB55B on subkeys.pgp.net
or http://evilazrael.net/pgp.txt
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://icculus.org/pipermail/physfs/attachments/20100929/c4e1e497/attachment.htm>
More information about the physfs
mailing list