[PATCH] coverity#704931 Explicit null dereferenced

Julien Nabet (via Code Review) gerrit at gerrit.libreoffice.org
Wed Mar 20 14:29:21 PDT 2013


Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/2881

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/81/2881/1

coverity#704931 Explicit null dereferenced

Change-Id: I03fe61ace1ab408d51e93f1e447591bb4eb9449c
---
M sw/source/core/text/porglue.cxx
1 file changed, 1 insertion(+), 1 deletion(-)



diff --git a/sw/source/core/text/porglue.cxx b/sw/source/core/text/porglue.cxx
index bceadd8..8ce717d 100644
--- a/sw/source/core/text/porglue.cxx
+++ b/sw/source/core/text/porglue.cxx
@@ -218,7 +218,7 @@
         }
 
         // Two adjoining FlyPortions are merged
-        if( pRight && pLeft->GetPortion() == pRight )
+        if( pRight && pLeft && pLeft->GetPortion() == pRight )
         {
             pRight->MoveAllGlue( pLeft );
             pRight = 0;

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I03fe61ace1ab408d51e93f1e447591bb4eb9449c
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Julien Nabet <serval2412 at yahoo.fr>



More information about the LibreOffice mailing list