diff options
author | Pierre Muller <muller@sourceware.org> | 2012-12-14 23:27:13 +0000 |
---|---|---|
committer | Pierre Muller <muller@sourceware.org> | 2012-12-14 23:27:13 +0000 |
commit | b25d79d436158060439a2ba989acc84259f3a369 (patch) | |
tree | a554de20ef604d09b52adbf03f40ba6eaa678e0b /gdb/solib-target.c | |
parent | 774094928e3d322fc18bf8d8327cd665819a3796 (diff) | |
download | gdb-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.c | 10 |
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; |