Review Request 126212: Do not crash if KDE platform integration is loaded but SNI is unavailable

Sven Brauch mail at svenbrauch.de
Mon Dec 7 17:45:44 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126212/
-----------------------------------------------------------

(Updated Dec. 7, 2015, 5:45 p.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Frameworks.


Changes
-------

Submitted with commit b45544f3d4dd9cb1873b92a609f45a68f5f6e471 by Sven Brauch to branch master.


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


Repository: knotifications


Description
-------

See bug 350785. The crash is quite nasty, makes dbus allocate all available file descriptors plus hangs the application; worth preventing imo. Yes, the setup leading to this issue is broken, but it's just an env var and it's hard to track down.


Diffs
-----

  src/kstatusnotifieritem.cpp c48f4d0 

Diff: https://git.reviewboard.kde.org/r/126212/diff/


Testing
-------

Now prints an error message instead of crashing.


Thanks,

Sven Brauch

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20151207/91e08d77/attachment.html>


More information about the Kde-frameworks-devel mailing list