[REVIEW 3.5 (and maybe 3.4?)] fdo#42771: Fix crash when loading an invalid .fodt
Michael Meeks
michael.meeks at suse.com
Mon Feb 13 03:59:44 PST 2012
On Fri, 2012-02-10 at 18:12 +0100, Jan Holesovsky wrote:
> http://cgit.freedesktop.org/libreoffice/core/commit/?id=9951de4a10386a85dc1bfa465304cd0005414b8e
Workaround looks fine to me :-) I'm amazed that we throw such a hard
exception here, but ...
> Returning a 'I was unable create a cursor' value via throwing runtime
> exception [in SwXFootnote::createTextCursorByRange()] is not really
> efficient ;-) - but I have no idea what I might break by changing that,
> so I just catch the exception in the code that already somehow counts on
> the possibility that the call fails anyway. This way, the filter fails
> nicely, with our "favorite" 'General Input/Output Error.'
You have my ack for -3-5 :-) any chance you can push it ?
Thanks,
Michael (feeling lame).
--
michael.meeks at suse.com <><, Pseudo Engineer, itinerant idiot
More information about the LibreOffice
mailing list