Now also preventing the user of "Shared" as mountpoint

This commit is contained in:
Vincent Petry 2013-11-22 18:36:27 +01:00
parent 476d8e6de0
commit 2d947835b9
2 changed files with 4 additions and 2 deletions

View File

@ -267,8 +267,8 @@ class OC_Mount_Config {
$applicable, $applicable,
$isPersonal = false) { $isPersonal = false) {
$mountPoint = OC\Files\Filesystem::normalizePath($mountPoint); $mountPoint = OC\Files\Filesystem::normalizePath($mountPoint);
if ($mountPoint === '' || $mountPoint === '/') { if ($mountPoint === '' || $mountPoint === '/' || $mountPoint == '/Shared') {
// can't mount at root // can't mount at root or "Shared" folder
return false; return false;
} }
if ($isPersonal) { if ($isPersonal) {

View File

@ -44,6 +44,8 @@ class Test_Mount_Config extends \PHPUnit_Framework_TestCase {
$isPersonal = false; $isPersonal = false;
$this->assertEquals(false, OC_Mount_Config::addMountPoint('', $storageClass, array(), $mountType, $applicable, $isPersonal)); $this->assertEquals(false, OC_Mount_Config::addMountPoint('', $storageClass, array(), $mountType, $applicable, $isPersonal));
$this->assertEquals(false, OC_Mount_Config::addMountPoint('/', $storageClass, array(), $mountType, $applicable, $isPersonal)); $this->assertEquals(false, OC_Mount_Config::addMountPoint('/', $storageClass, array(), $mountType, $applicable, $isPersonal));
$this->assertEquals(false, OC_Mount_Config::addMountPoint('Shared', $storageClass, array(), $mountType, $applicable, $isPersonal));
$this->assertEquals(false, OC_Mount_Config::addMountPoint('/Shared', $storageClass, array(), $mountType, $applicable, $isPersonal));
} }
} }