Better skip condition check for session tests.

diff --git a/ext/session/tests/skipif.inc b/ext/session/tests/skipif.inc
index d952e73ca8a6e449e3d6ba62d120455da931a143..6e3eae0862c42bd8f545db2b7d055c08189dfbc2 100644 (file)
--- a/ext/session/tests/skipif.inc
+++ b/ext/session/tests/skipif.inc
@@ -7,16 +7,19 @@ if (!extension_loaded("session") && ini_get("enable_dl")) {
 if (!extension_loaded("session")) {
     die("skip Session module not loaded");
 }
-if (!file_exists(ini_get("session.save_path"))) {
-       die("skip Session save_path doesn't exist");
-}
 $save_path = ini_get("session.save_path");
-if (!@is_writable($save_path)) {
-       if (($p = strpos($save_path, ';')) !== false) {
-               $save_path = substr($save_path, ++$p);
+if ($save_path) {
+       if (!file_exists($save_path)) {
+               die("skip Session save_path doesn't exist");
        }
-       if (!@is_writable($save_path)) {
-               die("skip\n");
+
+       if ($save_path && !@is_writable($save_path)) {
+               if (($p = strpos($save_path, ';')) !== false) {
+                       $save_path = substr($save_path, ++$p);
+               }
+               if (!@is_writable($save_path)) {
+                       die("skip\n");
+               }
        }
 }
 ?>