extern/egroupware
1
0
Fork 1
mirror of https://github.com/EGroupware/egroupware.git synced 2025-01-31 02:21:40 +01:00
Code Issues Packages Projects Releases Wiki Activity

Make sure replace only and only VT char (Vertical Tab)

Browse Source
This commit is contained in:
Hadi Nategh 2018-03-02 12:13:13 +01:00
parent 4ada67ee81
commit a987ddbe85
2 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
api/src/Mail.php
View File

@ -6443,7 +6443,7 @@ class Mail
*/ */
static function clean_subject_for_filename($filename) static function clean_subject_for_filename($filename)
{ {
static $filter_pattern = '$[\f\n\t\v\\:*#?<>%"\|/\\\?]$'; static $filter_pattern = '$[\f\n\t\x0b\:*#?<>%"\|/\\\?]$';
return preg_replace($filter_pattern, "_", trim($filename)); return preg_replace($filter_pattern, "_", trim($filename));
} }

2
mail/tests/SaveToVfsTest.php
View File

@ -65,6 +65,7 @@ class SaveToVfsTest extends \EGroupware\Api\AppTest
$this->assertNotContains('*', $cleaned); $this->assertNotContains('*', $cleaned);
$this->assertNotContains('/', $cleaned); $this->assertNotContains('/', $cleaned);
$this->assertNotContains('?', $cleaned); $this->assertNotContains('?', $cleaned);
$this->assertNotContains('\x0b', $cleaned);
// Length should stay the same // Length should stay the same
$this->assertEquals(strlen($filename), strlen($cleaned), 'Length changed'); $this->assertEquals(strlen($filename), strlen($cleaned), 'Length changed');
@ -94,6 +95,7 @@ class SaveToVfsTest extends \EGroupware\Api\AppTest
array('Contains a /', true), array('Contains a /', true),
array('Contains a ?', true), array('Contains a ?', true),
array('Contains a %', true), array('Contains a %', true),
array('Contains a '.chr(11), true),
array('This one contains them all < > " : | \ * / ? % are not allowed', true) array('This one contains them all < > " : | \ * / ? % are not allowed', true)
); );
} }