* remote.c (remote_write_bytes): Add default case to switch

initializing ``todo''. Ditto for ``nr_bytes''.
* top.c (catch_errors): Always initialize ``val''.
* solib.c (info_sharedlibrary_command): Handle bfd_get_arch_size
returning an unknown size.
* gdbtypes.c (count_virtual_fns): Always initialize ``vfuncs''.
* breakpoint.c (break_at_finish_at_depth_command_1): Initialise
extra_args to NULL.
(break_at_finish_command_1): Ditto.

6 files changed, 33 insertions, 4 deletions

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 5902026..9b527bb 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,15 @@
+Mon Nov 27 11:45:52 2000  Andrew Cagney  <cagney@b1.cygnus.com>
+
+	* remote.c (remote_write_bytes): Add default case to switch
+ 	initializing ``todo''. Ditto for ``nr_bytes''.
+ 	* top.c (catch_errors): Always initialize ``val''.
+	* solib.c (info_sharedlibrary_command): Handle bfd_get_arch_size
+ 	returning an unknown size.
+	* gdbtypes.c (count_virtual_fns): Always initialize ``vfuncs''.
+	* breakpoint.c (break_at_finish_at_depth_command_1): Initialise
+ 	extra_args to NULL.
+	(break_at_finish_command_1): Ditto.
+ 
 Mon Nov 27 11:27:06 2000  Andrew Cagney  <cagney@b1.cygnus.com>
 
 	* TODO: Add GFDL updates to 5.1 release criteria.
diff --git a/gdb/breakpoint.c b/gdb/breakpoint.c
index 6f648f6..d7b128c 100644
--- a/gdb/breakpoint.c
+++ b/gdb/breakpoint.c
@@ -4905,7 +4905,9 @@ break_at_finish_at_depth_command_1 (char *arg, int flag, int from_tty)
 {
   struct frame_info *frame;
   CORE_ADDR low, high, selected_pc = 0;
-  char *extra_args, *level_arg, *addr_string;
+  char *extra_args = NULL;
+  char *level_arg;
+  char *addr_string;
   int extra_args_len = 0, if_arg = 0;
 
   if (!arg ||
@@ -4983,7 +4985,7 @@ break_at_finish_command_1 (char *arg, int flag, int from_tty)
   struct symtabs_and_lines sals;
   struct symtab_and_line sal;
   struct cleanup *old_chain;
-  char *extra_args;
+  char *extra_args = NULL;
   int extra_args_len = 0;
   int i, if_arg = 0;
 
diff --git a/gdb/gdbtypes.c b/gdb/gdbtypes.c
index d9f6e0e..c54d472 100644
--- a/gdb/gdbtypes.c
+++ b/gdb/gdbtypes.c
@@ -1986,6 +1986,8 @@ count_virtual_fns (struct type *dclass)
   struct type *pbc = primary_base_class (dclass);
   if (pbc)
     vfuncs = count_virtual_fns (pbc);
+  else
+    vfuncs = 0;
 
   for (fn = 0; fn < TYPE_NFN_FIELDS (dclass); fn++)
     for (oi = 0; oi < TYPE_FN_FIELDLIST_LENGTH (dclass, fn); oi++)
diff --git a/gdb/remote.c b/gdb/remote.c
index 2b8279c..9b49b97 100644
--- a/gdb/remote.c
+++ b/gdb/remote.c
@@ -3363,7 +3363,10 @@ remote_write_bytes (CORE_ADDR memaddr, char *myaddr, int len)
       todo = min (len, max_buf_size / 2);
       break;
     case PACKET_SUPPORT_UNKNOWN:
-      internal_error ("remote_write_bytes: bad switch");
+      internal_error ("%s:%d: remote_write_bytes: bad internal state",
+		      __FILE__, __LINE__);
+    default:
+      internal_error ("%s:%d: bad switch", __FILE__, __LINE__);
     }
   
   /* Append <memaddr> */
@@ -3427,7 +3430,10 @@ remote_write_bytes (CORE_ADDR memaddr, char *myaddr, int len)
       *p = '\0';
       break;
     case PACKET_SUPPORT_UNKNOWN:
-      internal_error ("remote_write_bytes: bad switch");
+      internal_error ("%s:%d: remote_write_bytes: bad internal state",
+		      __FILE__, __LINE__);
+    default:
+      internal_error ("%s:%d: bad switch", __FILE__, __LINE__);
     }
   
   putpkt_binary (buf, (int) (p - buf));
diff --git a/gdb/solib.c b/gdb/solib.c
index 288a54a..b8ffaa3 100644
--- a/gdb/solib.c
+++ b/gdb/solib.c
@@ -613,6 +613,11 @@ info_sharedlibrary_command (char *ignore, int from_tty)
       addr_width = 16 + 4;
       addr_fmt = "016l";
     }
+  else
+    {
+      internal_error ("%s:%d: bfd_get_arch_size() returned unknown size %d",
+		      __FILE__, __LINE__, arch_size);
+    }
 
   update_solib_list (from_tty, 0);
 
diff --git a/gdb/top.c b/gdb/top.c
index 0b04c3c..1022f84 100644
--- a/gdb/top.c
+++ b/gdb/top.c
@@ -605,6 +605,8 @@ catch_errors (catch_errors_ftype *func, PTR args, char *errstring,
   caught = SIGSETJMP (catch);
   if (!caught)
     val = (*func) (args);
+  else
+    val = 0;
   catch_return = saved_catch;
 
   /* FIXME: cagney/1999-11-05: A correct FUNC implementation will