Bug 87156 - LibreLogo: Hangs on input with extra hard return at the top
Summary: LibreLogo: Hangs on input with extra hard return at the top
Status: RESOLVED WORKSFORME
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
4.3.5.1 rc
Hardware: Other All
: medium major
Assignee: Not Assigned
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-12-09 14:39 UTC by Robinson Tryon (qubit)
Modified: 2015-07-04 13:41 UTC (History)
3 users (show)

See Also:
Crash report or crash signature:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Robinson Tryon (qubit) 2014-12-09 14:39:31 UTC
TESTING with LO 4.3.5.1 on Ubuntu 14.04

Repro Steps:
- Open Writer
- Type 'RECTANGLE [ 50, 100 ]'
- Enable LibreLogo toolbar (View -> Toolbars -> Logo)
- Click 'Start LibreLogo program...'

A rectangle will be drawn

- Now add a hard return before 'RECTANGLE [ 50, 100 ]' and click the Start LibreLogo button again

RESULT: LibreOffice hangs

(sometimes LibreOffice only hangs the 2nd time you click the button)


NOREPRO on 4.4.0.0.beta2
Comment 1 Robinson Tryon (qubit) 2014-12-09 14:51:44 UTC
With further testing, reproduction can be sporadic. Here's a more consistent way to hang LibreOffice that I believe is triggering the same underlying issue:

(In reply to Robinson Tryon (qubit) from comment #0)
> Repro Steps:
> - Open Writer
> - Type 'RECTANGLE [ 50, 100 ]'
> - Enable LibreLogo toolbar (View -> Toolbars -> Logo)
- DOUBLE-click 'Start LibreLogo program...'

RESULT: LibreOffice hangs

Also hangs LO 4.4.0.0.beta2.
Comment 2 Julien Nabet 2014-12-09 21:57:43 UTC
On pc Debian x86-64 with master sources updated today, I could reproduce this.

It seems an infinite loop in SwFindParaText::Find, here's a bt:
0  0x00002aaadcbd3621 in sw::Ring<SwPaM>::Ring_node_traits::get_next (
    n=0x7ffffffee480)
    at /home/julien/compile-libreoffice/libreoffice/sw/inc/ring.hxx:100
#1  0x00002aaadcbd1e1c in boost::intrusive::circular_list_algorithms<sw::Ring<SwPaM>::Ring_node_traits>::unique (this_node=@0x7ffffffee240: 0x7ffffffee480)
    at /home/julien/compile-libreoffice/libreoffice/workdir/UnpackedTarball/boost/boost/intrusive/circular_list_algorithms.hpp:105
#2  0x00002aaadcbd1c4a in sw::Ring<SwPaM>::MoveTo (this=0x7ffffffee480, 
    pDestRing=0x7ffffffee480)
    at /home/julien/compile-libreoffice/libreoffice/sw/inc/ring.hxx:142
#3  0x00002aaadcc834e0 in SwFindParaText::Find (this=0x7ffffffee5d0, 
    pCrsr=0x2b86678, fnMove=0x2aaade4df300 <aFwrd>, pRegion=0x7ffffffee480, 
    bInReadOnly=false)
    at /home/julien/compile-libreoffice/libreoffice/sw/source/core/crsr/findtxt.cxx:599

Michael: one for you?
Comment 3 Julien Nabet 2014-12-09 22:39:31 UTC
Some gdb traces:

Breakpoint 1, SwFindParaText::Find (this=0x7ffffffee5d0, pCrsr=0x2903098, fnMove=0x2aaaca170300 <aFwrd>, pRegion=0x7ffffffee480, bInReadOnly=true)
    at /home/julien/compile-libreoffice/libreoffice/sw/source/core/crsr/findtxt.cxx:565
565	    if( bInReadOnly && bReplace )
(gdb) p rSearchOpt
$1 = (const com::sun::star::util::SearchOptions &) @0x7ffffffee800: {algorithmType = com::sun::star::util::SearchAlgorithms_REGEXP, searchFlag = 65536, searchString = "\\n", 
  replaceString = "\\n", Locale = {Language = "fr", Country = "FR", Variant = ""}, changedChars = 2, deletedChars = 2, insertedChars = 2, transliterateFlags = 1073745152}
Comment 4 Gordo 2015-06-30 14:38:11 UTC
Could not reproduce.

There is now a dialogue that says:  Error (extra or missing spaces at brackets?).
I removed the space before the 50 and after the 100 and it works as expected.

Windows Vista 64
Version: 4.4.4.3
Build ID: 2c39ebcf046445232b798108aa8a7e7d89552ea8

Bug present in 4.3.7.2.

I want to move this to WFM but this needs to be tested in Linux.
Comment 5 Julien Nabet 2015-06-30 16:17:28 UTC
I don't remember how to add an hard return on the command line of LibreLogo (the little box where I can put 1 command only) so can't retest.
Comment 6 Gordo 2015-06-30 17:20:15 UTC
I used Enter.  It hangs in 4.3.7.2 with that.
Comment 7 Julien Nabet 2015-06-30 18:36:50 UTC
So indeed, I don't reproduce this with LO Debian package 4.4.4.2

Robinson: is it ok for you with last stable LO version?
Comment 8 Julien Nabet 2015-07-04 13:41:45 UTC
Rethinking about this, let's put this one to WFM.
Robinson: if you can still reproduce this, don't hesitate to reopen this tracker.