<html>
  <head>
    <meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    Hi Dmitry!<br>
    <br>
    I think you should surround the if body with braces, or you'll get
    unconditional return from the function otherwise.<br>
    <br>
    <pre style="color: rgb(0, 0, 0); font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; background-color: rgb(238, 238, 238);"> 217   if (last_slash == NULL)
<span class="new" style="color: blue; font-weight: bold;"> 218     free(debug_pathname);</span>
 219     return -1;
</pre>
    <br>
    Should be<br class="Apple-interchange-newline">
    <br>
    <pre style="color: rgb(0, 0, 0); font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; background-color: rgb(238, 238, 238);"> 217   if (last_slash == NULL) {
<span class="new" style="color: blue; font-weight: bold;"> 218     free(debug_pathname);</span>
 219     return -1;
 220   } 
</pre>
    <br>
    Sincerely yours,<br>
    Ivan<br>
    <br>
    <br>
    <div class="moz-cite-prefix">On 25.11.2013 21:49, Dmitry Samersoff
      wrote:<br>
    </div>
    <blockquote cite="mid:52938D9C.2060806@oracle.com" type="cite">
      <pre wrap="">Hi Everyone,

Please review an XS fix - missed free() call discovered by cppcheck tool.

<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~dsamersoff/JDK-6969276/webrev.01/">http://cr.openjdk.java.net/~dsamersoff/JDK-6969276/webrev.01/</a>

-Dmitry

</pre>
    </blockquote>
    <br>
  </body>
</html>