explicitly close source stream on local storage

Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>

Author: kesselb

lib/private/Files/Storage/Local.php

@@ -568,6 +568,9 @@ public function moveFromStorage(IStorage $sourceStorage, $sourceInternalPath, $t
 
 	public function writeStream(string $path, $stream, int $size = null): int {
 		$result = $this->file_put_contents($path, $stream);
+		if (is_resource($stream)) {
+			fclose($stream);
+		}
 		if ($result === false) {
 			throw new GenericFileException("Failed write steam to $path");
 		} else {

tests/lib/Files/Storage/Storage.php

@@ -658,6 +658,7 @@ public function testWriteStream() {
 
 		$storage->writeStream('test.txt', $source);
 		$this->assertTrue($storage->file_exists('test.txt'));
-		$this->assertEquals(file_get_contents($textFile), $storage->file_get_contents('test.txt'));
+		$this->assertStringEqualsFile($textFile, $storage->file_get_contents('test.txt'));
+		$this->assertEquals('resource (closed)', gettype($source));
 	}
 }