[Kde-imaging] [Bug 215846] PicasaWeb plugin crashes Digikam after start sending the pic [delete, KIPIPicasawebExportPlugin::PicasawebWindow::slotUploadImages] (double deletion)

[Dario Andres]

https://bugs.kde.org/show_bug.cgi?id=215846


Dario Andres <andresbajotierra at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
            Summary|PicasaWeb plugin crashes    |PicasaWeb plugin crashes
                   |Digikam after start sending |Digikam after start sending
                   |the pic [delete,            |the pic [delete,
                   |KIPIPicasawebExportPlugin:: |KIPIPicasawebExportPlugin::
                   |PicasawebWindow::slotUpload |PicasawebWindow::slotUpload
                   |Images]                     |Images] (double deletion)
     Ever Confirmed|0                           |1




--- Comment #11 from Dario Andres <andresbajotierra gmail com>  2009-12-14 19:24:49 ---
It seems that the crash is related to the deletion of the "m_urls" var.

I have experienced a similar crash on Gwenview:
kipi-plugins svn rev. 1062112

#12 0xb5aa5901 in operator delete(void*) () from /usr/lib/libstdc++.so.6
#13 0xb19f95ba in KIPIPicasawebExportPlugin::PicasawebWindow::closeEvent
(this=0x9ac6c78, e=0xbf958dac) at
/home/kde-devel/kde/src/KDE/kipi-plugins/picasawebexport/picasawebwindow.cpp:232
#14 0xb5cd2b4f in QWidget::event (this=0x9ac6c78, event=0xbf958dac) at
kernel/qwidget.cpp:8132
...

The culprit line is in fact: "delete m_urls;"

This variable is also deleted on other situations (upload, close, object
destruction), but it is never set to 0 after deleting it... so that will cause
a double-delete error later in some cases. <- That would be a simple patch

-- 
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.