diff options
| author | Andrew Burgess <andrew.burgess@embecosm.com> | 2016-08-11 16:04:59 +0100 |
|---|---|---|
| committer | Andrew Burgess <andrew.burgess@embecosm.com> | 2016-11-04 22:13:06 +0000 |
| commit | 848ac659685fba46ce8816400db705f60c8040f7 (patch) | |
| tree | 77ddadb777db7938f2e69a567a6baac396dc0914 | |
| parent | c42608e136ef1eb52f65002d5e7efe6d62b7f285 (diff) | |
| download | gdb-848ac659685fba46ce8816400db705f60c8040f7.zip gdb-848ac659685fba46ce8816400db705f60c8040f7.tar.gz gdb-848ac659685fba46ce8816400db705f60c8040f7.tar.bz2 | |
objcopy: Close some memory leaks in error paths
Closes some memory leaks within objcopy for error paths.
binutils/ChangeLog:
* objcopy.c (copy_section): Add extra calls to free for error
paths.
| -rw-r--r-- | binutils/ChangeLog | 5 | ||||
| -rw-r--r-- | binutils/objcopy.c | 3 |
2 files changed, 8 insertions, 0 deletions
diff --git a/binutils/ChangeLog b/binutils/ChangeLog index f1e1f9b..09f76b9 100644 --- a/binutils/ChangeLog +++ b/binutils/ChangeLog @@ -1,3 +1,8 @@ +2016-11-04 Andrew Burgess <andrew.burgess@embecosm.com> + + * objcopy.c (copy_section): Add extra calls to free for error + paths. + 2016-11-04 Tom Tromey <tom@tromey.com> * dwarf-mode.el (dwarf-browse): Set default-directory. Bump diff --git a/binutils/objcopy.c b/binutils/objcopy.c index 1821c0c..d40cfcb 100644 --- a/binutils/objcopy.c +++ b/binutils/objcopy.c @@ -3375,6 +3375,7 @@ copy_section (bfd *ibfd, sec_ptr isection, void *obfdarg) { status = 1; bfd_nonfatal_message (NULL, ibfd, isection, NULL); + free (memhunk); return; } @@ -3428,6 +3429,7 @@ copy_section (bfd *ibfd, sec_ptr isection, void *obfdarg) { status = 1; bfd_nonfatal_message (NULL, obfd, osection, NULL); + free (memhunk); return; } free (memhunk); @@ -3449,6 +3451,7 @@ copy_section (bfd *ibfd, sec_ptr isection, void *obfdarg) { status = 1; bfd_nonfatal_message (NULL, obfd, osection, NULL); + free (memhunk); return; } free (memhunk); |