Bug 68395 - import of classes results in sig11 (with a little help)
Summary: import of classes results in sig11 (with a little help)
Status: RESOLVED FIXED
Alias: None
Product: umbrello
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Gentoo Packages Linux
: NOR crash
Target Milestone: ---
Assignee: Umbrello Development Group
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-11-16 22:30 UTC by Carsten Lohrke
Modified: 2003-11-21 19:11 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
Patch to fix 68395 (crash on class import) (3.88 KB, patch)
2003-11-20 22:27 UTC, lparrab
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Carsten Lohrke 2003-11-16 22:30:10 UTC
Version:            (using KDE KDE 3.1.93)
Installed from:    Gentoo Packages

while importing classes a operation properties dialog opens - pressing cancel crashes umbrello
Comment 1 lparrab 2003-11-20 22:27:31 UTC
Created attachment 3310 [details]
Patch to fix 68395 (crash on class import)
Comment 2 lparrab 2003-11-20 22:31:09 UTC
Umbrello crashes when the class has an overloaded operation, one of which takes no parameters.
The most comon case would be a class having two or more constructors and one of them doesnt take any parameters
The problem is:
1 - The doc always create the operation empty ( just with name ), so if the operation with no parameters has already been added to the classifier, the signature check will fail and the dialog to change the name and/or parameters pops up (first error)
2 - The user doesnt know why he gets the dialog, so just presses cancel and the doc returns 0L ( which is correct, since the user aborted the op creation)
3 - the class parser doesnt check for 0L and tries to set attributes and return type (second error)

Someone please review the patch, and I will apply it.
Comment 3 Sebastian Stein 2003-11-20 23:56:19 UTC
Subject: Re: [Uml-devel]  import of classes results in sig11 (with a little help)

lparrab@gmx.net <lparrab@gmx.net> [031120 22:39]:
> Umbrello crashes when the class has an overloaded operation, one of which
> takes no parameters.
> The most comon case would be a class having two or more constructors and
> one of them doesnt take any parameters

I think your patch works. I can't reproduce a crash. Ok, I can, but I think
it is another problem I will open another bug report for it ;-9

Steinchen

Comment 4 lparrab 2003-11-21 19:11:16 UTC
Patch applied