diff options
| author | Carl Hetherington <cth@carlh.net> | 2021-11-30 00:00:17 +0100 |
|---|---|---|
| committer | Carl Hetherington <cth@carlh.net> | 2021-11-30 00:01:29 +0100 |
| commit | 01e979c79f7d0aa20fac1bb24c699e0636168294 (patch) | |
| tree | 4f2dd067b772969912dd79a41364e5a3a3d38544 /src | |
| parent | b51dca65c6a50063fa65b47a97e86f6e0dbbf911 (diff) | |
Be a little more careful to handle exceptions from boost::filesystem::file_size
There has been a report of file_size throwing an exception and
crashing DCP-o-matic, but I haven't been able to reproduce it yet
(possibly related to there being missing content files).
Diffstat (limited to 'src')
| -rw-r--r-- | src/lib/file_log.cc | 6 | ||||
| -rw-r--r-- | src/wx/config_dialog.cc | 2 |
2 files changed, 6 insertions, 2 deletions
diff --git a/src/lib/file_log.cc b/src/lib/file_log.cc index 5cc9c5569..4d6a0e6ea 100644 --- a/src/lib/file_log.cc +++ b/src/lib/file_log.cc @@ -69,7 +69,11 @@ FileLog::head_and_tail (int amount) const uintmax_t head_amount = amount; uintmax_t tail_amount = amount; - uintmax_t size = boost::filesystem::file_size (_file); + boost::system::error_code ec; + uintmax_t size = boost::filesystem::file_size (_file, ec); + if (size == static_cast<uintmax_t>(-1)) { + return ""; + } if (size < (head_amount + tail_amount)) { head_amount = size; diff --git a/src/wx/config_dialog.cc b/src/wx/config_dialog.cc index 53c67d101..e0effec53 100644 --- a/src/wx/config_dialog.cc +++ b/src/wx/config_dialog.cc @@ -633,7 +633,7 @@ CertificateChainEditor::import_private_key () chain->set_key (dcp::file_to_string (p)); _set (chain); update_private_key (); - } catch (dcp::MiscError& e) { + } catch (std::exception& e) { error_dialog (this, _("Could not read certificate file."), std_to_wx(e.what())); } } |