Apache OpenOffice (AOO) Bugzilla – Full Text Issue Listing |
Summary: | ~EntryList_Impl() trashing heap? | ||||||
---|---|---|---|---|---|---|---|
Product: | General | Reporter: | dankegel <dank> | ||||
Component: | code | Assignee: | fa <dan> | ||||
Status: | CLOSED FIXED | QA Contact: | issues@framework <issues> | ||||
Severity: | Trivial | ||||||
Priority: | P3 | CC: | dan, issues, jens-heiner.rechtien, matthias.huetsch, pavel | ||||
Version: | OOo 1.1 RC2 | Keywords: | crash, oooqa | ||||
Target Milestone: | OOo 1.1.1 | ||||||
Hardware: | PC | ||||||
OS: | Linux, all | ||||||
Issue Type: | PATCH | Latest Confirmation in: | --- | ||||
Developer Difficulty: | --- | ||||||
Attachments: |
|
Description
dankegel
2003-07-31 17:58:12 UTC
Adding keyword "crash" because this is the kind of bug that is likely to cause a crash on some systems. adding cc: sewardj. I set target to 1.1.1 though I'd much prefer to see this fixed in 1.1, as I have a feeling the OpenOffice folks do not yet take this kind of error seriously enough to try to fix it in the weeks remaining before 1.1. fixed summary... Hi Mathias, Kay Ramme told me that you're volunteering to take care of this kind of tasks. Handing it to you... . Resetting resolution, because there is no yet. Even though the stack provided in the attachment to issue 8492 looks strange at first glance (call from ~GroupData_Impl() in sfx to ~EntryList_Impl in svtools), this may well be true (and wrong) and thus valgrind's complaints may be correct. As shown below, libsvt...so does indeed export a *global* symbol "__1cOEntryList_Impl2T6M_v_", ~EntryList_Impl(), while libsfx...so defines two *local* symbols of the same name. AFAIK, the runtime linkers symbol resolution rules will satisfy all references to ~EntryList_Impl() in sfx with the global definition of this symbol in svtools, and not with the local definitions within sfx. .../svtools 238 % nm -D unxsols4.pro/lib/libsvt645ss.so | grep EntryList_Impl [1338] | 4176280| 32|FUNC |GLOB |0 |8 |__1cOEntryList_Impl2T5B6M_v_ [13680] | 4176180| 48|FUNC |GLOB |0 |8 |__1cOEntryList_Impl2t5B6MpnWSvxIconChoiceCtrl_Impl_HH_v_ [2452] | 4176228| 52|FUNC |GLOB |0 |8 |__1cOEntryList_Impl2t5B6MpnWSvxIconChoiceCtrl_Impl_HHH_v_ [2723] | 4176280| 32|FUNC |GLOB |0 |8 |__1cOEntryList_Impl2T6M_v_ [9962] | 4176180| 48|FUNC |GLOB |0 |8 |__1cOEntryList_Impl2t6MpnWSvxIconChoiceCtrl_Impl_HH_v_ [17492] | 4176228| 52|FUNC |GLOB |0 |8 |__1cOEntryList_Impl2t6MpnWSvxIconChoiceCtrl_Impl_HHH_v_ [16151] | 4176312| 32|FUNC |GLOB |0 |8 |__1cOEntryList_ImplFClear6M_v_ [8326] | 4176344| 76|FUNC |GLOB |0 |8 |__1cOEntryList_ImplGInsert6MpnWSvxIconChoiceCtrlEntry_L_v_ [1793] | 4176420| 40|FUNC |GLOB |0 |8 |__1cOEntryList_ImplGRemove6ML_pnWSvxIconChoiceCtrlEntry__ [5942] | 4176460| 48|FUNC |GLOB |0 |8 |__1cOEntryList_ImplGRemove6MpnWSvxIconChoiceCtrlEntry__v_ [12838] | 4176508| 100|FUNC |GLOB |0 |8 |__1cOEntryList_ImplMRemoved_Impl6MpnWSvxIconChoiceCtrlEntry__v_ .../svtools 240 % nm ../sfx2/unxsols4.pro/lib/libsfx645ss.so | grep EntryList_Impl [5203] | 2390488| 20|FUNC |LOCL |0 |8 |__1cOEntryList_Impl2T6M_v_ [5009] | 2340624| 20|FUNC |LOCL |0 |8 |__1cOEntryList_Impl2T6M_v_ .../svtools 241 % cd ../sfx2/ .../sfx2 242 % find . -name "*.?xx" -exec grep EntryList_Impl {} /dev/null \; ./source/doc/doctemplates.cxx:DECLARE_LIST( EntryList_Impl, EntryData_Impl* ); ./source/doc/doctemplates.cxx: EntryList_Impl maEntries; ./source/doc/doctempl.cxx:DECLARE_LIST( EntryList_Impl, EntryData_Impl* ); ./source/doc/doctempl.cxx: EntryList_Impl maEntries; That's what I have so far. Heiner, can you please have a look into this (during your other valgrind cleanup work)? Thanks, Matthias adding myself to CC... . Removed nonsense dependency (issue 21786 depending on this one). move relaease target to OOo-2.0 For whatever reason, this is causing the OS X linker to bail out on sfx2 module on OS X. We need this fixed now and now in 2.0. Patch attached, renames the distinct uses of EntryList_Impl in sfx2 to be specific to their particular file (doctmpl.cxx, doctemplates.cxx). Created attachment 12919 [details]
cd sfx2, patch -p0 < /path/to/patchfile Cleans up EntryList_Impl naming & usage
changing to Patch, and targetting for 1.1.1 mh->mba: are you able to review ? Patch is approved. I'm waiting for a CWS now. You can commit the patch in the next ooofix CWS. Fixed in CWS ooo111regression Verified. Closed. |