It looks good.Checking 'name' instead of 'debug_name' before 'debug_name' is allocated would be even better.
But it'd be a multi-line fix. :)
Thanks, Serguei On 11/25/13 9:49 AM, Dmitry Samersoff wrote:
Hi Everyone, Please review an XS fix - missed free() call discovered by cppcheck tool. http://cr.openjdk.java.net/~dsamersoff/JDK-6969276/webrev.01/ -Dmitry