[PATCH] SW : Fix crash by checking find result

Arnaud Versini (via Code Review) gerrit at gerrit.libreoffice.org
Sat May 4 08:47:38 PDT 2013


Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/3783

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/83/3783/1

SW : Fix crash by checking find result

Change-Id: I6caf18b110dd0a40f09e185c00cb238261457db3
---
M sw/source/core/txtnode/ndtxt.cxx
1 file changed, 4 insertions(+), 1 deletion(-)



diff --git a/sw/source/core/txtnode/ndtxt.cxx b/sw/source/core/txtnode/ndtxt.cxx
index 437cf44..07f36ef 100644
--- a/sw/source/core/txtnode/ndtxt.cxx
+++ b/sw/source/core/txtnode/ndtxt.cxx
@@ -3211,7 +3211,10 @@
                 if (aHiddenMulti.IsSelected(nDummyCharPos))
                     continue;
                 std::vector<block>::iterator aFind = std::find_if(aBlocks.begin(), aBlocks.end(), containsPos(nDummyCharPos));
-                aFind->m_aAttrs.push_back(pAttr);
+                if (aFind != aBlocks.end())
+                {
+                    aFind->m_aAttrs.push_back(pAttr);
+                }
             }
         }
     }

-- 
To view, visit https://gerrit.libreoffice.org/3783
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I6caf18b110dd0a40f09e185c00cb238261457db3
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Arnaud Versini <arnaud.versini at gmail.com>



More information about the LibreOffice mailing list