El día 23 de diciembre de 2010 13:35, Arjen Markus <arjen.mar...@deltares.nl> escribió: > Hi José, > > at a first glance this patch (and the one for plgradient) seems okay. > Only one tiny thing: I am not sure free() can be used with a NULL > argument. So, I'd say you have to guard against that. (If no one > picks this up, I will)
Hello, Here: http://linux.die.net/man/3/free says "free(ptr) has already been called before, undefined behaviour occurs. If ptr is NULL, no operation is performed. " I ever check the multiple malloc() returns in this way, following the man page of free() Thanks for your time checking the patches. -- ***************************************** José Luis García Pallero jgpall...@gmail.com (o< / / \ V_/_ Use Debian GNU/Linux and enjoy! ***************************************** ------------------------------------------------------------------------------ Learn how Oracle Real Application Clusters (RAC) One Node allows customers to consolidate database storage, standardize their database environment, and, should the need arise, upgrade to a full multi-node Oracle RAC database without downtime or disruption http://p.sf.net/sfu/oracle-sfdevnl _______________________________________________ Plplot-devel mailing list Plplot-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/plplot-devel
