diff options
author | Benjamin Peterson <bp@benjamin.pe> | 2017-12-01 12:59:36 -0800 |
---|---|---|
committer | Cary Coutant <ccoutant@gmail.com> | 2017-12-01 12:59:36 -0800 |
commit | bd2e3511f74940beec6a39914480ed6ff88c0a71 (patch) | |
tree | 4fd2c80329f90e8957276529424e4cd5fe634981 /gold/options.cc | |
parent | 71739b698d47945463f183619078ff680c277f0f (diff) | |
download | gdb-bd2e3511f74940beec6a39914480ed6ff88c0a71.zip gdb-bd2e3511f74940beec6a39914480ed6ff88c0a71.tar.gz gdb-bd2e3511f74940beec6a39914480ed6ff88c0a71.tar.bz2 |
Fix internal error from command line with unbalanced --start-lib/--end-lib.
The problem is that while the command line isn't trivially empty,
it contains no input files. As gold tries to configure the number
of threads to use based on the number of input files, this causes
the assertion failure above. Fix this problem by making the logic
in gold.cc more robust and also adding a better error message
about --start-lib to options.cc.
gold/
PR gold/22406
* gold.cc (queue_initial_tasks) Check for number of real input files.
* options.cc (Command_line::process) Check for unterminated --start-lib
options.
* testsuite/Makefile.am: Add new test script.
* testsuite/Makefile.in: Regenerate.
* testsuite/check_empty_command_lines.sh: New test script.
Diffstat (limited to 'gold/options.cc')
-rw-r--r-- | gold/options.cc | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/gold/options.cc b/gold/options.cc index ed63b6f..f54fb9b 100644 --- a/gold/options.cc +++ b/gold/options.cc @@ -1569,6 +1569,12 @@ Command_line::process(int argc, const char** argv) usage(); } + if (this->inputs_.in_lib()) + { + fprintf(stderr, _("%s: missing lib end\n"), program_name); + usage(); + } + // Normalize the options and ensure they don't contradict each other. this->options_.finalize(); } |