From: Hui Zhu <teawater@gmail.com>
Date: Wed, 15 Jun 2011 14:40:22 +0000 (+0000)
Subject: 2011-06-15  Hui Zhu  <teawater@gmail.com>
X-Git-Tag: sid-snapshot-20110701~198
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=c2fa21f135a1293e90ab268a9317d29faa3ac47c;p=platform%2Fupstream%2Fbinutils.git

2011-06-15  Hui Zhu  <teawater@gmail.com>

	* remote.c (remote_trace_set_readonly_regions): Add a check for
	target_buf_size.
---

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 982eafc..b40d0a6 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+2011-06-15  Hui Zhu  <teawater@gmail.com>
+
+	* remote.c (remote_trace_set_readonly_regions): Add a check for
+	target_buf_size.
+
 2011-06-14  Tom Tromey  <tromey@redhat.com>
 
 	* coffread.c (coffread_objfile): Rename from current_objfile.
diff --git a/gdb/remote.c b/gdb/remote.c
index b60a89c..e307986 100644
--- a/gdb/remote.c
+++ b/gdb/remote.c
@@ -9977,6 +9977,7 @@ remote_trace_set_readonly_regions (void)
   bfd_size_type size;
   bfd_vma vma;
   int anysecs = 0;
+  int offset = 0;
 
   if (!exec_bfd)
     return;			/* No information to give.  */
@@ -9985,6 +9986,7 @@ remote_trace_set_readonly_regions (void)
   for (s = exec_bfd->sections; s; s = s->next)
     {
       char tmp1[40], tmp2[40];
+      int sec_length;
 
       if ((s->flags & SEC_LOAD) == 0 ||
       /*  (s->flags & SEC_CODE) == 0 || */
@@ -9996,8 +9998,15 @@ remote_trace_set_readonly_regions (void)
       size = bfd_get_section_size (s);
       sprintf_vma (tmp1, vma);
       sprintf_vma (tmp2, vma + size);
-      sprintf (target_buf + strlen (target_buf), 
-	       ":%s,%s", tmp1, tmp2);
+      sec_length = 1 + strlen (tmp1) + 1 + strlen (tmp2);
+      if (offset + sec_length + 1 > target_buf_size)
+	{
+	  warning (_("\
+Too many sections for read-only sections definition packet."));
+	  break;
+	}
+      sprintf (target_buf + offset, ":%s,%s", tmp1, tmp2);
+      offset += sec_length;
     }
   if (anysecs)
     {