Moving to core-libs-dev. Code reviews don't take place on jdk-dev but on the component area mailing lists.

Thanks,
David

On 8/02/2018 6:10 AM, yumin qi wrote:
Hi,

   Please review the fix (extra small) for:
   bug 8194154: https://bugs.openjdk.java.net/browse/JDK-8194154
   webrev: http://cr.openjdk.java.net/~minqi/8194154/

   Summary: canonicalize will check and fold double (or more) slashes, but
in function collapsible(char*) and splitNames(char*, char**), it failed to
process strings like "//". This results in the former does not give a
correct number of substrings and the later does not give a correct array of
substrings. The fix add a check if the character is '/' after a '/'.

     The test case added will crash without the fix.

  Thanks
  Yumin

Reply via email to