D14576: [Desktop Scripting ConfigGroup] Add more nullptr checks

Anthony Fieroni noreply at phabricator.kde.org
Fri Aug 3 11:06:35 BST 2018


anthonyfieroni added inline comments.

INLINE COMMENTS

> configgroup.cpp:67
>          d->synchTimer->stop();
>          d->configGroup->sync();
>      }

Here as well should be sync(), if it destroys object before timer is elapsed e.g. before 1.5 sec it will crash.

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D14576

To: broulik, #plasma
Cc: anthonyfieroni, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180803/7e6b92e2/attachment.html>


More information about the Plasma-devel mailing list