annotate src/share/vm/utilities/growableArray.cpp @ 1721:413ad0331a0c

6977924: Changes for 6975078 produce build error with certain gcc versions Summary: The changes introduced for 6975078 assign badHeapOopVal to the _allocation field in the ResourceObj class. In 32 bit linux builds with certain versions of gcc this assignment will be flagged as an error while compiling allocation.cpp. In 32 bit builds the constant value badHeapOopVal (which is cast to an intptr_t) is negative. The _allocation field is typed as an unsigned intptr_t and gcc catches this as an error. Reviewed-by: jcoomes, ysr, phh
author johnc
date Wed, 18 Aug 2010 10:59:06 -0700
parents c18cbe5936b8
children f95d63e2154a
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1 /*
1552
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 596
diff changeset
2 * Copyright (c) 1997, 2005, Oracle and/or its affiliates. All rights reserved.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
3 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
a61af66fc99e Initial load
duke
parents:
diff changeset
4 *
a61af66fc99e Initial load
duke
parents:
diff changeset
5 * This code is free software; you can redistribute it and/or modify it
a61af66fc99e Initial load
duke
parents:
diff changeset
6 * under the terms of the GNU General Public License version 2 only, as
a61af66fc99e Initial load
duke
parents:
diff changeset
7 * published by the Free Software Foundation.
a61af66fc99e Initial load
duke
parents:
diff changeset
8 *
a61af66fc99e Initial load
duke
parents:
diff changeset
9 * This code is distributed in the hope that it will be useful, but WITHOUT
a61af66fc99e Initial load
duke
parents:
diff changeset
10 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
a61af66fc99e Initial load
duke
parents:
diff changeset
11 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
a61af66fc99e Initial load
duke
parents:
diff changeset
12 * version 2 for more details (a copy is included in the LICENSE file that
a61af66fc99e Initial load
duke
parents:
diff changeset
13 * accompanied this code).
a61af66fc99e Initial load
duke
parents:
diff changeset
14 *
a61af66fc99e Initial load
duke
parents:
diff changeset
15 * You should have received a copy of the GNU General Public License version
a61af66fc99e Initial load
duke
parents:
diff changeset
16 * 2 along with this work; if not, write to the Free Software Foundation,
a61af66fc99e Initial load
duke
parents:
diff changeset
17 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
a61af66fc99e Initial load
duke
parents:
diff changeset
18 *
1552
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 596
diff changeset
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 596
diff changeset
20 * or visit www.oracle.com if you need additional information or have any
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 596
diff changeset
21 * questions.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
22 *
a61af66fc99e Initial load
duke
parents:
diff changeset
23 */
a61af66fc99e Initial load
duke
parents:
diff changeset
24 # include "incls/_precompiled.incl"
a61af66fc99e Initial load
duke
parents:
diff changeset
25 # include "incls/_growableArray.cpp.incl"
a61af66fc99e Initial load
duke
parents:
diff changeset
26
a61af66fc99e Initial load
duke
parents:
diff changeset
27 #ifdef ASSERT
a61af66fc99e Initial load
duke
parents:
diff changeset
28 void GenericGrowableArray::set_nesting() {
a61af66fc99e Initial load
duke
parents:
diff changeset
29 if (on_stack()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
30 _nesting = Thread::current()->resource_area()->nesting();
a61af66fc99e Initial load
duke
parents:
diff changeset
31 }
a61af66fc99e Initial load
duke
parents:
diff changeset
32 }
a61af66fc99e Initial load
duke
parents:
diff changeset
33
a61af66fc99e Initial load
duke
parents:
diff changeset
34 void GenericGrowableArray::check_nesting() {
a61af66fc99e Initial load
duke
parents:
diff changeset
35 // Check for insidious allocation bug: if a GrowableArray overflows, the
a61af66fc99e Initial load
duke
parents:
diff changeset
36 // grown array must be allocated under the same ResourceMark as the original.
a61af66fc99e Initial load
duke
parents:
diff changeset
37 // Otherwise, the _data array will be deallocated too early.
a61af66fc99e Initial load
duke
parents:
diff changeset
38 if (on_stack() &&
a61af66fc99e Initial load
duke
parents:
diff changeset
39 _nesting != Thread::current()->resource_area()->nesting()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
40 fatal("allocation bug: GrowableArray could grow within nested ResourceMark");
a61af66fc99e Initial load
duke
parents:
diff changeset
41 }
a61af66fc99e Initial load
duke
parents:
diff changeset
42 }
a61af66fc99e Initial load
duke
parents:
diff changeset
43 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
44
a61af66fc99e Initial load
duke
parents:
diff changeset
45 void* GenericGrowableArray::raw_allocate(int elementSize) {
596
83ef1482304c 6806226: Signed integer overflow in growable array code causes JVM crash
jmasa
parents: 0
diff changeset
46 assert(_max >= 0, "integer overflow");
83ef1482304c 6806226: Signed integer overflow in growable array code causes JVM crash
jmasa
parents: 0
diff changeset
47 size_t byte_size = elementSize * (size_t) _max;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
48 if (on_stack()) {
596
83ef1482304c 6806226: Signed integer overflow in growable array code causes JVM crash
jmasa
parents: 0
diff changeset
49 return (void*)resource_allocate_bytes(byte_size);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
50 } else if (on_C_heap()) {
596
83ef1482304c 6806226: Signed integer overflow in growable array code causes JVM crash
jmasa
parents: 0
diff changeset
51 return (void*)AllocateHeap(byte_size, "GrET in " __FILE__);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
52 } else {
596
83ef1482304c 6806226: Signed integer overflow in growable array code causes JVM crash
jmasa
parents: 0
diff changeset
53 return _arena->Amalloc(byte_size);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
54 }
a61af66fc99e Initial load
duke
parents:
diff changeset
55 }