Re: [PATCH] avoid Possible null pointer dereference in linguistic/source/spelldsp.hxx

Hi Ivan,

So it would be this :
diff --git a/linguistic/source/spelldsp.hxx b/linguistic/source/spelldsp.hxx
index 9ae9cd4..e2186f9 100644
--- a/linguistic/source/spelldsp.hxx
+++ b/linguistic/source/spelldsp.hxx
@@ -73,7 +73,7 @@ class SpellCheckerDispatcher :
         ::com::sun::star::linguistic2::XSearchableDictionaryList > 
xDicList;
 
     LngSvcMgr                   &rMgr;
-    linguistic::SpellCache      *pCache; // Spell Cache (holds known words)
+    mutable linguistic::SpellCache      *pCache; // Spell Cache (holds
known words)
 
     // disallow copy-constructor and assignment-operator for now
     SpellCheckerDispatcher(const SpellCheckerDispatcher &);
@@ -134,7 +134,7 @@ public:
 inline linguistic::SpellCache & SpellCheckerDispatcher::GetCache() const
 {
     if (!pCache)
-        ((SpellCheckerDispatcher *) this)->pCache = new
linguistic::SpellCache();
+        pCache = new linguistic::SpellCache();
     return *pCache;
 }
 
What about the Riccardo's suggestion ?

I don't know which solution would be the most appropriate and why ?

Julien.

--
View this message in context: 
http://nabble.documentfoundation.org/PATCH-avoid-Possible-null-pointer-dereference-in-linguistic-source-spelldsp-hxx-tp3735163p3735284.html
Sent from the Dev mailing list archive at Nabble.com.