- Fixed memory leak when calling SplFileInfo's constructor twice

diff --git a/ext/spl/spl_directory.c b/ext/spl/spl_directory.c
index 4b0f48fedd07b28bde1bee41801110f3e3a29ec7..a7bbf26159dfb3eedaad84ce898f25ba7ef9fc8c 100755 (executable)
--- a/ext/spl/spl_directory.c
+++ b/ext/spl/spl_directory.c
@@ -376,6 +376,10 @@ static zend_object_value spl_filesystem_object_clone(zval *zobject TSRMLS_DC)
 void spl_filesystem_info_set_filename(spl_filesystem_object *intern, char *path, int len, int use_copy TSRMLS_DC) /* {{{ */
 {
        char *p1, *p2;
+       
+       if (intern->file_name) {
+               efree(intern->file_name);
+       }
 
        intern->file_name = use_copy ? estrndup(path, len) : path;
        intern->file_name_len = len;
@@ -396,7 +400,10 @@ void spl_filesystem_info_set_filename(spl_filesystem_object *intern, char *path,
        } else {
                intern->_path_len = 0;
        }
-
+       
+       if (intern->_path) {
+               efree(intern->_path);
+       }
        intern->_path = estrndup(path, intern->_path_len);
 } /* }}} */
 

diff --git a/ext/spl/tests/SplFileInfo_001.phpt b/ext/spl/tests/SplFileInfo_001.phpt
new file mode 100644 (file)
index 0000000..72060f0
--- /dev/null
+++ b/ext/spl/tests/SplFileInfo_001.phpt
@@ -0,0 +1,11 @@
+--TEST--
+Testing SplFileInfo calling the constructor twice
+--FILE--
+<?php
+$x = new splfileinfo(1);
+$x->__construct(1);
+
+echo "done!\n";
+?>
+--EXPECT--
+done!