Issue 74134 - WordPrefect .wpd crashes on import
Summary: WordPrefect .wpd crashes on import
Status: CLOSED IRREPRODUCIBLE
Alias: None
Product: Writer
Classification: Application
Component: open-import (show other issues)
Version: OOo 2.2
Hardware: All Unix, all
: P2 Trivial with 1 vote (vote)
Target Milestone: ---
Assignee: aziem
QA Contact: issues@sw
URL: http://www.weddingstar.com/templates/...
Keywords: crash, oooqa
Depends on:
Blocks:
 
Reported: 2007-02-02 15:25 UTC by aziem
Modified: 2013-08-07 14:44 UTC (History)
6 users (show)

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


Attachments
http://www.weddingstar.com/templates/popups/flatpanel2a.wpd (6.40 KB, application/octet-stream)
2007-02-02 15:25 UTC, aziem
no flags Details
Stack from linux crash (13.29 KB, text/plain)
2007-02-13 17:08 UTC, stefan.baltzer
no flags Details

Note You need to log in before you can comment on or make changes to this issue.
Description aziem 2007-02-02 15:25:19 UTC
This document crashes OpenOffice.org OOF680_m5 (pre-2.2).
Comment 2 kpalagin 2007-02-03 05:29:07 UTC
Can't reproduce with m5 (build 9114) on WinXP - document opens without crash. 
Document appears to be empty.
Comment 3 aziem 2007-02-03 14:49:04 UTC
My test was on Fedora Core 5, Linux 2.6.18.
Comment 4 michael.ruess 2007-02-05 08:50:57 UTC
Reassigned to SBA.
Comment 5 kpalagin 2007-02-05 12:44:17 UTC
For me Writer just hungs (seemingly without CPU usage) with OO 9114 on Suse 
10.2 (KDE) when trying to import that file.
Comment 6 stefan.baltzer 2007-02-13 17:02:55 UTC
SBA: Added JW on C/C.
Opens empty on Windows XP in OOF680.m6.
Crashes on Linux (Suse 10.1) in Dev Build OOF680m7.
I sent a crash report naming this issue in the description.
I will attach the stack info from the recovery dialog.
Comment 7 stefan.baltzer 2007-02-13 17:08:08 UTC
Created attachment 42986 [details]
Stack from linux crash
Comment 8 stefan.baltzer 2007-02-13 17:12:34 UTC
SBA: Reassigned to fridrich_strba
Crashes on Solaris, too.
Set OS to "Unix, X11"
Comment 9 fridrich.strba 2007-02-13 18:08:25 UTC
OK, the synopsis is following: the document is utterly broken. wplook.exe, the
WP recovery and document analysis tool is not recognizing the file-format at
all. Although, it is (according to the header), a document in WP 6.1+
file-format. The libwpd corruption-removal mechanism is trying to skip all
inconsistent functions and consier only those that do not seem corrupted. This
is achieved by a dry-run parsing of the function before the sharp parsing.
Nevertheless, this is not possible with single byte functions, because one just
reads the byte and cannot check whether it is well-formed. Unfortunately, in WP6
file-format there are some table related functions that have a single byte
version. In this document, due to the corruption, we were trying to insert a row
into a table that was not started. This was resulting on some operation done on
null pointer and that has not defined behaviour. Therefore, I kind of fixed this
in libwpg CVS by ignoring table related functions if table is not opened. It
makes this document open, but the result is really not worth it (font of 500+
points and the document results in ~1000 pages (2 chars per page) of more or
less garbage.
I will fix this issue in a CWS that is uploading a bugfix release of libwpd
0.8.9. Changing the target for 2.2.1
Comment 10 fridrich.strba 2007-05-07 14:17:03 UTC
.
Comment 11 fridrich.strba 2007-08-23 20:29:41 UTC
In my version of upcoming 2.3 it does not crash although it does not produce
anything usable either given the corruption of the document.
Jack, could you please verify whether this works in a vanilla build?
Comment 12 thackert 2007-09-09 18:58:51 UTC
Hello fridrich_strba,
when I open the *wpd file in OOG m4, OOo crashes. If I use the program wpd2sxw
under Debian SID (AMD64 bit version) to convert the document to sxw without
error message. Afterwards I can open it in OOo. Maybe this helps :)
Thomas.
Comment 13 Mathias_Bauer 2007-12-04 13:53:43 UTC
Fridrich, we must find the right target for this issue. As I don't know your
workload I leave it up to you to choose between 2.4, 3.0 and 3.x.
Comment 14 fridrich.strba 2007-12-04 14:25:30 UTC
OK, libwpd 0.8.12 has a load of fixes. The only problem is that myself, I am
completely discouraged to do any CWS work ATM. So, if someone wants, this
version of library is working well in Go-oo and it is LGPL-ed. The thing would
be to upload it, make some QA guy to run the incantations on it and integrate.

