diff options
| author | Michael Snyder <msnyder@vmware.com> | 2007-08-08 18:01:35 +0000 |
|---|---|---|
| committer | Michael Snyder <msnyder@vmware.com> | 2007-08-08 18:01:35 +0000 |
| commit | decbce07bc550a328a79dbb2b7e2e662e35a9047 (patch) | |
| tree | 88e30e77f6a28cf6d5ea07c44cddb538cdc42544 /gdb/dwarf2read.c | |
| parent | 48219bd292371b412631b7ea1e10d2cc6223b0ad (diff) | |
| download | gdb-decbce07bc550a328a79dbb2b7e2e662e35a9047.zip gdb-decbce07bc550a328a79dbb2b7e2e662e35a9047.tar.gz gdb-decbce07bc550a328a79dbb2b7e2e662e35a9047.tar.bz2 | |
2007-08-08 Michael Snyder <msnyder@access-company.com>
* dwarf2read.c (add_partial_symbol): Memory leak.
Diffstat (limited to 'gdb/dwarf2read.c')
| -rw-r--r-- | gdb/dwarf2read.c | 16 |
1 files changed, 11 insertions, 5 deletions
diff --git a/gdb/dwarf2read.c b/gdb/dwarf2read.c index c12e3da..9cc1b56 100644 --- a/gdb/dwarf2read.c +++ b/gdb/dwarf2read.c @@ -1886,7 +1886,7 @@ add_partial_symbol (struct partial_die_info *pdi, struct dwarf2_cu *cu) { struct objfile *objfile = cu->objfile; CORE_ADDR addr = 0; - char *actual_name; + char *actual_name = NULL; const char *my_prefix; const struct partial_symbol *psym = NULL; CORE_ADDR baseaddr; @@ -1894,8 +1894,6 @@ add_partial_symbol (struct partial_die_info *pdi, struct dwarf2_cu *cu) baseaddr = ANOFFSET (objfile->section_offsets, SECT_OFF_TEXT (objfile)); - actual_name = NULL; - if (pdi_needs_namespace (pdi->tag)) { actual_name = partial_die_full_name (pdi, cu); @@ -1959,7 +1957,11 @@ add_partial_symbol (struct partial_die_info *pdi, struct dwarf2_cu *cu) { /* Static Variable. Skip symbols without location descriptors. */ if (pdi->locdesc == NULL) - return; + { + if (built_actual_name) + xfree (actual_name); + return; + } addr = decode_locdesc (pdi->locdesc, cu); /*prim_record_minimal_symbol (actual_name, addr + baseaddr, mst_file_data, objfile); */ @@ -1994,7 +1996,11 @@ add_partial_symbol (struct partial_die_info *pdi, struct dwarf2_cu *cu) union or class entry that does not have a byte size attribute and that has a DW_AT_declaration attribute." */ if (!pdi->has_byte_size && pdi->is_declaration) - return; + { + if (built_actual_name) + xfree (actual_name); + return; + } /* NOTE: carlton/2003-10-07: See comment in new_symbol about static vs. global. */ |