Bug 43562 - [Qt] Temporary files should not be created in the current directory
Summary: [Qt] Temporary files should not be created in the current directory
Alias: None
Product: WebKit
Classification: Unclassified
Component: Platform (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Simon Hausmann
Keywords: Qt, QtTriaged
Depends on:
Reported: 2010-08-05 08:31 PDT by Simon Hausmann
Modified: 2010-08-24 06:32 PDT (History)
3 users (show)

See Also:

Patch (1.56 KB, patch)
2010-08-05 08:37 PDT, Simon Hausmann
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Simon Hausmann 2010-08-05 08:31:53 PDT
Currently the way FileSystemQt.cpp creates temporary files makes them end up in the current directory.

We should be using the dedicated temporary directory on the system instead.
Comment 1 Simon Hausmann 2010-08-05 08:36:23 PDT
The use of the current directory creates problems on Symbian (with different drives, etc.) and in general the system temporary directory is the better place, as it may be subject to cleanup (gc on system startup) and it has better permissions in place.
Comment 2 Simon Hausmann 2010-08-05 08:37:13 PDT
Created attachment 63596 [details]
Comment 3 Laszlo Gombos 2010-08-05 08:50:26 PDT
Comment on attachment 63596 [details]

looks good to me, r+.
Comment 4 WebKit Commit Bot 2010-08-05 22:09:00 PDT
Comment on attachment 63596 [details]

Clearing flags on attachment: 63596

Committed r64816: <http://trac.webkit.org/changeset/64816>
Comment 5 WebKit Commit Bot 2010-08-05 22:09:05 PDT
All reviewed patches have been landed.  Closing bug.
Comment 6 Simon Hausmann 2010-08-10 06:39:45 PDT
Revision r64816 cherry-picked into qtwebkit-2.1 with commit 195b286c999effb1c2e675ce35c3936c5cea946d