ext/phar: offsetExists should return false when file entry is not based on PharFileInfo

Author: Girgias

ext/phar/phar_object.c

@@ -3529,6 +3529,10 @@ PHP_METHOD(Phar, offsetExists)
 		}
 		RETURN_TRUE;
 	} else {
+		/* If the info class is not based on PharFileInfo, directories are not directly instantiable */
+		if (UNEXPECTED(!instanceof_function(phar_obj->spl.info_class, phar_ce_entry))) {
+			RETURN_FALSE;
+		}
 		RETURN_BOOL(zend_hash_exists(&phar_obj->archive->virtual_dirs, file_name));
 	}
 }

ext/phar/tests/phar_oo_011.phpt

@@ -43,13 +43,13 @@ __halt_compiler();
 bool(true)
 object(PharFileInfo)#%d (2) {
   ["pathName":"SplFileInfo":private]=>
-  string(%s) "phar:///%s/phar_oo_011.phar.php/hi"
+  string(%d) "phar://%s/phar_oo_011.phar.php/hi"
   ["fileName":"SplFileInfo":private]=>
   string(2) "hi"
 }
 bool(true)
-phar:///%s/phar_oo_011.phar.php/hi/f.php
-bool(true)
+phar://%s/phar_oo_011.phar.php/hi/f.php
+bool(false)
 LogicException: Cannot use SplFileObject with directories
 bool(true)
 hi