[binutils-gdb] Simplify print_sys_errmsg

[gdb-buildbot@sergiodj.net]

*** TEST RESULTS FOR COMMIT 5df96a4e6e8f1521274acb5beb54063c35aeec6e ***

commit 5df96a4e6e8f1521274acb5beb54063c35aeec6e
Author:     Tom Tromey <tromey@adacore.com>
AuthorDate: Thu Oct 31 16:46:16 2019 -0600
Commit:     Tom Tromey <tromey@adacore.com>
CommitDate: Fri Nov 1 10:34:21 2019 -0600

    Simplify print_sys_errmsg
    
    On irc, Christian pointed out that print_sys_errmsg could be
    simplified by avoiding alloca entirely.  This patch implements this.
    
    gdb/ChangeLog
    2019-11-01  Tom Tromey  <tromey@adacore.com>
    
            * utils.c (print_sys_errmsg): Simplify.
    
    Change-Id: Ic399dade274ea61b63ef0540b3a3be2f0f80160a

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 500c4868c8..34e332b033 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,7 @@
+2019-11-01  Tom Tromey  <tromey@adacore.com>
+
+	* utils.c (print_sys_errmsg): Simplify.
+
 2019-11-01  Tom Tromey  <tromey@adacore.com>
 
 	* gdbsupport/mingw-strerror.c (safe_strerror): Constify result.
diff --git a/gdb/utils.c b/gdb/utils.c
index e06eeddeef..f7fae35729 100644
--- a/gdb/utils.c
+++ b/gdb/utils.c
@@ -629,15 +629,10 @@ void
 print_sys_errmsg (const char *string, int errcode)
 {
   const char *err = safe_strerror (errcode);
-  char *combined = (char *) alloca (strlen (err) + strlen (string) + 3);
-  strcpy (combined, string);
-  strcat (combined, ": ");
-  strcat (combined, err);
-
   /* We want anything which was printed on stdout to come out first, before
      this message.  */
   gdb_flush (gdb_stdout);
-  fprintf_unfiltered (gdb_stderr, "%s.\n", combined);
+  fprintf_unfiltered (gdb_stderr, "%s: %s.\n", string, err);
 }
 
 /* Control C eventually causes this to be called, at a convenient time.  */