From 9b94d385f94562049bf93f2fc0dd0558d6a56dd1 Mon Sep 17 00:00:00 2001
From: Ivan Timofeev <timofeev.i.s@gmail.com>
Date: Fri, 4 Nov 2011 23:01:17 +0400
Subject: cppcheck: avoid possible null pointer dereferences

---
 basic/source/comp/exprtree.cxx | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

(limited to 'basic')

diff --git a/basic/source/comp/exprtree.cxx b/basic/source/comp/exprtree.cxx
index 1c7ec1be4767..41caa05a276c 100644
--- a/basic/source/comp/exprtree.cxx
+++ b/basic/source/comp/exprtree.cxx
@@ -983,8 +983,14 @@ SbiExpression* SbiExprList::Get( short n )
 
 void SbiExprList::addExpression( SbiExpression* pExpr )
 {
+    if( !pFirst )
+    {
+        pFirst = pExpr;
+        return;
+    }
+
     SbiExpression* p = pFirst;
-    while( p && p->pNext )
+    while( p->pNext )
         p = p->pNext;
 
     p->pNext = pExpr;
-- 
cgit