[PATCH] coverity#982464: Explicit null dereferenced

Julien Nabet (via Code Review) gerrit at gerrit.libreoffice.org
Tue May 7 16:38:11 PDT 2013


Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/3822

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/22/3822/1

coverity#982464: Explicit null dereferenced

Change-Id: Idb75b5b9040ce7f54775f8bc02c150f87e811af0
---
M sc/source/filter/lotus/tool.cxx
1 file changed, 2 insertions(+), 1 deletion(-)



diff --git a/sc/source/filter/lotus/tool.cxx b/sc/source/filter/lotus/tool.cxx
index 136e607..799d21d 100644
--- a/sc/source/filter/lotus/tool.cxx
+++ b/sc/source/filter/lotus/tool.cxx
@@ -101,7 +101,8 @@
     pDoc->ApplyAttr( nCol, nRow, nTab, *pJustify );
     ScSetStringParam aParam;
     aParam.setTextInput();
-    pDoc->SetString(ScAddress(nCol,nRow,nTab), String(pString, pLotusRoot->eCharsetQ), &aParam);
+    if (pString)
+        pDoc->SetString(ScAddress(nCol,nRow,nTab), String(pString, pLotusRoot->eCharsetQ), &aParam);
 }
 
 

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

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



More information about the LibreOffice mailing list