Bug 69510

Summary: crash in writer with Arabic strings followed by more than 2 numbers
Product: LibreOffice Reporter: Abdulaziz Ayed <aalayed>
Component: WriterAssignee: Caolán McNamara <caolanm>
Status: RESOLVED FIXED QA Contact:
Severity: critical    
Priority: high CC: aalabdulrazzaq, a.alharthi, dr.khaled.hosny, kaplanlior, zolnaitamas2000
Version: 4.2.0.0.alpha0+ Master   
Hardware: Other   
OS: All   
Whiteboard: target:4.2.0 target:4.1.4
i915 platform: i915 features:

Description Abdulaziz Ayed 2013-09-18 07:56:57 UTC
Hi all ;

there is a crash when you write RTL string followed by 2 numbers.

when I bibsect the core I find one patch with the id 0d9ddccd8810a81a6f4d737870969d0dcf367d23
from (Zolnai Tamás) cause crash, then I told him about that, then he revert it with id 
835d9e0da9416c7e586766ad734f45d048886e0a

but still the crash happen.

last log messeage date from my core is 15 sep 2013.
Comment 1 Abdulaziz Ayed 2013-09-18 08:16:51 UTC
sorry :
after revert its fixied but after 2-3 days its again happend.

steps to reproduce:

1-put text direction to LTR.
2-write Arabic strings "عبدالعزيز" followed by more than 2 numbers.
Comment 2 abdulmajeed 2013-09-18 08:38:32 UTC
Can reproduce in master linux 64bit 



Note:
Tested it in 3.6.2 works fine
Comment 3 Lior Kaplan 2013-09-18 11:00:20 UTC
Is this bug only relevant for master, or is it also reproducible in 4.1.1?
Comment 4 Abdulaziz Ayed 2013-09-18 11:33:21 UTC
(In reply to comment #3)
> Is this bug only relevant for master, or is it also reproducible in 4.1.1?

not reproducible in 4.1.1
Comment 5 Khaled Hosny 2013-09-19 09:37:07 UTC
I can’t reproduce this in master, do you have a backtrace?
Comment 6 Abdulaziz Ayed 2013-09-19 11:36:53 UTC
(In reply to comment #5)
> I can’t reproduce this in master, do you have a backtrace?

this is backtrace log

http://pastebin.ca/2454686
Comment 7 Khaled Hosny 2013-09-19 13:34:59 UTC
Looks like an assertion failure in of of the string classes, may be related to the current wave of String → OUString conversions.
Comment 8 Commit Notification 2013-10-29 11:21:29 UTC
Caolan McNamara committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=47690627565bec866ea2de74b5504160479cff74

Resolves: fdo#69510 don't access beyond end of string



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.
Comment 9 Caolán McNamara 2013-10-29 11:21:56 UTC
proposed this as a 4-1 fix in gerrit
Comment 10 Commit Notification 2013-10-29 12:27:07 UTC
Caolan McNamara committed a patch related to this issue.
It has been pushed to "libreoffice-4-1":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=66edf3c164176180c15dc768ce2b4518a9178183&h=libreoffice-4-1

Resolves: fdo#69510 don't access beyond end of string


It will be available in LibreOffice 4.1.4.

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.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.