re PR libgcj/21703 (hang when rapidly calling String.intern())

PR libgcj/21703:
	* java/lang/ref/natReference.cc (find_slot): Handle case where
	table has no NULL entries.
	* java/lang/natString.cc (_Jv_StringFindSlot): Handle case where
	table has no NULL entries.

From-SVN: r100153
This commit is contained in:
Tom Tromey 2005-05-25 15:23:56 +00:00 committed by Tom Tromey
parent 56c0cf75a0
commit 93f8e21b71
3 changed files with 22 additions and 6 deletions

View File

@ -1,3 +1,11 @@
2005-05-21 Tom Tromey <tromey@redhat.com>
PR libgcj/21703:
* java/lang/ref/natReference.cc (find_slot): Handle case where
table has no NULL entries.
* java/lang/natString.cc (_Jv_StringFindSlot): Handle case where
table has no NULL entries.
2005-05-22 Andreas Jaeger <aj@suse.de>
* java/lang/mprec.c (mult, lshift, b2d, d2b): Add parenthesis to

View File

@ -1,6 +1,6 @@
// natString.cc - Implementation of java.lang.String native methods.
/* Copyright (C) 1998, 1999, 2000, 2001, 2002, 2003, 2004 Free Software Foundation
/* Copyright (C) 1998, 1999, 2000, 2001, 2002, 2003, 2004, 2005 Free Software Foundation
This file is part of libgcj.
@ -64,7 +64,7 @@ _Jv_StringFindSlot (jchar* data, jint len, jint hash)
int index = start_index;
/* step must be non-zero, and relatively prime with strhash_size. */
jint step = (hash ^ (hash >> 16)) | 1;
for (;;)
do
{
jstring* ptr = &strhash[index];
jstring value = (jstring) UNMASK_PTR (*ptr);
@ -81,8 +81,12 @@ _Jv_StringFindSlot (jchar* data, jint len, jint hash)
&& memcmp(JvGetStringChars(value), data, 2*len) == 0)
return (ptr);
index = (index + step) & (strhash_size - 1);
JvAssert (index != start_index);
}
while (index != start_index);
// Note that we can have INDEX == START_INDEX if the table has no
// NULL entries but does have DELETED_STRING entries.
JvAssert (deleted_index >= 0);
return &strhash[deleted_index];
}
/* Calculate a hash code for the string starting at PTR at given LENGTH.

View File

@ -1,6 +1,6 @@
// natReference.cc - Native code for References
/* Copyright (C) 2001, 2002, 2003 Free Software Foundation
/* Copyright (C) 2001, 2002, 2003, 2005 Free Software Foundation
This file is part of libgcj.
@ -78,7 +78,7 @@ find_slot (jobject key)
int start_index = hcode & (hash_size - 1);
int index = start_index;
int deleted_index = -1;
for (;;)
do
{
object_list *ptr = &hash[index];
if (ptr->reference == key)
@ -96,8 +96,12 @@ find_slot (jobject key)
JvAssert (ptr->reference == DELETED_REFERENCE);
}
index = (index + step) & (hash_size - 1);
JvAssert (index != start_index);
}
while (index != start_index);
// Note that we can have INDEX == START_INDEX if the table has no
// NULL entries but does have DELETED entries.
JvAssert (deleted_index >= 0);
return &hash[deleted_index];
}
static void