diff options
author | Roland Reichwein <mail@reichwein.it> | 2020-06-06 15:13:39 +0200 |
---|---|---|
committer | Roland Reichwein <mail@reichwein.it> | 2020-06-06 15:13:39 +0200 |
commit | 357cf76409d30341a2c4eedcf2568f0abd56e88d (patch) | |
tree | aa9a6121fa9e7827b854ec4c744d60c4f1375e50 /plugins | |
parent | c899a9cb581aa67be94231eba02f432a199512e7 (diff) |
More runtime error checking
Diffstat (limited to 'plugins')
-rw-r--r-- | plugins/statistics/statistics.cpp | 6 | ||||
-rw-r--r-- | plugins/weblog/weblog.cpp | 9 |
2 files changed, 13 insertions, 2 deletions
diff --git a/plugins/statistics/statistics.cpp b/plugins/statistics/statistics.cpp index 3ebd301..b1778f7 100644 --- a/plugins/statistics/statistics.cpp +++ b/plugins/statistics/statistics.cpp @@ -44,7 +44,11 @@ namespace { return 0; } - result += stoull(elements[column]); + try { + result += stoull(elements[column]); + } catch(...) { + std::cerr << "Error: Stats value " << elements[column] << " malformed." << std::endl; + } } return result; diff --git a/plugins/weblog/weblog.cpp b/plugins/weblog/weblog.cpp index 1e1b6b2..a64ee1b 100644 --- a/plugins/weblog/weblog.cpp +++ b/plugins/weblog/weblog.cpp @@ -244,6 +244,9 @@ namespace { size_t page) { try { + if (page > std::numeric_limits<int>::max()) + throw std::runtime_error("Bad page index: "s + std::to_string(page)); + HtmlPage htmlPage{GetRequestParam, "<h1>"s + GetRequestParam("WEBLOG_NAME") + "</h1>"s}; fs::path link{ GetRequestParam("plugin_path")}; @@ -422,7 +425,11 @@ std::string weblog_plugin::generate_page( size_t page {0}; auto it {query.find("page")}; if (it != query.end()) { - page = stoul(it->second); + try { + page = stoul(it->second); + } catch(...) { + // ignore: keep default 0 + } } if (is_index_page(rel_target)) |