Issue 118760 - split the first table cell vertically,then undo&redo, the presentation app will crash
Summary: split the first table cell vertically,then undo&redo, the presentation app wi...
Status: CLOSED FIXED
Alias: None
Product: Impress
Classification: Application
Component: editing (show other issues)
Version: 3.4.0 Beta (OOo)
Hardware: PC Linux, all
: P3 Critical with 2 votes (vote)
Target Milestone: ---
Assignee: Tang Meng
QA Contact:
URL:
Keywords: crash
Depends on:
Blocks:
 
Reported: 2012-01-11 07:23 UTC by Li Feng Wang
Modified: 2012-03-27 02:56 UTC (History)
5 users (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: ---
jsc: 3.4_release_blocker+

Attachments
error screenshot (11.13 KB, image/pjpeg)
2012-03-14 03:16 UTC, Li Feng Wang 		no flags Details
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description Li Feng Wang 2012-01-11 07:23:45 UTC 
reproduce steps
1)create new table with row more than 5
2)merge first column,select all rows
3)split first cell to 5 cells Vertically 
4)undo and redo

presentation application will crash
Comment 1 Marcus 2012-01-11 20:20:26 UTC 
same behavior for me with OOo 3.3.0 on Linux
Comment 2 Pritesh Patel 2012-01-14 20:35:11 UTC 
I can also verify that this also happens on windows 7 64x
Comment 3 Li Feng Wang 2012-03-14 03:13:42 UTC 
reproduce on build r1296433

now it will pop up a error, then crash. the error screenshot is in attachment.
Comment 4 Li Feng Wang 2012-03-14 03:16:31 UTC 
Created attachment 77323 [details]
error screenshot
Comment 5 Oliver-Rainer Wittmann 2012-03-15 12:56:04 UTC 
I reproduce the crash also in OOo 3.1
Comment 6 Tang Meng 2012-03-16 06:29:00 UTC 
The undo remove columns algorithm is incorrect. All columns will insert the same cell. The incorrect code is in TableModel::UndoRemoveColumns(sal_Int32, ColumnVector&, CellVector&). The solution is to correct the algorithm, insert cells into column one by one.
Comment 7 Wang Lei 2012-03-16 08:09:29 UTC 
Submit code for Tang Meng.

Fix this issue with revision 1301361
Comment 8 Oliver-Rainer Wittmann 2012-03-16 08:38:57 UTC 
marking issue as fixed.
Comment 9 Li Feng Wang 2012-03-27 02:56:37 UTC 
Have no problem on build r1303653(developer snapshot)