Created attachment 114521 [details] simple testcase to make NegOp crash Split off of bug 89387 Something is seriously wrong with the new MatOp handling, the attached document, calculating a mixed array of one number and one string using NegOp, makes the application immediately crash with a Fatal Error "Data array is too long". My assumption is it happens because the iterator of the result matrix is invalid because new segments are created when the error string result is to be inserted as element, but that's just a raw guess.
I'm marking this as blocker. If it can't be fixed we'll have to revert the MatOp commits of bug 89387. The same crash happens also in other constellations, e.g. value in A1, empty A2, and in B1:B2 have {=A1:A2/1} I played with MatOp to have only double returns and removing the SharedString specializations, but still it crashes, so the assumption about the mixed double/string result segments doesn't seem to hold.
Created attachment 114567 [details] crash AddOp in 4.4.0 Situation is even worse than I thought. Attached document crashes AddOp already in 4.4.0 with the same Fatal Error.
Markus Mohrhard committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=f9e38cbd3afdf0c8b2ebef0c4995e2443eac4d9a we need a more intelligent increment for the iterator, tdf#90391 It will be available in 4.5.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Markus Mohrhard committed a patch related to this issue. It has been pushed to "libreoffice-4-4": http://cgit.freedesktop.org/libreoffice/core/commit/?id=dd5e8ff4314a87cf86d473e21902df48668d1dd3&h=libreoffice-4-4 we need a more intelligent increment for the iterator, tdf#90391 It will be available in 4.4.3. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Xisco Fauli committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/5c532b42fb3f61257bf8edce4b3dcb21810f780e tdf#90391: sc_subsequent_filters: Add unittest It will be available in 7.4.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.