[Uml-devel] branches/KDE/3.5/kdesdk/umbrello/umbrello

[Oliver Kellogg]

SVN commit 604948 by okellogg:

moveObject(): Avoid null pointer deref.

 M  +3 -0      umllistview.cpp  


--- branches/KDE/3.5/kdesdk/umbrello/umbrello/umllistview.cpp #604947:604948
@@ -1369,6 +1369,9 @@
             UMLCanvasObject *o = static_cast<UMLCanvasObject*>(newItem->getUMLObject());
             if (o == NULL) {
                 kdDebug() << "moveObject: newItem's UMLObject is NULL" << endl;
+            } else if (newParentObj == NULL) {
+                kdError() << "UMLListView::moveObject(" << o->getName()
+                    << "): newParentObj is NULL" << endl;
             } else {
                 UMLPackage *pkg = static_cast<UMLPackage*>(newParentObj);
                 o->setUMLPackage( pkg );