Testcase for handle leak

Author: unknown

Diff for: Examples/HandleTest/HandleTestDaemon.php

@@ -0,0 +1,65 @@
+<?php
+require_once 'config.php';
+/**
+ * Test case for resource leaks on the automatic restart of the Daemon.
+ * Check for open files using 'lsof handledeamontest*.log'
+ * The main log will be open two times, one inherited from the first restart.
+ * The new logfile in the restarted process will get handle id 0, 
+ * that will be considered STDIN and be closed before the next restart. 
+ * Therefore we only see two open logfiles.
+ * 
+ * The amount of other open log files will graddaly increase. 
+ * 
+ */
+class HandleTestDeamon extends \Core_Daemon {
+    
+    /** use long interval to allow for restart.*/
+    protected $loop_interval = 10;
+
+    /**
+     * Exeucte method, fails on second try.
+     * @throws \Exception
+     */
+    protected function execute() {
+        static $count=0;
+        $count++;
+        $this->log("execute start $count");      
+        $this->openfiles();
+        //Need a minum time alive to force a restart.
+        if ($count>1) {
+            $this->log("execute fail $count");
+            throw new \Exception("FAIL");
+        }
+    }
+    
+    /**
+     * Open a set of files and store them in a static to cause a handle leak.
+     * @staticvar type $files
+     * @param type $max
+     */
+    protected function openfiles($max = 5)
+    {
+        static $files;
+        if (empty($files)) {
+            $this->log("opening files");
+            for ($i=0;$i<$max;$i++) {
+                $f = fopen("./handledeamontest".$i.".log", "a+");
+                fwrite($f, "test file pid=".getmypid()."\n");
+                fflush($f);
+                $files[]=$f;
+            }
+        }
+    }
+
+    protected function log_file() {
+        //Reuse the same logfile
+        return "./handledeamontest.log";
+    }
+
+    protected function setup() {     
+    }
+}
+
+HandleTestDeamon::getInstance()->run();
+
+

Diff for: Examples/HandleTest/config.php

@@ -0,0 +1,40 @@
+<?php
+
+/**
+ *
+ * Note: Nothing in this config file is specifically required to run a PHP Simple Daemon application. You can integrate it
+ * into an existing bootstrap if you want.
+ *
+ * Note: When using external tools (like crontab or process managers like supervisord) the working directory may be different
+ * than what you expect: Using relative paths and "./" based paths may not work as expected. You should always use absolute
+ * paths when accessing filesystem resources. Setting a constant like we do with BASE_PATH below can be helpful.
+ *
+ */
+
+date_default_timezone_set('America/Los_Angeles');
+
+// The custom error handlers that ship with PHP Simple Daemon respect all PHP INI error settings.
+ini_set('error_log', '/var/log/phpcli');
+ini_set('display_errors', 0);
+
+// Define a simple Auto Loader:
+// Add the current application and the PHP Simple Daemon ./Core library to the existing include path
+// Then set an __autoload function that uses Zend Framework naming conventions.
+define("BASE_PATH", dirname(__FILE__));
+set_include_path(implode(PATH_SEPARATOR, array(
+    realpath(BASE_PATH),
+    realpath(BASE_PATH . '/../../'),
+    realpath(BASE_PATH . '/../../Core'),
+    get_include_path(),
+)));
+
+function __autoload($class_name)
+{
+    $class_name = str_replace('\\', '/', $class_name);
+    $class_name = str_replace('_', '/', $class_name);
+    require_once "$class_name.php";
+}
+
+function pathify($class_name) {
+    return str_replace("_", "/", $class_name) . ".php";
+}