dont throw errors on double close in stream wrapper

icewind1991 · Mar 12, 2021 · 93bce47 · 93bce47
1 parent 22ef9fc
commit 93bce47
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/src/Wrapper.php b/src/Wrapper.php
@@ -92,7 +92,9 @@ public function stream_eof() {
 	}
 
 	public function stream_close() {
-		return fclose($this->source);
+		if (is_resource($this->source)) {
+			return fclose($this->source);
+		}
 	}
 
 	public function dir_readdir() {

diff --git a/tests/WrapperTest.php b/tests/WrapperTest.php
@@ -140,4 +140,15 @@ public function testRewindDir() {
 		rewinddir($wrapped);
 		$this->assertSame($content[0], readdir($wrapped));
 	}
+
+	public function testDoubleClose() {
+		$source = fopen('php://temp', 'r+');
+		rewind($source);
+
+		$wrapped = $this->wrapSource($source);
+
+		fclose($source);
+		fclose($wrapped);
+		$this->assertFalse(is_resource($source));
+	}
 }