--- old/src/share/vm/runtime/arguments.cpp 2015-06-16 15:47:59.000000000 -0500 +++ new/src/share/vm/runtime/arguments.cpp 2015-06-16 15:47:59.000000000 -0500 @@ -609,11 +609,11 @@ if (is_neg) { int_v = -int_v; } - if (CommandLineFlags::intAtPut(name, &int_v, origin)) { + if (CommandLineFlags::intAtPut(name, &int_v, origin) == Flag::SUCCESS) { return true; } uint uint_v = (uint) v; - if (!is_neg && CommandLineFlags::uintAtPut(name, &uint_v, origin)) { + if (!is_neg && CommandLineFlags::uintAtPut(name, &uint_v, origin) == Flag::SUCCESS) { return true; } intx_v = (intx) v; @@ -2043,12 +2043,12 @@ // Settings to encourage splitting. if (!FLAG_IS_CMDLINE(NewRatio)) { if (FLAG_SET_CMDLINE(uintx, NewRatio, 2) != Flag::SUCCESS) { - return false; + status = false; } } if (!FLAG_IS_CMDLINE(ScavengeBeforeFullGC)) { if (FLAG_SET_CMDLINE(bool, ScavengeBeforeFullGC, false) != Flag::SUCCESS) { - return false; + status = false; } } } @@ -3097,11 +3097,15 @@ #endif // INCLUDE_MANAGEMENT // CreateMinidumpOnCrash is removed, and replaced by CreateCoredumpOnCrash } else if (match_option(option, "-XX:+CreateMinidumpOnCrash")) { - FLAG_SET_CMDLINE(bool, CreateCoredumpOnCrash, true); + if (FLAG_SET_CMDLINE(bool, CreateCoredumpOnCrash, true) != Flag::SUCCESS) { + return JNI_EINVAL; + } jio_fprintf(defaultStream::output_stream(), "CreateMinidumpOnCrash is replaced by CreateCoredumpOnCrash: CreateCoredumpOnCrash is on\n"); } else if (match_option(option, "-XX:-CreateMinidumpOnCrash")) { - FLAG_SET_CMDLINE(bool, CreateCoredumpOnCrash, false); + if (FLAG_SET_CMDLINE(bool, CreateCoredumpOnCrash, false) != Flag::SUCCESS) { + return JNI_EINVAL; + } jio_fprintf(defaultStream::output_stream(), "CreateMinidumpOnCrash is replaced by CreateCoredumpOnCrash: CreateCoredumpOnCrash is off\n"); } else if (match_option(option, "-XX:", &tail)) { // -XX:xxxx