<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>43562</bug_id>
          
          <creation_ts>2010-08-05 08:31:53 -0700</creation_ts>
          <short_desc>[Qt] Temporary files should not be created in the current directory</short_desc>
          <delta_ts>2010-08-24 06:32:20 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Platform</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>Qt, QtTriaged</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Simon Hausmann">hausmann</reporter>
          <assigned_to name="Simon Hausmann">hausmann</assigned_to>
          <cc>commit-queue</cc>
    
    <cc>hui_huang</cc>
    
    <cc>laszlo.gombos</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>260623</commentid>
    <comment_count>0</comment_count>
    <who name="Simon Hausmann">hausmann</who>
    <bug_when>2010-08-05 08:31:53 -0700</bug_when>
    <thetext>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.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>260626</commentid>
    <comment_count>1</comment_count>
    <who name="Simon Hausmann">hausmann</who>
    <bug_when>2010-08-05 08:36:23 -0700</bug_when>
    <thetext>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.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>260627</commentid>
    <comment_count>2</comment_count>
      <attachid>63596</attachid>
    <who name="Simon Hausmann">hausmann</who>
    <bug_when>2010-08-05 08:37:13 -0700</bug_when>
    <thetext>Created attachment 63596
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>260635</commentid>
    <comment_count>3</comment_count>
      <attachid>63596</attachid>
    <who name="Laszlo Gombos">laszlo.gombos</who>
    <bug_when>2010-08-05 08:50:26 -0700</bug_when>
    <thetext>Comment on attachment 63596
Patch

looks good to me, r+.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>261088</commentid>
    <comment_count>4</comment_count>
      <attachid>63596</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-08-05 22:09:00 -0700</bug_when>
    <thetext>Comment on attachment 63596
Patch

Clearing flags on attachment: 63596

Committed r64816: &lt;http://trac.webkit.org/changeset/64816&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>261089</commentid>
    <comment_count>5</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-08-05 22:09:05 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>262656</commentid>
    <comment_count>6</comment_count>
    <who name="Simon Hausmann">hausmann</who>
    <bug_when>2010-08-10 06:39:45 -0700</bug_when>
    <thetext>Revision r64816 cherry-picked into qtwebkit-2.1 with commit 195b286c999effb1c2e675ce35c3936c5cea946d</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>63596</attachid>
            <date>2010-08-05 08:37:13 -0700</date>
            <delta_ts>2010-08-05 22:09:00 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-43562-20100805173711.patch</filename>
            <type>text/plain</type>
            <size>1596</size>
            <attacher name="Simon Hausmann">hausmann</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1dlYkNvcmUvQ2hhbmdlTG9nIGIvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXgg
OTE0MzJmMGE3NjAxYmE2MGY1YTBmOWY4YWQ5ODRjMWI5NjQ2YmQ5OC4uNWJjOWRiNjcxMzE0YWY2
MjBlYWRjYmJkOWFmMTRhMTUyZmU4MGNjNCAxMDA2NDQKLS0tIGEvV2ViQ29yZS9DaGFuZ2VMb2cK
KysrIGIvV2ViQ29yZS9DaGFuZ2VMb2cKQEAgLTEsNSArMSwxOCBAQAogMjAxMC0wOC0wNSAgU2lt
b24gSGF1c21hbm4gIDxzaW1vbi5oYXVzbWFubkBub2tpYS5jb20+CiAKKyAgICAgICAgUmV2aWV3
ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgW1F0XSBUZW1wb3JhcnkgZmlsZXMgc2hv
dWxkIG5vdCBiZSBjcmVhdGVkIGluIHRoZSBjdXJyZW50IGRpcmVjdG9yeQorICAgICAgICBodHRw
czovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NDM1NjIKKworICAgICAgICBEb24n
dCBjcmVhdGUgdGVtcG9yYXJ5IGZpbGVzIGluIHRoZSBjdXJyZW50IGRpcmVjdG9yeSwgdXNlIFFE
aXI6OnRlbXBQYXRoKCkKKyAgICAgICAgaW5zdGVhZCwgYXMgc3VnZ2VzdGVkIGJ5IHRoZSBRVGVt
cG9yYXJ5RmlsZSBkb2N1bWVudGF0aW9uLgorCisgICAgICAgICogcGxhdGZvcm0vcXQvRmlsZVN5
c3RlbVF0LmNwcDoKKyAgICAgICAgKFdlYkNvcmU6Om9wZW5UZW1wb3JhcnlGaWxlKToKKworMjAx
MC0wOC0wNSAgU2ltb24gSGF1c21hbm4gIDxzaW1vbi5oYXVzbWFubkBub2tpYS5jb20+CisKICAg
ICAgICAgUmV2aWV3ZWQgYnkgVG9yIEFybmUgVmVzdGLDuC4KIAogICAgICAgICBbUXRdIENsZWFu
IHVwIHRoZSBpbnB1dCBtZXRob2QgaGFuZGxpbmcKZGlmZiAtLWdpdCBhL1dlYkNvcmUvcGxhdGZv
cm0vcXQvRmlsZVN5c3RlbVF0LmNwcCBiL1dlYkNvcmUvcGxhdGZvcm0vcXQvRmlsZVN5c3RlbVF0
LmNwcAppbmRleCAwM2RmYzhmYmU1YWRkYjZhMWU5Njc0MzAzMzVjMjM2NGIyZDMyM2JjLi45NmMy
MDkzYTQ1NzEwMjEwZTRhMTQzMjVmYThmMmJlOGU2MGRlYTA2IDEwMDY0NAotLS0gYS9XZWJDb3Jl
L3BsYXRmb3JtL3F0L0ZpbGVTeXN0ZW1RdC5jcHAKKysrIGIvV2ViQ29yZS9wbGF0Zm9ybS9xdC9G
aWxlU3lzdGVtUXQuY3BwCkBAIC0xMTYsNyArMTE2LDcgQEAgVmVjdG9yPFN0cmluZz4gbGlzdERp
cmVjdG9yeShjb25zdCBTdHJpbmcmIHBhdGgsIGNvbnN0IFN0cmluZyYgZmlsdGVyKQogQ1N0cmlu
ZyBvcGVuVGVtcG9yYXJ5RmlsZShjb25zdCBjaGFyKiBwcmVmaXgsIFBsYXRmb3JtRmlsZUhhbmRs
ZSYgaGFuZGxlKQogewogI2lmbmRlZiBRVF9OT19URU1QT1JBUllGSUxFCi0gICAgUVRlbXBvcmFy
eUZpbGUqIHRlbXBGaWxlID0gbmV3IFFUZW1wb3JhcnlGaWxlKFFMYXRpbjFTdHJpbmcocHJlZml4
KSk7CisgICAgUVRlbXBvcmFyeUZpbGUqIHRlbXBGaWxlID0gbmV3IFFUZW1wb3JhcnlGaWxlKFFE
aXI6OnRlbXBQYXRoKCkgKyBRTGF0aW4xQ2hhcignLycpICsgUUxhdGluMVN0cmluZyhwcmVmaXgp
KTsKICAgICB0ZW1wRmlsZS0+c2V0QXV0b1JlbW92ZShmYWxzZSk7CiAgICAgUUZpbGUqIHRlbXAg
PSB0ZW1wRmlsZTsKICAgICBpZiAodGVtcC0+b3BlbihRSU9EZXZpY2U6OlJlYWRXcml0ZSkpIHsK
</data>

          </attachment>
      

    </bug>

</bugzilla>