check fstat result

zendframework · Feb 5, 2018 · 7c44b5a · 7c44b5a
1 parent b57ec28
commit 7c44b5a
Show file tree

Hide file tree

Showing 2 changed files with 26 additions and 1 deletion.
diff --git a/src/Stream.php b/src/Stream.php
@@ -103,7 +103,11 @@ public function getSize()
         }
 
         $stats = fstat($this->resource);
-        return $stats['size'];
+        if ($stats !== false) {
+            return $stats['size'];
+        }
+
+        return null;
     }
 
     /**

diff --git a/test/StreamTest.php b/test/StreamTest.php
@@ -632,4 +632,25 @@ public function testCanReadContentFromNotSeekableResource()
 
         $this->assertSame('FOO BAR', $stream->__toString());
     }
+
+    /**
+     * @group 42
+     */
+    public function testSizeReportsNullForPhpInputStreams()
+    {
+        $resource = fopen('php://input', 'r');
+        $stream = new Stream($resource);
+        $this->assertNull($stream->getSize());
+    }
+
+    /**
+     * @group 42
+     */
+    public function testSizeReportsNullForRemoteResources()
+    {
+        $resource_url = 'http://www.php.net/images/logos/php-logo.png';
+        $resource = fopen($resource_url, 'r');
+        $stream = new Stream($resource);
+        $this->assertNull($stream->getSize());
+    }
 }