src/cpu/x86/vm/frame_x86.cpp

*** 1,7 ****
  /*
!  * Copyright (c) 1997, 2014, Oracle and/or its affiliates. All rights reserved.
   * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
   *
   * This code is free software; you can redistribute it and/or modify it
   * under the terms of the GNU General Public License version 2 only, as
   * published by the Free Software Foundation.
--- 1,7 ----
  /*
!  * Copyright (c) 1997, 2015, Oracle and/or its affiliates. All rights reserved.
   * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
   *
   * This code is free software; you can redistribute it and/or modify it
   * under the terms of the GNU General Public License version 2 only, as
   * published by the Free Software Foundation.
*** 122,140 ****
  
      }
  
      intptr_t* sender_sp = NULL;
      address   sender_pc = NULL;
  
      if (is_interpreted_frame()) {
        // fp must be safe
        if (!fp_safe) {
          return false;
        }
  
        sender_pc = (address) this->fp()[return_addr_offset];
!       sender_sp = (intptr_t*) addr_at(sender_sp_offset);
  
      } else {
        // must be some sort of compiled/runtime frame
        // fp does not have to be safe (although it could be check for c1?)
  
--- 122,142 ----
  
      }
  
      intptr_t* sender_sp = NULL;
      address   sender_pc = NULL;
+     intptr_t* saved_fp =  NULL;
  
      if (is_interpreted_frame()) {
        // fp must be safe
        if (!fp_safe) {
          return false;
        }
  
        sender_pc = (address) this->fp()[return_addr_offset];
!       sender_sp = (intptr_t*) this->fp()[interpreter_frame_sender_sp_offset];
!       saved_fp = (intptr_t*) this->fp()[link_offset];
  
      } else {
        // must be some sort of compiled/runtime frame
        // fp does not have to be safe (although it could be check for c1?)
  
*** 144,164 ****
        }
  
        sender_sp = _unextended_sp + _cb->frame_size();
        // On Intel the return_address is always the word on the stack
        sender_pc = (address) *(sender_sp-1);
      }
  
  
      // If the potential sender is the interpreter then we can do some more checking
      if (Interpreter::contains(sender_pc)) {
  
        // ebp is always saved in a recognizable place in any code we generate. However
        // only if the sender is interpreted/call_stub (c1 too?) are we certain that the saved ebp
        // is really a frame pointer.
  
-       intptr_t *saved_fp = (intptr_t*)*(sender_sp - frame::sender_sp_offset);
        bool saved_fp_safe = ((address)saved_fp < thread->stack_base()) && (saved_fp > sender_sp);
  
        if (!saved_fp_safe) {
          return false;
        }
--- 146,167 ----
        }
  
        sender_sp = _unextended_sp + _cb->frame_size();
        // On Intel the return_address is always the word on the stack
        sender_pc = (address) *(sender_sp-1);
+       // Note: frame::sender_sp_offset is only valid for compiled frame
+       saved_fp = (intptr_t*)*(sender_sp - frame::sender_sp_offset);
      }
  
  
      // If the potential sender is the interpreter then we can do some more checking
      if (Interpreter::contains(sender_pc)) {
  
        // ebp is always saved in a recognizable place in any code we generate. However
        // only if the sender is interpreted/call_stub (c1 too?) are we certain that the saved ebp
        // is really a frame pointer.
  
        bool saved_fp_safe = ((address)saved_fp < thread->stack_base()) && (saved_fp > sender_sp);
  
        if (!saved_fp_safe) {
          return false;
        }
*** 192,202 ****
        return false;
      }
  
      // Could be the call_stub
      if (StubRoutines::returns_to_call_stub(sender_pc)) {
-       intptr_t *saved_fp = (intptr_t*)*(sender_sp - frame::sender_sp_offset);
        bool saved_fp_safe = ((address)saved_fp < thread->stack_base()) && (saved_fp > sender_sp);
  
        if (!saved_fp_safe) {
          return false;
        }
--- 195,204 ----