gdb/

2012-11-26  Maxime Villard  <rustyBSD@gmx.fr>
	    Pedro Alves  <palves@redhat.com>

	* common/linux-osdata.c (linux_xfer_osdata_fds): Decrease buffer
	size parameter passed to readlink by one byte.
	* fbsd-nat.c (fbsd_pid_to_exec_file): Ditto.
	* linux-nat.c (linux_child_pid_to_exec_file): Ditto.
	* nbsd-nat.c (nbsd_pid_to_exec_file): Ditto.
	* inf-child.c (inf_child_fileio_readlink): Decrease local buffer's
	size by one byte.

gdb/gdbserver/
2012-11-26  Maxime Villard  <rustyBSD@gmx.fr>

	* hostio.c (handle_readlink): Decrease buffer size
	parameter passed to readlink by one byte.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 57aee09db71433d02de93ef37e005bf0f3bdb6e7..b14fc8e002c263dd2acbcacde5805fc2e45fe931 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,14 @@
+2012-11-26  Maxime Villard  <rustyBSD@gmx.fr>
+           Pedro Alves  <palves@redhat.com>
+
+       * common/linux-osdata.c (linux_xfer_osdata_fds): Decrease buffer
+       size parameter passed to readlink by one byte.
+       * fbsd-nat.c (fbsd_pid_to_exec_file): Ditto.
+       * linux-nat.c (linux_child_pid_to_exec_file): Ditto.
+       * nbsd-nat.c (nbsd_pid_to_exec_file): Ditto.
+       * inf-child.c (inf_child_fileio_readlink): Decrease local buffer's
+       size by one byte.
+
 2012-11-26  Yao Qi  <yao@codesourcery.com>
 
        * f-valprint.c (f77_create_arrayprint_offset_tbl): Remove

diff --git a/gdb/common/linux-osdata.c b/gdb/common/linux-osdata.c
index d54f9d368c40c31c31d323f136a54645d86411f2..b275495e9cfee5f9a54742aca17bd82a32ce4cb9 100644 (file)
--- a/gdb/common/linux-osdata.c
+++ b/gdb/common/linux-osdata.c
@@ -737,7 +737,7 @@ linux_xfer_osdata_fds (gdb_byte *readbuf,
                            continue;
 
                          fdname = xstrprintf ("%s/%s", pathname, dp2->d_name);
-                         rslt = readlink (fdname, buf, 1000);
+                         rslt = readlink (fdname, buf, sizeof (buf) - 1);
                          if (rslt >= 0)
                            buf[rslt] = '\0';
 

diff --git a/gdb/fbsd-nat.c b/gdb/fbsd-nat.c
index 254a01a25ba44b43f987fc4393709051c8a36247..5eaecdd609c88f754212526cfde3b257dcfe716a 100644 (file)
--- a/gdb/fbsd-nat.c
+++ b/gdb/fbsd-nat.c
@@ -55,7 +55,7 @@ fbsd_pid_to_exec_file (int pid)
 #endif
 
   path = xstrprintf ("/proc/%d/file", pid);
-  if (readlink (path, buf, MAXPATHLEN) == -1)
+  if (readlink (path, buf, MAXPATHLEN - 1) == -1)
     {
       xfree (buf);
       buf = NULL;

diff --git a/gdb/gdbserver/ChangeLog b/gdb/gdbserver/ChangeLog
index efebfb111a06ab84c1fd6a0dd65612f23437f335..0b1fb8109c6a59b894cb647f2e5c9d411259cdef 100644 (file)
--- a/gdb/gdbserver/ChangeLog
+++ b/gdb/gdbserver/ChangeLog
@@ -1,3 +1,8 @@
+2012-11-26  Maxime Villard  <rustyBSD@gmx.fr>
+
+       * hostio.c (handle_readlink): Decrease buffer size
+       parameter passed to readlink by one byte.
+
 2012-11-26  Yao Qi  <yao@codesourcery.com>
 
        * configure.ac (build_warnings): Append '-Wempty-body'.

diff --git a/gdb/gdbserver/hostio.c b/gdb/gdbserver/hostio.c
index 72e334c414ce19ec5f042dda95575090d469c971..e89e100c6729ad95d7c8c12d00ca5591895002b2 100644 (file)
--- a/gdb/gdbserver/hostio.c
+++ b/gdb/gdbserver/hostio.c
@@ -483,7 +483,7 @@ handle_readlink (char *own_buf, int *new_packet_len)
       return;
     }
 
-  ret = readlink (filename, linkname, sizeof linkname);
+  ret = readlink (filename, linkname, sizeof (linkname) - 1);
   if (ret == -1)
     {
       hostio_error (own_buf);

diff --git a/gdb/inf-child.c b/gdb/inf-child.c
index ae2dd1e10128165ed92e1ec2107da0dd455c491f..3530e75768ec70510280be5cc6ef4c2cf187e4c1 100644 (file)
--- a/gdb/inf-child.c
+++ b/gdb/inf-child.c
@@ -346,7 +346,7 @@ inf_child_fileio_readlink (const char *filename, int *target_errno)
   /* We support readlink only on systems that also provide a compile-time
      maximum path length (MAXPATHLEN), at least for now.  */
 #if defined (HAVE_READLINK) && defined (MAXPATHLEN)
-  char buf[MAXPATHLEN];
+  char buf[MAXPATHLEN - 1];
   int len;
   char *ret;
 

diff --git a/gdb/linux-nat.c b/gdb/linux-nat.c
index 45f7e240600284c2c4b15436cbcce44f1d5320a4..f5ca97769dafac0d410fa9f9a5e20a28f562c8c3 100644 (file)
--- a/gdb/linux-nat.c
+++ b/gdb/linux-nat.c
@@ -4319,7 +4319,7 @@ linux_child_pid_to_exec_file (int pid)
   memset (name2, 0, MAXPATHLEN);
 
   sprintf (name1, "/proc/%d/exe", pid);
-  if (readlink (name1, name2, MAXPATHLEN) > 0)
+  if (readlink (name1, name2, MAXPATHLEN - 1) > 0)
     return name2;
   else
     return name1;

diff --git a/gdb/nbsd-nat.c b/gdb/nbsd-nat.c
index 14b562f140f84166ac06030427b1f9f08208feae..7f5df6611abaea6eb6719f218369796ada9739c9 100644 (file)
--- a/gdb/nbsd-nat.c
+++ b/gdb/nbsd-nat.c
@@ -34,7 +34,7 @@ nbsd_pid_to_exec_file (int pid)
   char *path;
 
   path = xstrprintf ("/proc/%d/exe", pid);
-  if (readlink (path, buf, MAXPATHLEN) == -1)
+  if (readlink (path, buf, MAXPATHLEN - 1) == -1)
     {
       xfree (buf);
       buf = NULL;