diff options
| author | Hui Zhu <teawater@gmail.com> | 2011-06-15 14:40:22 +0000 |
|---|---|---|
| committer | Hui Zhu <teawater@gmail.com> | 2011-06-15 14:40:22 +0000 |
| commit | c2fa21f135a1293e90ab268a9317d29faa3ac47c (patch) | |
| tree | bbde2debfb671f5fd2632832beaa3145bc522d9f /gdb/remote.c | |
| parent | 90b3661c07d98d5ac5adaeddd67678c7234ead2a (diff) | |
| download | gdb-c2fa21f135a1293e90ab268a9317d29faa3ac47c.zip gdb-c2fa21f135a1293e90ab268a9317d29faa3ac47c.tar.gz gdb-c2fa21f135a1293e90ab268a9317d29faa3ac47c.tar.bz2 | |
2011-06-15 Hui Zhu <teawater@gmail.com>
* remote.c (remote_trace_set_readonly_regions): Add a check for
target_buf_size.
Diffstat (limited to 'gdb/remote.c')
| -rw-r--r-- | gdb/remote.c | 13 |
1 files changed, 11 insertions, 2 deletions
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) { |