aboutsummaryrefslogtreecommitdiff
path: root/gdb/solib-target.c
diff options
context:
space:
mode:
authorPierre Muller <muller@sourceware.org>2012-12-14 23:27:13 +0000
committerPierre Muller <muller@sourceware.org>2012-12-14 23:27:13 +0000
commitb25d79d436158060439a2ba989acc84259f3a369 (patch)
treea554de20ef604d09b52adbf03f40ba6eaa678e0b /gdb/solib-target.c
parent774094928e3d322fc18bf8d8327cd665819a3796 (diff)
downloadgdb-b25d79d436158060439a2ba989acc84259f3a369.zip
gdb-b25d79d436158060439a2ba989acc84259f3a369.tar.gz
gdb-b25d79d436158060439a2ba989acc84259f3a369.tar.bz2
* solib-target.c (solib_target_current_sos): Remove 'const'
qualifier from type of library_document local variable to be able to free it and avoid a memory leak. Use cleanup chain to avoid leak even if exceptino is generated.
Diffstat (limited to 'gdb/solib-target.c')
-rw-r--r--gdb/solib-target.c10
1 files changed, 9 insertions, 1 deletions
diff --git a/gdb/solib-target.c b/gdb/solib-target.c
index 0f3f850..54b4855 100644
--- a/gdb/solib-target.c
+++ b/gdb/solib-target.c
@@ -246,7 +246,8 @@ static struct so_list *
solib_target_current_sos (void)
{
struct so_list *new_solib, *start = NULL, *last = NULL;
- const char *library_document;
+ char *library_document;
+ struct cleanup *old_chain;
VEC(lm_info_p) *library_list;
struct lm_info *info;
int ix;
@@ -258,8 +259,15 @@ solib_target_current_sos (void)
if (library_document == NULL)
return NULL;
+ /* solib_target_parse_libraries may throw, so we use a cleanup. */
+ old_chain = make_cleanup (xfree, library_document);
+
/* Parse the list. */
library_list = solib_target_parse_libraries (library_document);
+
+ /* library_document string is not needed behind this point. */
+ do_cleanups (old_chain);
+
if (library_list == NULL)
return NULL;