diff options
author | Eelco Dolstra <eelco.dolstra@logicblox.com> | 2017-12-15T10·39+0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-12-15T10·39+0100 |
commit | 1dffbff57d427da098bb750b5a86e57d7d0a401b (patch) | |
tree | c8a1b94b50b0a5c4da40c80e9b34cd22fa797703 /src/libexpr/json-to-value.cc | |
parent | 5d5b931fb178046ba286b8ef2b56a00b3a85c51c (diff) | |
parent | 626a94d70ea7a5c5b87f4d6061633c019ca8ea13 (diff) |
Merge pull request #1736 from bgamari/stoi-exceptions
Gracefully handle exceptions from stoi
Diffstat (limited to 'src/libexpr/json-to-value.cc')
-rw-r--r-- | src/libexpr/json-to-value.cc | 14 |
1 files changed, 10 insertions, 4 deletions
diff --git a/src/libexpr/json-to-value.cc b/src/libexpr/json-to-value.cc index 9380de3a66b3..8b1404595548 100644 --- a/src/libexpr/json-to-value.cc +++ b/src/libexpr/json-to-value.cc @@ -106,10 +106,16 @@ static void parseJSON(EvalState & state, const char * & s, Value & v) tmp_number += *s++; } - if (number_type == tFloat) - mkFloat(v, stod(tmp_number)); - else - mkInt(v, stoi(tmp_number)); + try { + if (number_type == tFloat) + mkFloat(v, stod(tmp_number)); + else + mkInt(v, stoi(tmp_number)); + } catch (std::invalid_argument e) { + throw JSONParseError("invalid JSON number"); + } catch (std::out_of_range e) { + throw JSONParseError("out-of-range JSON number"); + } } else if (strncmp(s, "true", 4) == 0) { |