Daniel D. Daugherty wrote:
Mandy Chung wrote:
Dan,

Can you review the fix for:
 6924497: HotSpotDiagnosticsMXBean.getDiagnosticOptions throws NPE

Webrev at:
 http://cr.openjdk.java.net/~mchung/6924497/webrev.00/
src/share/classes/sun/management/Flag.java
   line 95 - I don't local var 'c' is needed here or on line 99.


It was used for verification.  It's no longer needed and I'll update it.
src/share/native/sun/management/Flag.c
   No comments.

Thanks for the review.

Mandy
I'll be on the look  out for the VM fix also.

Dan





The VM fix for 6898160 updates the jmm_GetVMGlobals function to filter out unsupported type and the returning array may contain null elements. The sun.management.Flag class needs to check for null elements and handle unsupported types gracefully.

Thanks
Mandy


Reply via email to