Bug 1252037

Description of problem:
Horde::Util , public static function shutdown() tries to erase twice the same file.

Version-Release number of selected component (if applicable):


How reproducible:
Use horde libs

Steps to Reproduce:
1. Include horde libs
2. Call exit
3. Horde::Util is trying to erase twice tmp files

Actual results:
Errors

Expected results:
Clean exit

Additional info:

The code of function + temp solution.

{
  $ptr = &self::$_shutdowndata;

// $ptr contains files and dirs.
// Dirs and contains are erased in the first pass.
// But $ptr continues containing files already erased.
// In this case "is_file()" over non existent file return false, and tries to // iterate a non existent  dir. An error is printed.

        foreach (array_keys($ptr['paths']) as $val) {

 /* Temp Solution  */
if (!file_exists($val)) continue;
/* */

            if (@is_file($val)) {
                self::_secureDelete($val);
                continue;
            }

            try {
                $it = new RecursiveIteratorIterator(
                    new RecursiveDirectoryIterator($val),
                    RecursiveIteratorIterator::CHILD_FIRST
                );
            } catch (UnexpectedValueException $e) {
                continue;
            }

            while ($it->valid()) {
                if (!$it->isDot()) {
                    if ($it->isDir()) {
                        @rmdir($it->key());
                    } elseif ($it->isFile()) {
                        self::_secureDelete($it->key());
                    } else {
                        @unlink($it->key());
                    }
                }
                $it->next();
            }

            @rmdir($val);
        }
}