* linux-low.c (fetch_register): Avoid passing a non string literal
authorPedro Alves <palves@redhat.com>
Mon, 15 Mar 2010 00:31:16 +0000 (00:31 +0000)
committerPedro Alves <palves@redhat.com>
Mon, 15 Mar 2010 00:31:16 +0000 (00:31 +0000)
format to `error'.
(usr_store_inferior_registers): Ditto.

gdb/gdbserver/ChangeLog
gdb/gdbserver/linux-low.c

index b8c26faf0608fc02bd478106f8016e4a536c3c8f..229cceaf0ba03b137f8eaeb60d59203cc7c78c3e 100644 (file)
@@ -1,3 +1,9 @@
+2010-03-15  Pedro Alves  <pedro@codesourcery.com>
+
+       * linux-low.c (fetch_register): Avoid passing a non string literal
+       format to `error'.
+       (usr_store_inferior_registers): Ditto.
+
 2010-03-14  Pedro Alves  <pedro@codesourcery.com>
 
        * linux-low.c (linux_write_memory): Bail out early if peeking
index 1b344844bb1abf30320bab46f039957fdd8d144f..4ccadbc37592afa5cb8eb6dad428677e1809181d 100644 (file)
@@ -2217,23 +2217,13 @@ fetch_register (struct regcache *regcache, int regno)
                (PTRACE_ARG3_TYPE) (uintptr_t) regaddr, 0);
       regaddr += sizeof (PTRACE_XFER_TYPE);
       if (errno != 0)
-       {
-         /* Warning, not error, in case we are attached; sometimes the
-            kernel doesn't let us at the registers.  */
-         char *err = strerror (errno);
-         char *msg = alloca (strlen (err) + 128);
-         sprintf (msg, "reading register %d: %s", regno, err);
-         error (msg);
-         goto error_exit;
-       }
+       error ("reading register %d: %s", regno, strerror (errno));
     }
 
   if (the_low_target.supply_ptrace_register)
     the_low_target.supply_ptrace_register (regcache, regno, buf);
   else
     supply_register (regcache, regno, buf);
-
-error_exit:;
 }
 
 /* Fetch all registers, or just one, from the child process.  */
@@ -2299,14 +2289,7 @@ usr_store_inferior_registers (struct regcache *regcache, int regno)
                return;
 
              if ((*the_low_target.cannot_store_register) (regno) == 0)
-               {
-                 char *err = strerror (errno);
-                 char *msg = alloca (strlen (err) + 128);
-                 sprintf (msg, "writing register %d: %s",
-                          regno, err);
-                 error (msg);
-                 return;
-               }
+               error ("writing register %d: %s", regno, strerror (errno));
            }
          regaddr += sizeof (PTRACE_XFER_TYPE);
        }