diff options
| author | Maxime Coste <mawww@kakoune.org> | 2018-02-26 15:20:19 +1100 |
|---|---|---|
| committer | Maxime Coste <mawww@kakoune.org> | 2018-02-26 15:20:19 +1100 |
| commit | b4dc16010df1e04e1e5c619ebbb62819b05e415b (patch) | |
| tree | 65cb9b7106d936c90818b397e732a522e486a0c5 /src/parameters_parser.cc | |
| parent | 18f09028243aa3105fe86251825bce74721a94d0 (diff) | |
Fix out of bound read in ParameterParser::get_switch
The empty string case was not correctly handled.
Fixes #1883
Diffstat (limited to 'src/parameters_parser.cc')
| -rw-r--r-- | src/parameters_parser.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/parameters_parser.cc b/src/parameters_parser.cc index 30e2c346..56b1e1cf 100644 --- a/src/parameters_parser.cc +++ b/src/parameters_parser.cc @@ -59,7 +59,7 @@ Optional<StringView> ParametersParser::get_switch(StringView name) const for (size_t i = 0; i < m_params.size(); ++i) { const auto& param = m_params[i]; - if (param[0_byte] == '-' and param.substr(1_byte) == name) + if (param.substr(0_byte, 1_byte) == "-" and param.substr(1_byte) == name) return it->value.takes_arg ? m_params[i+1] : StringView{}; if (param == "--") |