diff options
author | Roeland Jago Douma <roeland@famdouma.nl> | 2021-02-13 15:27:11 +0100 |
---|---|---|
committer | Roeland Jago Douma <roeland@famdouma.nl> | 2021-02-15 10:07:13 +0100 |
commit | 507d5cf8ad79242ee309c18499f27ed5a81de59a (patch) | |
tree | b9bd53d2469056f165e146dff74cc9e0ca66e85c /apps/dav/lib/CalDAV/Reminder | |
parent | 84de7a8d6356c4222f46b7b8139a4b6ac5f1aa96 (diff) | |
download | nextcloud-server-507d5cf8ad79242ee309c18499f27ed5a81de59a.tar.gz nextcloud-server-507d5cf8ad79242ee309c18499f27ed5a81de59a.zip |
Check substr results
Else this might lead to unexpeted errors.
Found by psalm.
Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
Diffstat (limited to 'apps/dav/lib/CalDAV/Reminder')
-rw-r--r-- | apps/dav/lib/CalDAV/Reminder/NotificationProvider/EmailProvider.php | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/apps/dav/lib/CalDAV/Reminder/NotificationProvider/EmailProvider.php b/apps/dav/lib/CalDAV/Reminder/NotificationProvider/EmailProvider.php index fbb52c45d38..38c875315bf 100644 --- a/apps/dav/lib/CalDAV/Reminder/NotificationProvider/EmailProvider.php +++ b/apps/dav/lib/CalDAV/Reminder/NotificationProvider/EmailProvider.php @@ -202,7 +202,7 @@ class EmailProvider extends AbstractProvider { $organizerEMail = substr($organizer->getValue(), 7); - if (!$this->mailer->validateMailAddress($organizerEMail)) { + if ($organizerEMail === false || !$this->mailer->validateMailAddress($organizerEMail)) { return null; } |