From ade8f45eafa16fa00f3b47aeebedd8cb2b9deb23 Mon Sep 17 00:00:00 2001 From: Andrew Cagney Date: Sun, 13 Jan 2002 21:58:16 +0000 Subject: Rewrite target_signal_to_name(). --- gdb/ChangeLog | 5 +++++ gdb/signals.c | 11 +++++------ 2 files changed, 10 insertions(+), 6 deletions(-) (limited to 'gdb') diff --git a/gdb/ChangeLog b/gdb/ChangeLog index d18a677..9ff1de2 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,5 +1,10 @@ 2002-01-13 Andrew Cagney + * signals.c (target_signal_to_name): Rewrite. Only use + signals[].name when in bounds and non-NULL. + +2002-01-13 Andrew Cagney + From Petr Ledvina : * signals.c (target_signal_to_name): Verify that SIG is within the bounds of the signals array. diff --git a/gdb/signals.c b/gdb/signals.c index cf0078d..d37ec06 100644 --- a/gdb/signals.c +++ b/gdb/signals.c @@ -210,14 +210,13 @@ target_signal_to_string (enum target_signal sig) char * target_signal_to_name (enum target_signal sig) { - if (sig == TARGET_SIGNAL_UNKNOWN) - /* I think the code which prints this will always print it along with - the string, so no need to be verbose. */ - return "?"; - else if ((sig >= TARGET_SIGNAL_FIRST) && (sig <= TARGET_SIGNAL_LAST)) + if ((sig >= TARGET_SIGNAL_FIRST) && (sig <= TARGET_SIGNAL_LAST) + && signals[sig].name != NULL) return signals[sig].name; else - return signals[sig].name; + /* I think the code which prints this will always print it along + with the string, so no need to be verbose (very old comment). */ + return "?"; } /* Given a name, return its signal. */ -- cgit v1.1