Bug 36863 - [Math] Crash updating formula
Summary: [Math] Crash updating formula
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Formula Editor (show other bugs)
Version:
(earliest affected)
3.4.0 Beta3
Hardware: x86 (IA32) All
: medium critical
Assignee: Michael Meeks
URL:
Whiteboard:
Keywords: regression
Depends on:
Blocks: mab3.4
  Show dependency tree / graph
 
Reported: 2011-05-05 00:30 UTC by C. Gagnon
Modified: 2011-05-07 04:58 UTC (History)
1 user (show)

See Also:
Crash report or crash signature:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description C. Gagnon 2011-05-05 00:30:18 UTC 
LibreOffice Math crashes every time I update (F9) a formula.
Comment 1 vitriol 2011-05-05 00:38:10 UTC 
Confirmed. Open Math, type a simple formula, press F9 > crash. Regression from 3.3.2
Comment 2 vitriol 2011-05-05 00:46:11 UTC 
Tested under Win7. Changing OS to All
Comment 3 Rainer Bielefeld Retired 2011-05-05 01:21:28 UTC 
[Reproducible] with "LibreOffice 3.4Beta3  – WIN7  Home Premium  (64bit) German UI [DEV300m103 (Build:3)]", it does not matter whether experimental formula editor or button formula cursor are active or not, <f9> crashes 100%

@Michael:
I saw you frequently active in Math bugs.
Please feel free to reassign if it's not your area!
Comment 4 Michael Meeks 2011-05-05 05:12:43 UTC 
The cockup is in the re-write to use STL types in starmath/source/parse.cxx - and the bug looks very local to the crash trace; just testing a fix:

#0  0xaf41c79a in SmParser::GetError (this=0x892e288, i=0) at /data/opt/libreoffice/bootstrap/starmath/source/parse.cxx:2520
#1  0xaf43902e in SmViewShell::ShowError (this=0x895ac70, pErrorDesc=0x0) at /data/opt/libreoffice/bootstrap/starmath/source/view.cxx:1465
#2  0xaf439c04 in SmViewShell::Execute (this=0x895ac70, rReq=...) at /data/opt/libreoffice/bootstrap/starmath/source/view.cxx:1622

(gdb) p m_nCurError
$1 = -1
(gdb) p i
$2 = 0
(gdb) p m_aErrDescList.size()
$3 = 0
Comment 5 Michael Meeks 2011-05-05 07:05:16 UTC 
fixed in libreoffice-3-4 - thanks for the report ! :-)
Comment 6 C. Gagnon 2011-05-07 04:49:36 UTC 
This problem is still present in LO 3.4.0 beta 4 for Mac.
Comment 7 vitriol 2011-05-07 04:58:35 UTC 
@C. Gagnon
I think that the patch is not yet integrated into Beta 4. Please, don't reopen a bug without confirmation of this! And don't change version tag with more recent version.