RTF import: change exception type thrown on destination errors
commit0d84e4bed71a1083c5189408aea5922a34b40686
authorVasily Melenchuk <vasily.melenchuk@cib.de>
Wed, 4 Nov 2020 10:37:01 +0000 (4 13:37 +0300)
committerThorsten Behrens <Thorsten.Behrens@CIB.de>
Wed, 4 Nov 2020 13:38:54 +0000 (4 14:38 +0100)
tree248f0393e3bfe45aca64d1f1e43333c06e417899
parentbf91ed2d18ca0aae82c6a8a4e1498f6762ac3838
RTF import: change exception type thrown on destination errors

Previous exception (std::out_of_range) was not catch in case
of invalid RTF imported: wrapping code checks only for
uno::exception. This is not a case for loading file, but
for parsing of clipboard containing invalid RTF it was causing
fatal error, freeze and/or LO crash due to unhandled exception.

I think there is no reason to add generic processing for
std::exception in RTF filter: this can complicate diagnosing
other potential problems. Instead let's throw UNO exception,
like in other parts of RTF filter code.

Change-Id: I064bbdc1559cc7700c96dbbeaf2911a2c8e0421e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/105285
Tested-by: Jenkins
Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
writerfilter/source/rtftok/rtfdocumentimpl.hxx