Wed, 15 May 2013 11:05:09 +0200
6843347: Boundary values in some public GC options cause crashes
Summary: Setting some public integer options to specific values causes crashes or undefined GC behavior. This patchset adds the necessary argument checking for these options.
Reviewed-by: jmasa, brutisso
never@2950 | 1 | /* |
bdelsart@3445 | 2 | * Copyright (c) 2011, 2012, Oracle and/or its affiliates. All rights reserved. |
never@2950 | 3 | * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
never@2950 | 4 | * |
never@2950 | 5 | * This code is free software; you can redistribute it and/or modify it |
never@2950 | 6 | * under the terms of the GNU General Public License version 2 only, as |
never@2950 | 7 | * published by the Free Software Foundation. |
never@2950 | 8 | * |
never@2950 | 9 | * This code is distributed in the hope that it will be useful, but WITHOUT |
never@2950 | 10 | * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
never@2950 | 11 | * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
never@2950 | 12 | * version 2 for more details (a copy is included in the LICENSE file that |
never@2950 | 13 | * accompanied this code). |
never@2950 | 14 | * |
never@2950 | 15 | * You should have received a copy of the GNU General Public License version |
never@2950 | 16 | * 2 along with this work; if not, write to the Free Software Foundation, |
never@2950 | 17 | * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. |
never@2950 | 18 | * |
never@2950 | 19 | * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
never@2950 | 20 | * or visit www.oracle.com if you need additional information or have any |
never@2950 | 21 | * questions. |
never@2950 | 22 | * |
never@2950 | 23 | */ |
never@2950 | 24 | |
never@2950 | 25 | // Platform-specific definitions for method handles. |
never@2950 | 26 | // These definitions are inlined into class MethodHandles. |
never@2950 | 27 | |
never@2950 | 28 | // Adapters |
never@2950 | 29 | enum /* platform_dependent_constants */ { |
never@3255 | 30 | adapter_code_size = NOT_LP64(23000 DEBUG_ONLY(+ 40000)) LP64_ONLY(35000 DEBUG_ONLY(+ 50000)) |
never@2950 | 31 | }; |
never@2950 | 32 | |
never@2950 | 33 | // Additional helper methods for MethodHandles code generation: |
never@2950 | 34 | public: |
never@2950 | 35 | static void load_klass_from_Class(MacroAssembler* _masm, Register klass_reg, Register temp_reg, Register temp2_reg); |
never@2950 | 36 | |
never@2950 | 37 | static void verify_klass(MacroAssembler* _masm, |
coleenp@4037 | 38 | Register obj_reg, SystemDictionary::WKID klass_id, |
never@2950 | 39 | Register temp_reg, Register temp2_reg, |
never@2950 | 40 | const char* error_message = "wrong klass") NOT_DEBUG_RETURN; |
never@2950 | 41 | |
never@2950 | 42 | static void verify_method_handle(MacroAssembler* _masm, Register mh_reg, |
never@2950 | 43 | Register temp_reg, Register temp2_reg) { |
coleenp@4037 | 44 | verify_klass(_masm, mh_reg, SystemDictionary::WK_KLASS_ENUM_NAME(java_lang_invoke_MethodHandle), |
never@2950 | 45 | temp_reg, temp2_reg, |
never@2950 | 46 | "reference is a MH"); |
never@2950 | 47 | } |
never@2950 | 48 | |
twisti@3969 | 49 | static void verify_ref_kind(MacroAssembler* _masm, int ref_kind, Register member_reg, Register temp) NOT_DEBUG_RETURN; |
twisti@3969 | 50 | |
never@3005 | 51 | // Similar to InterpreterMacroAssembler::jump_from_interpreted. |
never@3005 | 52 | // Takes care of special dispatch from single stepping too. |
twisti@3969 | 53 | static void jump_from_method_handle(MacroAssembler* _masm, Register method, |
twisti@3969 | 54 | Register temp, Register temp2, |
twisti@3969 | 55 | bool for_compiler_entry); |
twisti@3969 | 56 | |
twisti@3969 | 57 | static void jump_to_lambda_form(MacroAssembler* _masm, |
twisti@3969 | 58 | Register recv, Register method_temp, |
twisti@3969 | 59 | Register temp2, Register temp3, |
twisti@3969 | 60 | bool for_compiler_entry); |