Bug 122497 - crash when a non existing data type is used for an argument of a new mehod
Summary: crash when a non existing data type is used for an argument of a new mehod
Status: RESOLVED FIXED
Alias: None
Product: umbrello
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Debian testing Linux
: NOR crash
Target Milestone: ---
Assignee: Umbrello Development Group
URL:
Keywords:
: 124278 (view as bug list)
Depends on:
Blocks:
 
Reported: 2006-02-22 17:00 UTC by Aurélien
Modified: 2006-03-26 14:51 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Aurélien 2006-02-22 17:00:50 UTC
Version:           1.5.1 (using KDE KDE 3.5.1)
Installed from:    Debian testing/unstable Packages
OS:                Linux

In an empty project, create a new class. Then add an operation, like: 
new_operation(a : char *)

Then a message saying that the type does not exist pops up (which is correct) and, after clicking, the application crashes.

Note: if one does the same in two steps: (1) create an operation and (2) add an argument of a non existing type, the same error is correctly reported and the system does NOT crash.
Comment 1 Oliver Kellogg 2006-03-10 21:23:57 UTC
SVN commit 517371 by okellogg:

createChildUMLObject(), itemRenamed(): Deletion of the UMLListViewItem
is done at the caller, UMLListViewItem::okRename().
BUG:122497


 M  +0 -6      umllistview.cpp  


--- branches/KDE/3.5/kdesdk/umbrello/umbrello/umllistview.cpp #517370:517371
@@ -2256,7 +2256,6 @@
                         kapp -> mainWidget(),
                         i18n( "The name you entered was not unique.\nIs this what you wanted?" ),
                         i18n( "Name Not Unique" ), i18n("Use Name"), i18n("Enter New Name") ) == KMessageBox::No ) {
-                delete item;
                 return false;
             }
         } else {
@@ -2416,7 +2415,6 @@
     if( !parent ) {
         kdError() << "UMLListView::createChildUMLObject: parent UMLObject is NULL"
         << endl;
-        delete item;
         m_bCreatingChildObject = false;
         return false;
     }
@@ -2431,7 +2429,6 @@
             KMessageBox::error( kapp->mainWidget(),
                                 Model_Utils::psText(st),
                                 i18n("Creation canceled") );
-            delete item;
             m_bCreatingChildObject = false;
             return false;
         }
@@ -2447,7 +2444,6 @@
             KMessageBox::error( kapp->mainWidget(),
                                 Model_Utils::psText(st),
                                 i18n("Creation canceled") );
-            delete item;
             m_bCreatingChildObject = false;
             return false;
         }
@@ -2465,7 +2461,6 @@
             KMessageBox::error( kapp->mainWidget(),
                                 Model_Utils::psText(st),
                                 i18n("Creation canceled") );
-            delete item;
             m_bCreatingChildObject = false;
             return false;
         }
@@ -2477,7 +2472,6 @@
                     kapp -> mainWidget(),
                     i18n( "The name you entered was not unique!\nCreation process has been canceled." ),
                     i18n( "Name Not Unique" ) );
-            delete item;
             m_bCreatingChildObject = false;
             return false;
         }
Comment 2 Oliver Kellogg 2006-03-26 14:51:52 UTC
*** Bug 124278 has been marked as a duplicate of this bug. ***