diff options
author | Robin Appelman <robin@icewind.nl> | 2018-09-28 15:40:08 +0200 |
---|---|---|
committer | Robin Appelman <robin@icewind.nl> | 2018-09-28 15:40:08 +0200 |
commit | f1f74a39b6ab830e6e70c5df93395442a4c4a886 (patch) | |
tree | d2615ea3ad76caa57ec9b8633d3c0410be95f251 /apps | |
parent | 02de67773c5ce1e38a90ce45b60ce249ed9a4042 (diff) | |
download | nextcloud-server-f1f74a39b6ab830e6e70c5df93395442a4c4a886.tar.gz nextcloud-server-f1f74a39b6ab830e6e70c5df93395442a4c4a886.zip |
throw an error if a node is smaller than expected in assemblystream
Signed-off-by: Robin Appelman <robin@icewind.nl>
Diffstat (limited to 'apps')
-rw-r--r-- | apps/dav/lib/Upload/AssemblyStream.php | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/apps/dav/lib/Upload/AssemblyStream.php b/apps/dav/lib/Upload/AssemblyStream.php index e20d8f9adba..1ab1a13cfce 100644 --- a/apps/dav/lib/Upload/AssemblyStream.php +++ b/apps/dav/lib/Upload/AssemblyStream.php @@ -57,6 +57,9 @@ class AssemblyStream implements \Icewind\Streams\File { /** @var int */ private $currentNode = 0; + /** @var int */ + private $currentNodeRead = 0; + /** * @param string $path * @param string $mode @@ -110,10 +113,16 @@ class AssemblyStream implements \Icewind\Streams\File { do { $data = fread($this->currentStream, $count); $read = strlen($data); + $this->currentNodeRead += $read; if (feof($this->currentStream)) { fclose($this->currentStream); + $currentNodeSize = $this->nodes[$this->currentNode]->getSize(); + if ($this->currentNodeRead < $currentNodeSize) { + throw new \Exception('Stream from assembly node shorter than expected, got ' . $this->currentNodeRead . ' bytes, expected ' . $currentNodeSize); + } $this->currentNode++; + $this->currentNodeRead = 0; if ($this->currentNode < count($this->nodes)) { $this->currentStream = $this->getStream($this->nodes[$this->currentNode]); } else { |