Can you please review this patch?

Thanks in advance!

Wenqian Pei
------------------------------------------------------------------发件人:Alan 
Bateman <alan.bate...@oracle.com>发送时间:2018年1月3日(星期三) 01:30收件人:裴文谦(右席) 
<wenqian.pe...@alibaba-inc.com>; core-libs-dev 
<core-libs-dev@openjdk.java.net>抄 送:陆传胜(传胜) <chuansheng....@alipay.com>; 
李三红(三红) <sanhong....@alibaba-inc.com>主 题:Re: RFR: 8194154 patch for crash at 
File.getCanonicalPath()
On 25/12/2017 09:40, Wenqian Pei wrote:
> Hi:
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8194154
>
> We found that if user defines -Duser.dir like "/home/a/b/c/", jvm will crash 
>at File.getCanonicalPath() in java process bootstrap (before invoking user's 
>java code). The native implematation of canonicalize_md.c:collapsible(char 
>*names) has problem in processing double '/', parameter 'names' need 
>normalized before JNI_CALL.
>
user.dir is a "read-only" property and should never be changed on the 
command-line or in a running VM (it breaks APIs in other areas to have 
user.dir be different to the actual working directory). Someday we need 
to figure out how to enforce this.

In the mean-time, the runtime shouldn't crash so I agree it should be fixed.

-Alan.

Reply via email to