found 352210 1.61-19
tags 352210 + patch
owner !
thanks

I'm sorry to say that 352210 still exists partly in the current
version: Now the system temp dir is used but the hard disk fills up
until it's full.

Digging through the php source and doing some tests it seems that the
code in dl.php that creates a directory and tries to delete it fails
- in all cases where the download is canceled (and the script ends
prematurely).

Adding ignore_user_abort(true); to dl.php fixes this issue on my
installation; cf. the attached patch.

Cheers,
gregor

-- 
 .''`.   http://info.comodo.priv.at/ | gpg key ID: 0x00F3CFE4
 : :' :  debian: the universal operating system - http://www.debian.org/
 `. `'   member of https://www.vibe.at/ | how to reply: http://got.to/quote/
   `-    NP: Chris Rea: That Girl Of Mine
--- dl.php.orig 2006-11-11 17:01:49.000000000 +0100
+++ dl.php      2006-11-11 17:59:16.000000000 +0100
@@ -34,2 +34,6 @@
 
+// Make sure the script is finished even if the client disconnects.
+// Otherwise the temporary directory is not deleted.
+ignore_user_abort(true);
+
 $svnrep = new SVNRepository($rep->path);

Attachment: signature.asc
Description: Digital signature

Reply via email to