D7146: Fix information leak via /tmp

Aleix Pol Gonzalez noreply at phabricator.kde.org
Sat Aug 5 15:54:36 UTC 2017


apol requested changes to this revision.
apol added inline comments.
This revision now requires changes to proceed.

INLINE COMMENTS

> notification.cpp:36
>  {
> -    mImagesDir = QDir::temp().absoluteFilePath(QStringLiteral("kdeconnect"));
> +    mImagesDir = QDir(".local/tmp/kdeconnect");
>      mImagesDir.mkpath(mImagesDir.absolutePath());

Use `QStandardPaths`, this isn't portable.

REPOSITORY
  R224 KDE Connect

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

To: nicolasfella, #kde_connect, apol
Cc: apol, #kde_connect, albertvaka, tfella, aboudhar, seebauer, progwolff, MayeulC, menasshock, ach, hkaelberer
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20170805/a7c360ba/attachment.html>


More information about the KDEConnect mailing list