Re: RR(XS): JDK-6969276 minor memory leak detected by cppcheck in agent/src/os/linux/symtab.c

2013-11-29 Thread Dmitry Samersoff
Ivan, Good catch. Thank you! -Dmitry On 2013-11-30 00:59, Ivan Gerasimov wrote: > Hi Dmitry! > > I think you should surround the if body with braces, or you'll get > unconditional return from the function otherwise. > > 217 if (last_slash == NULL) > 218 free(debug_pathname); > 219

Re: RR(XS): JDK-6969276 minor memory leak detected by cppcheck in agent/src/os/linux/symtab.c

2013-11-29 Thread Ivan Gerasimov
Hi Dmitry! I think you should surround the if body with braces, or you'll get unconditional return from the function otherwise. 217 if (last_slash == NULL) 218 free(debug_pathname); 219 return -1; Should be 217 if (last_slash == NULL) { 218 free(debug_pathname); 219

hg: jdk8/tl/jdk: 8029348: ProblemList.txt updates (11/2013)

2013-11-29 Thread chris . hegarty
Changeset: 5bcaf730ceb8 Author:tyan Date: 2013-11-29 09:29 + URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/5bcaf730ceb8 8029348: ProblemList.txt updates (11/2013) Reviewed-by: chegar ! test/ProblemList.txt