Issue 22440

Summary: setup segfaults from SiBasic::Error() when -nogui passed in
Product: Installation Reporter: fa <dan>
Component: codeAssignee: Olaf Felka <olaf-openoffice>
Status: CLOSED FIXED QA Contact: issues@installation <issues>
Severity: Trivial    
Priority: P3 CC: issues
Version: OOo 1.1 RC5   
Target Milestone: OOo 1.1.1   
Hardware: All   
OS: Linux, all   
Issue Type: PATCH Latest Confirmation in: ---
Developer Difficulty: ---
Attachments:
Description Flags
cd setup2, patch -p1 < /path/to/patchfile none

Description fa 2003-11-13 20:45:11 UTC
SiBasic::Error() does not check m_rEnv.HasVCL() (which returns false if -nogui
is passed into setup from the command line) before running off to call InfoBox,
which uses the VCL.  Well, since -nogui was passed the VCL is not inited, and
this causes setup to segfault in SalFrameData::Init().

Attached patch checks m_rEnv.HasVCL() and if TRUE, calls InfoBox, but if FALSE,
prints the error out to stderr.
Comment 1 fa 2003-11-13 20:46:06 UTC
Created attachment 11243 [details]
cd setup2, patch -p1 < /path/to/patchfile
Comment 2 fa 2003-11-13 21:13:34 UTC
committed to cws_srx645_ooo111fix1, approved by Nesshof on IRC
Comment 3 Olaf Felka 2003-11-14 08:51:34 UTC
Hi Dirk,
is this something you should give some attention?
Comment 4 Martin Hollmichel 2003-11-18 12:33:00 UTC
setting to fixed, as already committed to ooo111fix1
Comment 5 dirk.voelzke 2004-02-05 09:18:23 UTC
Please verify.
Comment 6 fa 2004-02-08 02:31:07 UTC
Verified.  Problem fixed.
Comment 7 Olaf Felka 2004-03-18 15:11:13 UTC
Fixed.