diff options
| author | Siddhesh Poyarekar <siddhesh@sourceware.org> | 2012-07-23 18:08:29 +0000 |
|---|---|---|
| committer | Siddhesh Poyarekar <siddhesh@sourceware.org> | 2012-07-23 18:08:29 +0000 |
| commit | 6c18f3e0f4d7e0399b39a0d554b5152af71a94e2 (patch) | |
| tree | 492cc1fb9f64c05b8723fcb0ede1a7d981f88bc6 /gdb/valops.c | |
| parent | 695c31738b1a1f052f777630bd914b7f3f783b9b (diff) | |
| download | binutils-6c18f3e0f4d7e0399b39a0d554b5152af71a94e2.zip binutils-6c18f3e0f4d7e0399b39a0d554b5152af71a94e2.tar.gz binutils-6c18f3e0f4d7e0399b39a0d554b5152af71a94e2.tar.bz2 | |
* p-valprint.c (pascal_object_print_value): Replace potentially
unsafe alloca with xmalloc/xfree.
* valops.c (search_struct_method): Likewise.
Diffstat (limited to 'gdb/valops.c')
| -rw-r--r-- | gdb/valops.c | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/gdb/valops.c b/gdb/valops.c index 97d889b..f6d8441 100644 --- a/gdb/valops.c +++ b/gdb/valops.c @@ -2281,8 +2281,13 @@ search_struct_method (const char *name, struct value **arg1p, if (offset < 0 || offset >= TYPE_LENGTH (type)) { - gdb_byte *tmp = alloca (TYPE_LENGTH (baseclass)); - CORE_ADDR address = value_address (*arg1p); + gdb_byte *tmp; + struct cleanup *back_to; + CORE_ADDR address; + + tmp = xmalloc (TYPE_LENGTH (baseclass)); + back_to = make_cleanup (xfree, tmp); + address = value_address (*arg1p); if (target_read_memory (address + offset, tmp, TYPE_LENGTH (baseclass)) != 0) @@ -2293,6 +2298,7 @@ search_struct_method (const char *name, struct value **arg1p, address + offset); base_valaddr = value_contents_for_printing (base_val); this_offset = 0; + do_cleanups (back_to); } else { |