Issue 73906 - Saving XML files without table-number-list to binary format creates invalid files
Summary: Saving XML files without table-number-list to binary format creates invalid f...
Status: CLOSED FIXED
Alias: None
Product: General
Classification: Code
Component: chart (show other issues)
Version: 3.3.0 or older (OOo)
Hardware: All All
: P3 Trivial (vote)
Target Milestone: ---
Assignee: kla
QA Contact: issues@graphics
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-01-26 13:14 UTC by bjoern.milcke
Modified: 2013-02-24 21:20 UTC (History)
2 users (show)

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

Attachments
Spreadsheet document with chart containing no table-number-list (10.47 KB, application/vnd.oasis.opendocument.spreadsheet)
2007-01-26 13:16 UTC, bjoern.milcke 		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 bjoern.milcke 2007-01-26 13:14:07 UTC 
1. Load the attached ODF document (it contains no table-number-list)
2. Save as binary SDC document (don't activate the chart before saving)
=> The document contains "-1" as table index
3. Re-Load the file
=> The office may consume enormous amounts of memory, and eventually crash

Note, that the attached ODF document is valid. The files that are exported by
OOo 1-2 contain the illegal table-number-list, so they are not strictly valid.
Comment 1 bjoern.milcke 2007-01-26 13:16:51 UTC 
Created attachment 42491 [details]
Spreadsheet document with chart containing no table-number-list
Comment 2 bjoern.milcke 2007-01-26 14:31:18 UTC 
Instead of "-1" (in USHORT: 65535) the table number "0" is now used instead.
This is the first sheet which always exists.
Comment 3 Frank Schönheit 2007-03-06 21:04:45 UTC 
fs->bm: please verify that those changes made it into CWS basmgr02, and send the
issue to a QA representative of your choice.
Comment 4 bjoern.milcke 2007-03-07 10:25:52 UTC 
->TK: Please check with the attached file. Maybe some automatic tests with
binary charts also make sense. See CWS basmgr02 in cws02.
Comment 5 frank 2007-03-09 10:39:15 UTC 
found fixed on cws basmgr02. No crash or delay was discovered using this cws.
Comment 6 frank 2007-07-02 15:03:05 UTC 
found integrated on master m215 using Solaris, Linux and Windows