Review request: JDK-6519127 Vista: user.home property not set correctly

Alexey Utkin alexey.utkin at oracle.com
Fri Jan 18 15:30:59 UTC 2013


Hi Everyone,
Please review the fix.

Bug description:
     https://jbs.oracle.com/bugs/browse/JDK-6519127

Here is the suggested fix:
     http://cr.openjdk.java.net/~uta/openjdk-webrevs/JDK-6519127/webrev.02/

1] The registry branch was removed from the code as obsolete. It was 
actual only for Windows 95 M3 Beta. This approach is obsolete and is not 
supported by MS for decades.
[http://blogs.msdn.com/b/oldnewthing/archive/2003/11/03/55532.aspx]
For compatibility reasons the Registry path
"HCU\Software\Microsoft\Windows\CurrentVersion\Explorer\Shell Folders"
is filling ones and is not supported in actual state.
Even more, in case of execute-as-service, impersonation, and "RunAs" 
call, User Registry hive could be unloaded or be outsider.

That fix should resolve the bunch of corner-case problems with 
impersonation and migration.

2] KF_FLAG_CREATE and CSIDL_FLAG_CREATE need to be use is case, than 
Java application is applied as Administrative Tool for Users management. 
Potentially the Java call could be the first profile request. For the 
case the real folder need to be created in time.

3] __try/__except trick for potentially absent DLL entry is known and 
successfully used in AWT. That simplifies code and checks function 
signatures on link time.

4] Function [GetJavaProperties] was changed for better 
"initialization-finished" condition checking in low-risky raise condition.

5] The pre-requirements for JDK build includes section
"Windows i586: Microsoft Visual Studio 2010 Compilers" about installed 
Windows SDK v 7.0a or later. That includes actual headers for Shell Lib
version 6.0.6000 with [SHGetKnownFolderPath] entry.
 
http://hg.openjdk.java.net/jdk8/build/raw-file/tip/README-builds.htmll#msvc32


Regards,
-uta




More information about the core-libs-dev mailing list