The tarball is at http://libwpd.sf.net
Comment 15 Mathias_Bauer 2007-12-05 12:35:46 UTC
So you mean exchanging our version 0.8.8 with 0.8.12 will help?
What about the patches against 0.8.8 we currently have in cvs? Will the file
become obsolete or do we need to adjust it?
In case we would need to adjust the patch: would you be so kind to help us doing
that? 
I can organize everything else (the CWS handling) by myself.
Comment 16 fridrich.strba 2007-12-05 12:53:42 UTC
The patch will become useless and libwpd will even build with gcc 4.3 if the
tarball is 0.8.12
Comment 17 Mathias_Bauer 2007-12-05 16:16:03 UTC
Sorry, I don't know what the patch does. But if you say me that we don't need
the patch if we upgrade to 0.8.12 I'll take that for granted.
Comment 18 fridrich.strba 2007-12-05 16:23:09 UTC
OK, libwpd 0.8.8 and lower had an exploitable buffer overflow. I had a CWS where
I could put it without shouting too loud, because if I release libwpd before
OOo, iDefense would go public. So I did it in the patches. Is it enough background?
BTW: each release of libwpd is tested with more then 46000 documents for
regressions, I don't see any thorough Hamburg QA beat the results.
Comment 19 Mathias_Bauer 2007-12-05 17:48:09 UTC
Thanks, Fridrich. I didn't ask because I was in any doubt about the quality of
your release. I just wasn't sure what might be in the patch. Thanks for
clarifying that.
Comment 20 fridrich.strba 2007-12-08 14:02:12 UTC
Mathias, I would still wait for about a week or two with this one. With my QA
guy, we inflicted to each and every of the 46'000 documents a random corruption
using this tool: http://www.digitaldwarf.be/products/mangle.c
We discovered still that 5 of the corrupted documents were able to crash libwpd.
We fixed those crashes and we are instead terminating with an error. Although
the fix is already in, I am waiting for some thorough stress tests by my QA guy
before I release 0.8.13 so that we are completely sure that 0.8.13 will be the
most stable libwpd ever.
Comment 21 Mathias_Bauer 2007-12-08 15:47:21 UTC
Thanks for letting me know. And of course thanks for the great filter work also. 
Comment 22 Mathias_Bauer 2008-04-07 11:38:17 UTC
Any news on this one?
Comment 23 Martin Hollmichel 2008-06-06 14:59:55 UTC
any news on this ?
Comment 24 fridrich.strba 2008-06-06 16:35:48 UTC
As I said before, you are all free to take the freshest libwpd tarball and have
an up-to-date importer. But, as I also said before, I don't have any motivation
to do any upstream CWS/QA work on it. The freshest libwpd works well in
ooo-build and that is what I care about.
Comment 25 fridrich.strba 2008-06-06 16:36:15 UTC
.
Comment 26 Mathias_Bauer 2010-05-18 14:39:22 UTC
Whatever happened in the meantime, I couldn't reproduce the crash with OOo320m15.
Back to submitter for verification.
Comment 27 Pedro Giffuni 2012-03-15 05:37:42 UTC
This was solved by removing libwpd altogether.