Rewrite target_signal_to_name().

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index d18a67787eb26062e649e7a2281419464d192037..9ff1de230923f10fa1bb95234b4b4c975b3cbcde 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+2002-01-13  Andrew Cagney  <ac131313@redhat.com>
+
+       * signals.c (target_signal_to_name): Rewrite.  Only use
+       signals[].name when in bounds and non-NULL.
+       
 2002-01-13  Andrew Cagney  <ac131313@redhat.com>
 
        From Petr Ledvina <ledvinap@kae.zcu.cz>:

diff --git a/gdb/signals.c b/gdb/signals.c
index cf0078d94fc76e1b66eb8cff3f72c6170f8141dd..d37ec06ee9ae4858d91a8af0f2b796b784877d6e 100644 (file)
--- 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.  */