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
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
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