Issue 111681 - sw: SwNumRule::SetInvalidRule crash
Summary: sw: SwNumRule::SetInvalidRule crash
Status: CLOSED FIXED
Alias: None
Product: Writer
Classification: Application
Component: code (show other issues)
Version: OOo 3.2
Hardware: All All
: P3 Trivial (vote)
Target Milestone: ---
Assignee: mst.ooo
QA Contact: issues@sw
URL:
Keywords:
: 106939 (view as issue list)
Depends on:
Blocks:
 
Reported: 2010-05-18 12:38 UTC by caolanm
Modified: 2017-05-20 11:41 UTC (History)
2 users (show)

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

Attachments
avoid crash (746 bytes, patch)
2010-05-18 12:38 UTC, caolanm 		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 caolanm 2010-05-18 12:38:16 UTC 
We have a reported crash at https://bugzilla.redhat.com/show_bug.cgi?id=591007
which indicates a NULL this for SwList::InvalidateListTree

We can't reproduce it though, so feel free to summarily close this out as a
WONTFIX, but as a heads up here's a last-ditch workaround patch
Comment 1 caolanm 2010-05-18 12:38:36 UTC 
Created attachment 69511 [details]
avoid crash
Comment 2 caolanm 2010-05-18 12:40:47 UTC 
*** Issue 106939 has been marked as a duplicate of this issue. ***
Comment 3 Oliver-Rainer Wittmann 2010-05-27 08:24:09 UTC 
Already back from sick leave, but still busy until the end of the week - I will
have a look next week.
Comment 4 Oliver-Rainer Wittmann 2010-06-04 09:03:36 UTC 
od->cmc:
I had a look a your patch. It will work and will avoid a crash at this code snippet.
But, if the list at which a certain text node is registered in does not exist,
further instabilities and crashes may occur.
Thus, I think we should integrate the applied patch. I will only adjust the
assertion a little bit.
Comment 5 Oliver-Rainer Wittmann 2010-06-04 09:11:03 UTC 
applied in cws sw33bf05 - changed file:
/sw/source/core/doc/number.cxx, change set 8ebd97a0c593
Comment 6 Oliver-Rainer Wittmann 2010-06-04 10:21:41 UTC 
applying the correct target
Comment 7 Oliver-Rainer Wittmann 2010-06-08 09:07:41 UTC 
od->mst: Please verify the integration of the patch in cws sw33bf05 - Thx.
Comment 8 mst.ooo 2010-06-08 15:27:01 UTC 
is applied