Review Request 121161: Set KIO::Integration::AccessManager to null so we don't crash on close.
Thomas Lübking
thomas.luebking at gmail.com
Tue Nov 18 12:53:28 UTC 2014
> On Nov. 17, 2014, 9:45 nachm., Thomas Lübking wrote:
> > attica-kde/kdeplugin/kdeplatformdependent.cpp, line 56
> > <https://git.reviewboard.kde.org/r/121161/diff/2/?file=328928#file328928line56>
> >
> > is
> >
> > KdePlatformDependent::~KdePlatformDependent()
> > {
> > if (QCoreApplication::closingDown())
> > m_accessManager->setParent(nullptr);
> > }
> >
> > sufficient?
>
> Albert Astals Cid wrote:
> Isn't that a more complex way of doing the same?
>
> Thomas Lübking wrote:
> Depends on whether KdePlatformDependent is ever deleted in a running application (where the leak would really matter)
> If not, then yes: superfluous complication.
>
> Jeremy Whiting wrote:
> Did I hear a "Ship it!" in there somewhere?
Leaving aside that don't maintain the attica plugin, you have a "+1" from here, IF
a) the conditional reparent on destruction is not sufficient (doesn't work)
OR
b) KdePlatformDependent object(s) are not supposed to be deleted during the runtime of the application
Ie. if the leak is inevitable or only theoretical.
- Thomas
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/121161/#review70542
-----------------------------------------------------------
On Nov. 17, 2014, 9:11 nachm., Jeremy Whiting wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/121161/
> -----------------------------------------------------------
>
> (Updated Nov. 17, 2014, 9:11 nachm.)
>
>
> Review request for kdelibs and Plasma.
>
>
> Repository: plasma-desktop
>
>
> Description
> -------
>
> Plugin unload order was making the attica_kde plugin crash on close,
> this works around it by leaking one AccessManager.
>
>
> Diffs
> -----
>
> attica-kde/kdeplugin/kdeplatformdependent.cpp 489c03b18b7bb940007ab51cb197105fbc25de9f
>
> Diff: https://git.reviewboard.kde.org/r/121161/diff/
>
>
> Testing
> -------
>
> knewstuff tests no longer crash on close.
>
>
> Thanks,
>
> Jeremy Whiting
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20141118/14aa461b/attachment.html>
More information about the Plasma-devel
mailing list