aboutsummaryrefslogtreecommitdiff
path: root/gold/options.cc
diff options
context:
space:
mode:
authorBenjamin Peterson <bp@benjamin.pe>2017-12-01 12:59:36 -0800
committerCary Coutant <ccoutant@gmail.com>2017-12-01 12:59:36 -0800
commitbd2e3511f74940beec6a39914480ed6ff88c0a71 (patch)
tree4fd2c80329f90e8957276529424e4cd5fe634981 /gold/options.cc
parent71739b698d47945463f183619078ff680c277f0f (diff)
downloadfsf-binutils-gdb-bd2e3511f74940beec6a39914480ed6ff88c0a71.zip
fsf-binutils-gdb-bd2e3511f74940beec6a39914480ed6ff88c0a71.tar.gz
fsf-binutils-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.cc6
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();
}