diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 765bf47436..7adf981f4e 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,10 @@ +2011-12-15 Doug Evans + + * exceptions.c (catcher_list_size): New function. + (last_message): Delete. + (exception_messages, exception_messages_size): New static globals. + (throw_it): Use exception_messages array to handle nested calls. + 2011-12-15 Tom Tromey * symfile.c (symbol_file_add_with_addrs_or_offsets): Remove bad diff --git a/gdb/exceptions.c b/gdb/exceptions.c index b9c96beedd..6cae8910cb 100644 --- a/gdb/exceptions.c +++ b/gdb/exceptions.c @@ -68,6 +68,22 @@ struct catcher /* Where to go for throw_exception(). */ static struct catcher *current_catcher; +/* Return length of current_catcher list. */ + +static int +catcher_list_size (void) +{ + int size; + struct catcher *catcher; + + for (size = 0, catcher = current_catcher; + catcher != NULL; + catcher = catcher->prev) + ++size; + + return size; +} + EXCEPTIONS_SIGJMP_BUF * exceptions_state_mc_init (volatile struct gdb_exception *exception, return_mask mask) @@ -220,8 +236,6 @@ throw_exception (struct gdb_exception exception) EXCEPTIONS_SIGLONGJMP (current_catcher->buf, exception.reason); } -static char *last_message; - void deprecated_throw_reason (enum return_reason reason) { @@ -359,23 +373,53 @@ print_any_exception (struct ui_file *file, const char *prefix, } } +/* A stack of exception messages. + This is needed to handle nested calls to throw_it: we don't want to + xfree space for a message before it's used. + This can happen if we throw an exception during a cleanup: + An outer TRY_CATCH may have an exception message it wants to print, + but while doing cleanups further calls to throw_it are made. + + This is indexed by the size of the current_catcher list. + It is a dynamically allocated array so that we don't care how deeply + GDB nests its TRY_CATCHs. */ +static char **exception_messages; + +/* The number of currently allocated entries in exception_messages. */ +static int exception_messages_size; + static void ATTRIBUTE_NORETURN ATTRIBUTE_PRINTF (3, 0) throw_it (enum return_reason reason, enum errors error, const char *fmt, va_list ap) { struct gdb_exception e; char *new_message; + int depth = catcher_list_size (); - /* Save the message. Create the new message before deleting the - old, the new message may include the old message text. */ + gdb_assert (depth > 0); + + /* Note: The new message may use an old message's text. */ new_message = xstrvprintf (fmt, ap); - xfree (last_message); - last_message = new_message; + + if (depth > exception_messages_size) + { + int old_size = exception_messages_size; + + exception_messages_size = depth + 10; + exception_messages = (char **) xrealloc (exception_messages, + exception_messages_size + * sizeof (char *)); + memset (exception_messages + old_size, 0, + (exception_messages_size - old_size) * sizeof (char *)); + } + + xfree (exception_messages[depth - 1]); + exception_messages[depth - 1] = new_message; /* Create the exception. */ e.reason = reason; e.error = error; - e.message = last_message; + e.message = new_message; /* Throw the exception. */ throw_exception (e);