Launchpad has imported 4 comments from the remote bug at
https://bugs.kde.org/show_bug.cgi?id=256944.

If you reply to an imported comment from within Launchpad, your comment
will be sent to the remote bug automatically. Read more about
Launchpad's inter-bugtracker facilities at
https://help.launchpad.net/InterBugTracking.

------------------------------------------------------------------------
On 2010-11-15T01:12:56+00:00 Eckhart Wörner wrote:

Version:           unspecified (using KDE 4.4.5) 
OS:                Linux

This bug has been copied over from http://bugs.debian.org/cgi-
bin/bugreport.cgi?bug=603480

----

In qguiplatformplugin_kde.cpp:fileDialogSetVisible it has
        if (!kdefd && visible) {

           if(qfd->options() & QFileDialog::DontUseNativeDialog)
                return false;

            kdefd = new 
KFileDialogBridge(KUrl::fromPath(qfd->directory().canonicalPath()),
                                          
qt2KdeFilter(qfd->nameFilters().join(";;")), qfd);

            qfd->setProperty("_k_bridge", QVariant::fromValue(kdefd));
        }

i.e. if you are making this visible and kdefd hasn't been created then create 
it now.
Most of the contents of the function are guarded by an if (visible) however on 
line 249 we have
        kdefd->setVisible(visible);

and if you make the object hidden before it's ever been made visible then kdefd 
will be NULL
and it seg's on line 249.

This is the underlying cause of [Debian] bug 599303 in ktoon which is marked RC.
It makes ktoon unusable for KDE users (although it works for Gnome users!)

I suggest guarding this line with an if (kdefd)  but I don't know the
structure to know if that is all that is needed.

Reproducible: Didn't try

Reply at:
https://bugs.launchpad.net/ubuntu/+source/ktoon/+bug/592129/comments/5

------------------------------------------------------------------------
On 2010-11-15T15:09:30+00:00 Cfeck wrote:

Proposed patch:

Index: qguiplatformplugin_kde.cpp
===================================================================
--- qguiplatformplugin_kde.cpp  (revision 1197348)
+++ qguiplatformplugin_kde.cpp  (working copy)
@@ -214,7 +214,7 @@
     virtual bool fileDialogSetVisible(QFileDialog *qfd, bool visible)
     {
         K_FD(qfd);
-        if (!kdefd && visible) {
+        if (!kdefd) {
             if(qfd->options() & QFileDialog::DontUseNativeDialog)
                 return false;

Reply at:
https://bugs.launchpad.net/ubuntu/+source/ktoon/+bug/592129/comments/6

------------------------------------------------------------------------
On 2010-11-21T02:30:53+00:00 Eckhart Wörner wrote:

(In reply to comment #1)
> Proposed patch:

Doesn't that make fileDialogSetVisible(false) unnecessarily expensive?
What are the reasons against the original submitter's proposal of
guarding the kdefd->setVisible(visible) call?

Reply at:
https://bugs.launchpad.net/ubuntu/+source/ktoon/+bug/592129/comments/7

------------------------------------------------------------------------
On 2020-12-09T01:34:23+00:00 Justin Zobel wrote:

Thank you for the report, Eckhart.

As it has been a while since this was reported, can you please test and
confirm if this issue is still occurring or if this bug report can be
marked as resolved.

I have set the bug status to "needsinfo" pending your response, please
change back to "reported" or "resolved/worksforme" when you respond,
thank you.

Reply at:
https://bugs.launchpad.net/ubuntu/+source/ktoon/+bug/592129/comments/18


** Changed in: kdebase-workspace
       Status: Confirmed => Incomplete

-- 
You received this bug notification because you are a member of Kubuntu
Bugs, which is subscribed to kde-workspace in Ubuntu.
https://bugs.launchpad.net/bugs/592129

Title:
  ktoon crashes on start (empty window with title Fatal error)

To manage notifications about this bug go to:
https://bugs.launchpad.net/kdebase-workspace/+bug/592129/+subscriptions

-- 
kubuntu-bugs mailing list
kubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/kubuntu-bugs

Reply via email to