diff options
author | Dave Korn <dave.korn@artimi.com> | 2010-10-15 16:09:36 +0000 |
---|---|---|
committer | Dave Korn <dave.korn@artimi.com> | 2010-10-15 16:09:36 +0000 |
commit | dc814f0f09ac2da6273b77183fbc66ceb3c9218f (patch) | |
tree | 32594d67c1341b0dc9feb8b32caf24b76a198d17 | |
parent | 37f1ae700118258507e1261d40e6afb6049b3f62 (diff) | |
download | binutils-redhat-dc814f0f09ac2da6273b77183fbc66ceb3c9218f.tar.gz |
Fix potential use-after-free bugs.
ld/ChangeLog:
* plugin.c (add_input_file): Take copy of input string.
(add_input_library): Likewise.
(set_extra_library_path): Likewise.
-rw-r--r-- | ld/ChangeLog | 6 | ||||
-rw-r--r-- | ld/plugin.c | 8 |
2 files changed, 11 insertions, 3 deletions
diff --git a/ld/ChangeLog b/ld/ChangeLog index d171298e81..85c784af45 100644 --- a/ld/ChangeLog +++ b/ld/ChangeLog @@ -1,3 +1,9 @@ +2010-10-15 Dave Korn <dave.korn.cygwin@gmail.com> + + * plugin.c (add_input_file): Take copy of input string. + (add_input_library): Likewise. + (set_extra_library_path): Likewise. + 2010-10-14 Dave Korn <dave.korn.cygwin@gmail.com> Apply LD plugin patch series (part 6/6). diff --git a/ld/plugin.c b/ld/plugin.c index b484bd045a..0c88ef8bea 100644 --- a/ld/plugin.c +++ b/ld/plugin.c @@ -433,7 +433,8 @@ static enum ld_plugin_status add_input_file (const char *pathname) { ASSERT (called_plugin); - if (!lang_add_input_file (pathname, lang_input_file_is_file_enum, NULL)) + if (!lang_add_input_file (xstrdup (pathname), lang_input_file_is_file_enum, + NULL)) return LDPS_ERR; return LDPS_OK; } @@ -443,7 +444,8 @@ static enum ld_plugin_status add_input_library (const char *pathname) { ASSERT (called_plugin); - if (!lang_add_input_file (pathname, lang_input_file_is_l_enum, NULL)) + if (!lang_add_input_file (xstrdup (pathname), lang_input_file_is_l_enum, + NULL)) return LDPS_ERR; return LDPS_OK; } @@ -454,7 +456,7 @@ static enum ld_plugin_status set_extra_library_path (const char *path) { ASSERT (called_plugin); - ldfile_add_library_path (path, FALSE); + ldfile_add_library_path (xstrdup (path), FALSE); return LDPS_OK; } |