[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-panel-devel
Subject: [Panel-devel] plasmagik and /tmp
From: "Aaron J. Seigo" <aseigo () kde ! org>
Date: 2007-10-11 17:02:59
Message-ID: 200710111102.59262.aseigo () kde ! org
[Download RAW message or body]
[Attachment #2 (multipart/signed)]
hi...
this is mostly for Andrea (adjam), but figure here is as good a place as any
to discuss =)
i noticed today a commit to plasmagik today that puts a directory in /tmp to
be zipped up when creating a package. this has a few problems (besides
leaking the new'd QDir on line 314) ... one can't assume /tmp is available
for use like that.
at a minimum instead this code should be using KStandardDirs to locate the
proper "tmp" resource dir. in fact, the best thing to do is to use KTempDir
here. you also get cleanup for free in that case and protect against symlink
attacks on the dir creation ...
hth..
--
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA EE75 D6B7 2EB1 A7F1 DB43
KDE core developer sponsored by Trolltech
[Attachment #5 (application/pgp-signature)]
_______________________________________________
Panel-devel mailing list
Panel-devel@kde.org
https://mail.kde.org/mailman/listinfo/panel-devel
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic