Skip to content

Commit

Permalink
Merge pull request #454 from kartolo/hotfix/2023110110000013_v6
Browse files Browse the repository at this point in the history
[Security] prevent user to input multiline value
  • Loading branch information
kartolo authored Dec 12, 2023
2 parents fb06570 + 1c69f4b commit c19d755
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 1 deletion.
8 changes: 8 additions & 0 deletions Classes/DirectMailUtility.php
Original file line number Diff line number Diff line change
Expand Up @@ -1625,6 +1625,14 @@ public static function updatePagesTSconfig($id, array $pageTs, $tsConfPrefix, $i
}
$set = array();
foreach ($pageTs as $f => $v) {
// only get the first line of input and ignore the rest
$v = strtok(trim($v), "\r\n");
// if token is not found (false)
if ($v === false) {
// then set empty string
$v = '';
}

$f = $tsConfPrefix . $f;
if ((!isset($impParams[$f]) && trim($v)) || strcmp(trim($impParams[$f]), trim($v))) {
$set[$f] = trim($v);
Expand Down
2 changes: 1 addition & 1 deletion ext_emconf.php
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
'description' => 'Advanced Direct Mail/Newsletter mailer system with sophisticated options for personalization of emails including response statistics.',
'category' => 'module',
'shy' => 0,
'version' => '6.0.2',
'version' => '6.0.3',
'dependencies' => 'cms,tt_address',
'conflicts' => 'sr_direct_mail_ext,it_dmail_fix,plugin_mgm,direct_mail_123',
'priority' => '',
Expand Down

0 comments on commit c19d755

Please sign in to comment.