Avoid use of sprintf in gdb/utils.c:make_hex_string

The use of sprintf is discouraged in GDB. Use xsnprintf instead.

gdb/ChangeLog:

        * utils.c (make_hex_string): Replace use of sprintf by use of
        xsnprintf.

Tested on x86_64-linux.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index ac916def8bcd7060914589f01db9869d10e4749e..c12de3a02858ca29c7462cd5bc09df43b584f456 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+2014-12-13  Joel Brobecker  <brobecker@adacore.com>
+
+       * utils.c (make_hex_string): Replace use of sprintf by use of
+       xsnprintf.
+
 2014-12-13  Joel Brobecker  <brobecker@adacore.com>
 
        * compile/compile-object-load.c (link_callbacks_multiple_definition)

diff --git a/gdb/utils.c b/gdb/utils.c
index 1ab183c2ce7f3c9adb2c2e679134648991c9773e..ea2b18a801e8fc803df7ca6a38cb785ed26f9b55 100644 (file)
--- a/gdb/utils.c
+++ b/gdb/utils.c
@@ -1124,7 +1124,7 @@ make_hex_string (const gdb_byte *data, size_t length)
 
   p = result;
   for (i = 0; i < length; ++i)
-    p += sprintf (p, "%02x", data[i]);
+    p += xsnprintf (p, 2, "%02x", data[i]);
   *p = '\0';
   return result;
 }