Review request for fix for 7200297

bill.pittore at oracle.com bill.pittore at oracle.com
Wed Nov 14 14:27:54 PST 2012


This bug has to do with the jdwp and hprof agents not parsing the 
sun.boot.library.path (dll_dir) correctly since the fix for 6819213 went 
in some years ago. This bug popped up while working on a particular 
platform that does not have the ability to set LD_LIBRARY_PATH before 
running the VM. As documented in the bug, on most platforms even if the 
sun.boot.library.path consists of multiple paths and the jdwp or hprof 
code fails to load a dependent lib, the system falls back to using 
LD_LIBRARY_PATH so the failure is masked. On some other platforms, this 
failover doesn't exist so we get an error when trying to load jdwp and 
hprof dependent libs.

Some notes on a couple of files.
*
debugInit.c, hprof_init.c*:
Rearranged the init order so that the jvmti pointer gets initialized 
before attempting to load the npt shared library.

*linker_md.c, hprof_md.c*
Much of the platform code in hprof and jdwp is duplicated and this 
change is no different. Based on the code in hotspot 
os_solaris/windows.cpp it parses the boot library path and attempts to 
find the library.
*
error_messages.c*
Fixed a bug in the error message code that blindly dereferenced the npt 
pointer even if it wasn't set because of an error in loading.

Webrev: http://cr.openjdk.java.net/~bpittore/7200297/webrev.00/

Ran the ute nsk/jdwp nsk/jvmti nsk/hprof tests, the JCK jdwp/jvmti tests 
and some command line testing on windows and linux.

thanks,
bill


-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20121114/a4f004fb/attachment.html 


More information about the serviceability-dev mailing list