Issue 6596 - Crash when trying to open a document
Summary: Crash when trying to open a document
Status: CLOSED FIXED
Alias: None
Product: Writer
Classification: Application
Component: ui (show other issues)
Version: OOo 1.0.0
Hardware: PC Windows 2000
: P1 (highest) Trivial (vote)
Target Milestone: ---
Assignee: michael.ruess
QA Contact: issues@sw
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2002-07-26 10:37 UTC by aaime
Modified: 2013-08-07 14:43 UTC (History)
1 user (show)

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


Attachments
The file that makes OOwriter crash (compressed) (254.96 KB, application/x-zip-compressed)
2002-07-26 10:42 UTC, aaime
no flags Details
little example (140.50 KB, application/octet-stream)
2003-02-10 10:46 UTC, caolanm
no flags Details
overlapping example (136.00 KB, application/octet-stream)
2003-02-11 08:43 UTC, caolanm
no flags Details

Note You need to log in before you can comment on or make changes to this issue.
Description aaime 2002-07-26 10:37:43 UTC
I've got a document saved with Office2000 and fast save
option enabled that makes OpenOffice 1.0 and 1.0.1 crash
during loading. The same document saved with "save as" under
office, thus without fast save, is opened properly.
Comment 1 aaime 2002-07-26 10:42:15 UTC
Created attachment 2365 [details]
The file that makes OOwriter crash (compressed)
Comment 2 aaime 2002-07-26 10:45:18 UTC
Sorry, it seems I'm wrong, the file always
makes OOwriter crash, with or without fast save.
Comment 3 stx123 2002-09-24 00:10:18 UTC
moving to component word processor
Comment 4 stefan.baltzer 2002-12-13 14:33:51 UTC
Reassigned to Michael.
Comment 5 michael.ruess 2002-12-16 11:16:50 UTC
MRU->DVO: doc seems to loop after import. The debug version throws
tons of the Assertion "redline table corrupted: overlapping redlines"

SW644MI! AssertFail(char const *,char const *,unsigned short) + 258 bytes
SW644MI! lcl_CheckRedline(class SwDoc const *) + 255 bytes
SW644MI! SwDoc::AppendRedline(class SwRedline *,unsigned char) + 6766
bytes
SW644MI! SetInDocAndDelete::operator()(class SwFltStackEntry *) + 237
bytes
SW644MI! SwWW8ImplReader::LoadDoc1(class SwPaM &,class WW8Glossary *)
+ 4442 bytes
SW644MI! WW8Reader::Read(class SwDoc &,class SwPaM &,class String
const &) + 337 bytes
SW644MI! SwReader::Read(class Reader const &) + 709 bytes
SW644MI! SwDocShell::ConvertFrom(class SfxMedium &) + 447 bytes
SFX644MI! SfxObjectShell::DoLoad(class SfxMedium *) + 3480 bytes
Comment 6 openoffice 2003-01-27 15:07:28 UTC
dvo->cmc: "The wheels have fallen off revision mark import"

There are several assertions when loading the bugdoc (in SRX644d), the
'wheels' one and one complaining about the date being 0.

Later on, you will find (my) assertions about the redline table being
corrupted, but I assume they are consequences of the former. If not,
send the bug back to me once you're done with it.

Note: The bugdoc is a zipped .doc file.
Comment 7 caolanm 2003-02-10 10:46:11 UTC
Created attachment 4628 [details]
little example
Comment 8 caolanm 2003-02-10 10:47:11 UTC
The little attachment shows that word does have changes that have both
no author and no time/date of change. So the asserts are reporting an
unexpected event that is truly the case. 
Comment 9 caolanm 2003-02-11 08:42:45 UTC
And as for the core asserts of overlapping revisions, there actually
*are* overlapping revisions here. There are ranges of text which have
both delete and append properties in use, i.e. text that claims to be
deleted before it is added :-)
Comment 10 caolanm 2003-02-11 08:43:15 UTC
Created attachment 4644 [details]
overlapping example
Comment 11 caolanm 2003-02-13 10:45:17 UTC
OK, the no 1 concern is crashing. And crashes can be done away with,
so we'll do that here.

My asserts about the redline having no author and no date of change
are actually correct behaviour in the filter, not logic gone awry. The
reality is that some of the redlines have an "unknown" author and
display no time of change (happens I think sometimes when redlines are
imported from word95/6 to 97/2000/xp), so I must remove those asserts.
I can also check for totally invalid empty redlines, which I have done,
so will be no crash in apps01 workspace for 1.1 Beta.

But there will still be complaints of overlapping redlines, this is
because they are truly overlapping in the winword file:-(, and that
requires some deeper thinking, see issue 11450 for tracking the
overlap issue.
Comment 12 caolanm 2003-02-19 10:56:06 UTC
reopen to reassign
Comment 13 caolanm 2003-02-19 11:00:46 UTC
cmc->mru: It will assert in a nonproduct version because of issue
11450. But it shouldn't crash.
Comment 14 michael.ruess 2003-02-20 11:43:47 UTC
Checked the fix with internal build CWS_apps01.
Comment 15 michael.ruess 2003-02-20 11:45:11 UTC
Fix will be available with OpenOffice 1.1 Beta.
Comment 16 michael.ruess 2003-02-22 13:25:50 UTC
Checked integration with internal Milestone srx644m4s3