Created attachment 95637 [details] This file opens properly on MSWord 2010, but LO crashes while opening the file STEPS: 1) Open the attached file in LO (Version: 4.3.0.0.alpha0+ Build ID: dd7787ed75e33b65ebee2a6c0aa152efded6f1b8 ) 2) LO crashes. Expected: LO shouldn't crash. I have attached the stack trace also.
Created attachment 95638 [details] This file has the stack
1) The original file added to this bug contains a table which spans across pages. 2) On a table cell a footnote has been added. 3) In the footnote again a table has been added. 4) If I remove the table LO doesn't crash.
Created attachment 95654 [details] this file contains table in footnote, but it doesn't crash
Created attachment 95682 [details] console logs + bt with symbols On pc Debian x86-64 with master sources updated yesterday, I could reproduce this. I attached console logs + bt
I haven't tested but could this patch make it? diff --git a/sw/source/core/layout/findfrm.cxx b/sw/source/core/layout/findfrm.cxx index fd6a778..86d007b 100644 --- a/sw/source/core/layout/findfrm.cxx +++ b/sw/source/core/layout/findfrm.cxx @@ -1013,7 +1013,7 @@ SwFrm *SwFrm::_FindPrev() if ( !bIgnoreTab && pThis->IsInTab() ) { SwLayoutFrm *pUp = pThis->GetUpper(); - while ( !pUp->IsCellFrm() ) + while ( pUp && !pUp->IsCellFrm() ) pUp = pUp->GetUpper(); OSL_ENSURE( pUp, "Cntnt in table but not in cell." ); if ( pUp->IsAnLower( pPrvCnt ) )
(In reply to comment #5) > I haven't tested but could this patch make it? > diff --git a/sw/source/core/layout/findfrm.cxx > b/sw/source/core/layout/findfrm.cxx > index fd6a778..86d007b 100644 > --- a/sw/source/core/layout/findfrm.cxx > +++ b/sw/source/core/layout/findfrm.cxx > @@ -1013,7 +1013,7 @@ SwFrm *SwFrm::_FindPrev() > if ( !bIgnoreTab && pThis->IsInTab() ) > { > SwLayoutFrm *pUp = pThis->GetUpper(); > - while ( !pUp->IsCellFrm() ) > + while ( pUp && !pUp->IsCellFrm() ) > pUp = pUp->GetUpper(); > OSL_ENSURE( pUp, "Cntnt in table but not in cell." ); > if ( pUp->IsAnLower( pPrvCnt ) ) Miklos told me that currently tables are not supported in footnotes. but I found a weird thing.. 1) if single table in a footnote on the last paragraph, then it doesn't crash else if the footnote is on any other page it crashes. Miklos has asked to handle it the same way as done for doc import.
(In reply to comment #5) > I haven't tested but could this patch make it? > diff --git a/sw/source/core/layout/findfrm.cxx > b/sw/source/core/layout/findfrm.cxx > index fd6a778..86d007b 100644 > --- a/sw/source/core/layout/findfrm.cxx > +++ b/sw/source/core/layout/findfrm.cxx > @@ -1013,7 +1013,7 @@ SwFrm *SwFrm::_FindPrev() > if ( !bIgnoreTab && pThis->IsInTab() ) > { > SwLayoutFrm *pUp = pThis->GetUpper(); > - while ( !pUp->IsCellFrm() ) > + while ( pUp && !pUp->IsCellFrm() ) > pUp = pUp->GetUpper(); > OSL_ENSURE( pUp, "Cntnt in table but not in cell." ); > if ( pUp->IsAnLower( pPrvCnt ) ) I tried your change, LO still crashes..
The first attachment doesn't crash, tested using Windows 8.1 with LibreOffice Version: 4.4.0.0.alpha0+ Build ID: a82ff18269e5b37348d402b7c21c3f200068265c TinderBox: Win-x86@39, Branch:master, Time: 2014-07-20_02:34:36 @Julien: you mind retesting and verifying? Kind regards, Joren
I confirm I've got no more crash too with master sources updated today.