diff options
| author | Siddhesh Poyarekar <siddhesh@sourceware.org> | 2012-07-06 05:36:07 +0000 |
|---|---|---|
| committer | Siddhesh Poyarekar <siddhesh@sourceware.org> | 2012-07-06 05:36:07 +0000 |
| commit | d5161074ccccf328ee055eee45b88284939b4fe2 (patch) | |
| tree | bba19e4695f13d82bc68beda30dc66cefcd2c5fd /gdb | |
| parent | a74bc5764616ecf97742696b679b8b917d522dc6 (diff) | |
| download | gdb-d5161074ccccf328ee055eee45b88284939b4fe2.zip gdb-d5161074ccccf328ee055eee45b88284939b4fe2.tar.gz gdb-d5161074ccccf328ee055eee45b88284939b4fe2.tar.bz2 | |
* cp-valprint.c (cp_print_value): Replace potentially unsafe
alloca with xmalloc/xfree.
Diffstat (limited to 'gdb')
| -rw-r--r-- | gdb/ChangeLog | 6 | ||||
| -rw-r--r-- | gdb/cp-valprint.c | 9 |
2 files changed, 12 insertions, 3 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 355c275..e432894 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,4 +1,10 @@ 2012-07-06 Siddhesh Poyarekar <siddhesh@redhat.com> + Jan Kratochvil <jan.kratochvil@redhat.com> + + * cp-valprint.c (cp_print_value): Replace potentially unsafe + alloca with xmalloc/xfree. + +2012-07-06 Siddhesh Poyarekar <siddhesh@redhat.com> * MAINTAINERS (Write After Approval): Add myself to the list. diff --git a/gdb/cp-valprint.c b/gdb/cp-valprint.c index 2e3beea..c066aa5 100644 --- a/gdb/cp-valprint.c +++ b/gdb/cp-valprint.c @@ -554,9 +554,11 @@ cp_print_value (struct type *type, struct type *real_type, if ((boffset + offset) < 0 || (boffset + offset) >= TYPE_LENGTH (real_type)) { - /* FIXME (alloca): unsafe if baseclass is really - really large. */ - gdb_byte *buf = alloca (TYPE_LENGTH (baseclass)); + gdb_byte *buf; + struct cleanup *back_to; + + buf = xmalloc (TYPE_LENGTH (baseclass)); + back_to = make_cleanup (xfree, buf); if (target_read_memory (address + boffset, buf, TYPE_LENGTH (baseclass)) != 0) @@ -568,6 +570,7 @@ cp_print_value (struct type *type, struct type *real_type, boffset = 0; thistype = baseclass; base_valaddr = value_contents_for_printing_const (base_val); + do_cleanups (back_to); } else { |