Issue 103588 - Chart in report cause crash
Summary: Chart in report cause crash
Status: CLOSED FIXED
Alias: None
Product: Base
Classification: Application
Component: ReportBuilder (show other issues)
Version: DEV300m52
Hardware: Unknown Windows XP
: P2 Trivial (vote)
Target Milestone: OOo 3.2
Assignee: r4zoli
QA Contact: issues@dba
URL:
Keywords: crash
Depends on:
Blocks:
 
Reported: 2009-07-16 14:27 UTC by r4zoli
Modified: 2009-09-08 17:20 UTC (History)
2 users (show)

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

Attachments
bugdoc (948.12 KB, application/vnd.sun.xml.base)
2009-07-16 14:33 UTC, r4zoli 		no flags Details
patch to correct the column count (1.22 KB, patch)
2009-07-23 09:48 UTC, IngridvdM 		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 r4zoli 2009-07-16 14:27:46 UTC 
- Open attached database.
- Open report Stock folder
- Open for edit Stock Type1 report
- Click on Chart area, select Data ranges open for edit.
- Select Data series
- Remove unnamed series from data series
- Edit remaining named Low (side remark, cannot rename this to the more
meaningful label)
- Set openvalues 2, close 5, high 4, low 3
- push OK
- Click out of chart
- Save report
- Runtime error! MS Visual C++ soffice.bin
- Push OK
- OOo Crash
- Restart OOo run file recovery, file recovered but usually no crash report 
only one time, error ID: r5fymkc
Comment 1 r4zoli 2009-07-16 14:33:17 UTC 
Created attachment 63586 [details]
bugdoc
Comment 2 Frank Schönheit 2009-07-20 14:19:58 UTC 
confirming, targeting, assigning, changing to P2, adding keyword
Comment 3 ocke.janssen 2009-07-23 07:47:12 UTC 
Could you please take a look at this. Thanks.
The crash occurs in SchXMLTableContext around line 1000

for( ++iRow, nRow = 0; iRow != rTable.aData.end(); iRow++, nRow++ )
        {
            aCategories[ nRow ] = (*iRow)[ 0 ].aString;
            sal_Int32 nTableColCount( static_cast< sal_Int32 >((*iRow).size()));
            for( nCol = 1; nCol < nTableColCount; nCol++ )
            {
                fVal = (*iRow)[ nCol ].fValue;
                if( bConstConvertNan &&
                    ::rtl::math::isNan( fVal ))
                    aData[ nRow ][ nCol - 1 ] = fNaN;
                else
                    aData[ nRow ][ nCol - 1 ] = fVal;
            }
aData[nRow] has only 3 children but is asked for the 4th
Comment 4 IngridvdM 2009-07-23 09:48:11 UTC 
Created attachment 63695 [details]
patch to correct the column count
Comment 5 IngridvdM 2009-07-23 09:49:50 UTC 
@Ocke, please try the attached patched.
Comment 6 ocke.janssen 2009-07-23 12:12:41 UTC 
patch works. thanks.
Comment 7 IngridvdM 2009-07-23 12:47:21 UTC 
Fixed in CWS chart40.
Comment 8 IngridvdM 2009-08-07 22:20:48 UTC 
Hm, after resync of the CWS this now crashes again. I have to look at this after
my vacation. Removing the issue from CWS chart40.
Comment 9 IngridvdM 2009-09-08 17:09:14 UTC 
Ok, this is fixed in dev300m57. Changes from chart40 are integrated and
obviously a second independent crash (statusbar updates?) was fixed also.
Comment 10 IngridvdM 2009-09-08 17:12:26 UTC 
@r4zoli, please verify that this issue is fixed in dev300m57 and close. Thanks!
Comment 11 r4zoli 2009-09-08 17:20:13 UTC 
Verified in DEV300_m57. 
Closing.