Steps: 1) Open writer with screen reader on 2) right-click > Character 3) Turn off screen reader 4) Switch tab 5) Close dialog 6) Turn on screen reader 7) Crash Crashes with git build but not TDF build.
Created attachment 127141 [details] backtrace This was on Ubuntu Mate 16.04 with orca 3.18.2. Version: 5.3.0.0.alpha0+ Build ID: d5fab973d0af95c433c5f6a9492014f7db642489 CPU Threads: 4; OS Version: Linux 4.4; UI Render: default; TinderBox: Linux-rpm_deb-x86_64@70-TDF, Branch:master, Time: 2016-08-24_00:37:02 Locale: en-US (en_US.UTF-8); Calc: group
Can not confirm on Windows 10 Pro 64-bit en-US with NVDA 2016.2.1 and Version: 5.2.1.2 (x64) Build ID: 31dd62db80d4e60af04904455ec9c9219178d620 CPU Threads: 8; OS Version: Windows 6.19; UI Render: default; Locale: en-US (en_US); Calc: group nor with recent build of master Version: 5.3.0.0.alpha0+ Build ID: 696e83b663d4f3e00f23947613f9f3916a4dd14d CPU Threads: 8; OS Version: Windows 6.19; UI Render: GL; TinderBox: Win-x86@42, Branch:master, Time: 2016-09-03_10:21:32 Locale: en-US (en_US); Calc: CL
What do you mean by "Crashes with git build but not TDF build"? Isn't the one in comment 1 a TDF build? I also tried with a TDF build, no crash with the steps from description: Version: 5.3.0.0.alpha0+ Build ID: 4a63c145dcce8411c5707f6b99877cc87a4f6c5d CPU Threads: 1; OS Version: Linux 4.4; UI Render: default; TinderBox: Linux-rpm_deb-x86_64@70-TDF, Branch:master, Time: 2016-08-31_19:54:16 Locale: en-US (en_US.UTF-8); Calc: group
(In reply to Aron Budea from comment #3) > What do you mean by "Crashes with git build but not TDF build"? LO built from git and not a TDF daily build. > Isn't the one in comment 1 a TDF build? No that was my mistake, that was the build info for master that i wasnt able to crash. @Julien: Can you test your master build? Backtrace is available, so we need a dev to look into it.
On pc Debian x86-64 with master sources updated today, I don't reproduce this. However, I don't use merge libs.
'needsConfirmationAdvise' is only used for unconfirmed bugs. Removing it from this bug. [NinjaEdit]
Dear Jay, On TDF LibreOfficeDev 5.4 and the Debian 5.2.5 package I cannot reproduce the crash with the steps you mentioned. Are you still able to reproduce this crash? Best regards.
Closing as RESOLVED WORKSFORME as per comment 7
(In reply to Xisco Faulí from comment #8) > Closing as RESOLVED WORKSFORME as per comment 7 Comment 7 is for a TDF build and i mentioned it crashed in a GIT build.
I gave a try with this: https://gerrit.libreoffice.org/#/c/37575/ I'm waiting for Jenkins TBs and push it if ok. Then I'll wait for your feedback before cherry-picking it on 5.3 branch Even if this patch solves your pb, I think it's more a band aid than a real fix. Indeed, accessibility part seems a bit messy. Sometimes, functions return 0, others -1, some throw Runtime exception, ... It'd be great to see more people fixing buggy things instead of adding brand new exciting things but quite uncomplete or buggy too.
Julien Nabet committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=c9275062ed745a8c5d52a3863299baf8bcf86ea7 tdf#101885: fix crash in AccFrameSelector It will be available in 5.4.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Looks like a sensible patch to me; I'd be inclined to return -1 or some other OOB data for when we have no parent (no idea what we do elsewhere for that). Either way fix looks sensible enough for me; a11y deals with lots of corner-cases like this, we can't control how long people keep handles for accessible peers - often long after the object has been destroyed.
Had forgotten about this one. Anyone, I cherry-picked it for review on 5.3 branch (see https://gerrit.libreoffice.org/#/c/38414/) Let's put this one to FIXED.
Julien Nabet committed a patch related to this issue. It has been pushed to "libreoffice-5-3": http://cgit.freedesktop.org/libreoffice/core/commit/?id=75d14731e79b66a75e2d3327b94721f6218eb4d7&h=libreoffice-5-3 tdf#101885: fix crash in AccFrameSelector It will be available in 5.3.5. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.