On Sun, Nov 19, 2017 at 8:08 AM, Jim Jagielski <j...@jagunet.com> wrote:
> I think I will spend some time creating some 4.1.5-dev builds for people to > play with... > > > On Nov 19, 2017, at 10:40 AM, Patricia Shanahan <p...@acm.org> wrote: > > > > Jim, > That would be great. :) > > > I am going to be away from computers for most of the day. If you have > everything you want tested in 4.1.5 checked in, I would like you to trigger > a request for general testing. > > > > In particular, anyone who is working a bug report that is a regression > between 4.1.3 and 4.1.4 should test it with the 4.1.5 branch, regardless of > the symptoms. > > > > Originally, I expected my null point bug to manifest as an access > violation, as is the case for the original report. I was monitoring > bugzilla for any similar reports. From dev@ discussion, I have learned > there are instances of catching and hiding all exceptions. That means data > structures could be left in an inconsistent state without AOO crashing or > the access violation being reported, and almost any symptom could result. > > > > On 11/19/2017 7:21 AM, Patricia Shanahan wrote: > >> The null check inside the mAutoAskUpdateAllLinks test is something of > an over-reaction on my part, in the direction of defensive programming. > mAutoAskUpdateAllLinks is only true in two specific cases, both of which I > have reviewed. The bug was doing the parent look-up outside that if block. > >> I have tested forcing the parent to null in a debugger. It does not > crash. I suspect it would not necessarily pick the right parent window if > multiple windows are open. > >> On 11/19/2017 7:11 AM, Jim Jagielski wrote: > >>> Just to be clear: > >>> > >>> SetUserAllowsLinkUpdate(pLink, GetUserAllowsLinkUpdate(parent)); > >>> > >>> is OK if parent == NULL? That is, does GetUserAllowsLinkUpdate() > >>> work OK if passed NULL? Or, if parent is NULL, should we skip > >>> calling SetUserAllowsLinkUpdate()? > >>> > >>>> On Nov 18, 2017, at 3:49 PM, Patricia Shanahan <p...@acm.org> wrote: > >>>> > >>>> I'm attaching the patch in case anyone else wants to build their own > test version. > >>>> > >>>> On 11/18/2017 11:11 AM, Patricia Shanahan wrote: > >>>>> There is a bug in my LinkManager changes. Specifically, I made the > totally newbie mistake of accessing memory based on a pointer without > allowing for it being null, and it turns out it can be null :-( > >>>>> The primary report for this is https://bz.apache.org/ooo/ > show_bug.cgi?id=127581, "Writer crashes after copying all content". > >>>>> I would have expected the failure to be some form of access > violation. However, recent e-mails have convinced me that some paths > include catch blocks that conceal all exceptions, so it may be the root > cause of other problems. > >>>>> I have checked a fix into the trunk, revision 1814994. However, > trunk has other changes. > >>>>> The simplest way to check whether other bug reports are due to this > is to build a version that is identical to 4.1.4 except for the one fix. > Anything that works with that but not with 4.1.4 is a duplicate. Of course, > the more duplicates we have the worse the problem and the more urgent a fix. > >>>>> I am going ahead and doing the fix in my 4.1.5 working copy, but > activating 4.1.5 is a broader decision. > >>>>> ------------------------------------------------------------ > --------- > >>>>> To unsubscribe, e-mail: dev-unsubscr...@openoffice.apache.org > >>>>> For additional commands, e-mail: dev-h...@openoffice.apache.org > >>>> <null_pointer.patch> > >>>> --------------------------------------------------------------------- > >>>> To unsubscribe, e-mail: dev-unsubscr...@openoffice.apache.org > >>>> For additional commands, e-mail: dev-h...@openoffice.apache.org > >>> > >>> > >>> --------------------------------------------------------------------- > >>> To unsubscribe, e-mail: dev-unsubscr...@openoffice.apache.org > >>> For additional commands, e-mail: dev-h...@openoffice.apache.org > >>> > >> --------------------------------------------------------------------- > >> To unsubscribe, e-mail: dev-unsubscr...@openoffice.apache.org > >> For additional commands, e-mail: dev-h...@openoffice.apache.org > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: dev-unsubscr...@openoffice.apache.org > > For additional commands, e-mail: dev-h...@openoffice.apache.org > > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@openoffice.apache.org > For additional commands, e-mail: dev-h...@openoffice.apache.org > > -- ---------------------------------------------------------------------- MzK "Only the truth will save you now." -- Ensei Tankado, "Digital Fortress"