[Libreoffice-commits] core.git: sw/qa sw/source

[Caolán McNamara]

 sw/qa/core/data/ww5/pass/crash-1.doc |binary
 sw/source/filter/ww8/ww8scan.cxx     |    5 ++---
 2 files changed, 2 insertions(+), 3 deletions(-)

New commits:
commit 02e9b490a9401744a8a0699ad5dd10281b6b0b30
Author: Caolán McNamara <caolanm at redhat.com>
Date:   Mon Aug 24 15:29:54 2015 +0100

    loop over successfully read bookmarks
    
    not over the amount that should be there
    
    Change-Id: Ia2c353f9d7e274a5bebf405e1743ef1d1872b4de

diff --git a/sw/qa/core/data/ww5/pass/crash-1.doc b/sw/qa/core/data/ww5/pass/crash-1.doc
new file mode 100644
index 0000000..5b94b9a
Binary files /dev/null and b/sw/qa/core/data/ww5/pass/crash-1.doc differ
diff --git a/sw/source/filter/ww8/ww8scan.cxx b/sw/source/filter/ww8/ww8scan.cxx
index 19e8132..d6231a1 100644
--- a/sw/source/filter/ww8/ww8scan.cxx
+++ b/sw/source/filter/ww8/ww8scan.cxx
@@ -4129,10 +4129,10 @@ OUString WW8PLCFx_Book::GetBookmark(long nStart,long nEnd, sal_uInt16 &nIndex)
 {
     bool bFound = false;
     sal_uInt16 i = 0;
-    if( pBook[0] && pBook[1] )
+    if (pBook[0] && pBook[1])
     {
         WW8_CP nStartAkt, nEndAkt;
-        do
+        while (i < aBookNames.size())
         {
             void* p;
             sal_uInt16 nEndIdx;
@@ -4155,7 +4155,6 @@ OUString WW8PLCFx_Book::GetBookmark(long nStart,long nEnd, sal_uInt16 &nIndex)
             }
             ++i;
         }
-        while (i < pBook[0]->GetIMax());
     }
     return bFound ? aBookNames[i] : OUString();
 }