aboutsummaryrefslogtreecommitdiffstats
path: root/apps
diff options
context:
space:
mode:
authorRobin Appelman <robin@icewind.nl>2018-09-28 15:40:08 +0200
committerRobin Appelman <robin@icewind.nl>2018-09-28 15:40:08 +0200
commitf1f74a39b6ab830e6e70c5df93395442a4c4a886 (patch)
treed2615ea3ad76caa57ec9b8633d3c0410be95f251 /apps
parent02de67773c5ce1e38a90ce45b60ce249ed9a4042 (diff)
downloadnextcloud-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.php9
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 {