Hi Mark, Looks fine as is.
The malloc could also be moved inside the if (SetupI18nProps... but that's a bigger change. Roger On 2/18/2015 9:44 AM, Mark Sheppard wrote:
Hi please oblige and review the small change http://cr.openjdk.java.net/~msheppar/8055204/webrev/ which addresses the parfait issue raised in https://bugs.openjdk.java.net/browse/JDK-8055204 frees previously allocated memory in an error return path. regards Mark