Issue 99564 - WW8: Certain documents with table causes crash during import
Summary: WW8: Certain documents with table causes crash during import
Status: CLOSED FIXED
Alias: None
Product: Writer
Classification: Application
Component: open-import (show other issues)
Version: OOo 3.0.1
Hardware: Unknown All
: P2 Trivial (vote)
Target Milestone: ---
Assignee: michael.ruess
QA Contact: issues@sw
URL: https://bugs.launchpad.net/bugs/332943
Keywords: regression
: 99582 102227 (view as issue list)
Depends on:
Blocks: 95768
  Show dependency tree
 
Reported: 2009-02-24 05:29 UTC by ccheney
Modified: 2013-08-07 14:43 UTC (History)
3 users (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: ---

Attachments
simplied test case (27.50 KB, application/msword)
2009-03-11 17:09 UTC, openoffice 		no flags Details
proposed patch (1.51 KB, patch)
2009-03-11 17:26 UTC, openoffice 		no flags Details | Diff
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 ccheney 2009-02-24 05:29:44 UTC 
I got a single bug report in Ubuntu that the following three doc files cause OOo
to crash. I verified the crashes on official OOo. Feel free to split this bug if
it isn't actually the same issue with all three documents.

http://launchpadlibrarian.net/22981365/CV-Europass.doc

http://launchpadlibrarian.net/22981369/CV-Landricina.doc

http://launchpadlibrarian.net/22981376/CV-ML.doc
Comment 1 michael.ruess 2009-02-24 15:30:46 UTC 
MRU->HBRINKM: all three documents seem to cause the same crash during import of
a certain complex table.
Import was fine until OOo 3.0.
Comment 2 hagar_de_lest 2009-03-09 13:57:32 UTC 
How can you set the target to 3.2???? This is just a showstopper for anyone
having to export .doc files with tables inside!!! Threads are multiplying in the
forums, even with simple tables!
Comment 3 vitriol 2009-03-11 11:34:33 UTC 
I's a repression from version 3.0. Should be a showstopper, for me.
Comment 4 kpalagin 2009-03-11 12:14:12 UTC 
Please reconsider 3.2.
Comment 5 bodrisch 2009-03-11 16:54:26 UTC 
It's very big problem, and I hope it will be solved in 3.1. It's imposible to 
work with .doc files with tables inside!
Comment 6 openoffice 2009-03-11 17:06:58 UTC 
analysis: Introduced by an issue solved in hb11 that prevents an empty paragraph imported after a nested 
table.

Precondition for the crash to happen: Section break right after top level table.
Symptom: SwDoc::AppendTxtNode tries to insert text node after end of document => crash
Comment 7 openoffice 2009-03-11 17:09:07 UTC 
Created attachment 60889 [details]
simplied test case
Comment 8 openoffice 2009-03-11 17:26:18 UTC 
Created attachment 60891 [details]
proposed patch
Comment 9 canis 2009-03-11 18:27:57 UTC 
Why 3.2? Is it mean that I'll have to wait till 09.2009 to begin use OOo again???
As this issue is a regression, please, fix it in nearest release!
Comment 10 stefan.baltzer 2009-03-11 18:31:25 UTC 
SBA: Adjusting summary to reflect the findings: The problem is reproducible in
very simple documents, as bugdoc shows. Added keyword "regression".
Given the risk (easy code fix, according to hbrinkm) and the impact (Crash with
easy documents) I advocate a fix for 3.1 and wrote so on Releases Mail Alias.
Comment 11 vitriol 2009-03-12 09:05:07 UTC 
Is issue 99582 a duplicate of this issue?
Comment 12 openoffice 2009-03-12 10:37:16 UTC 
Sending        inc/pam.hxx
Sending        source/core/crsr/pam.cxx
Sending        source/filter/ww8/ww8par2.cxx
Committed revision 269369.
Comment 13 openoffice 2009-03-12 10:49:06 UTC 
adjusted target
Comment 14 openoffice 2009-03-13 11:38:04 UTC 
ready for QA
Comment 15 michael.ruess 2009-03-16 11:32:43 UTC 
Verified in CWS fwk105.
Comment 16 openoffice 2009-03-20 11:55:46 UTC 
*** Issue 99582 has been marked as a duplicate of this issue. ***
Comment 17 sgautier.ooo 2009-03-28 11:35:14 UTC 
Verified in OOO310_m7 .deb version, none of the 4 test docs crashed. Closing -
Sophie
Comment 18 michael.ruess 2009-05-25 16:04:47 UTC 
*** Issue 102227 has been marked as a duplicate of this issue. ***