summaryrefslogtreecommitdiffstats
path: root/apps/dav/lib/CalDAV/Reminder
diff options
context:
space:
mode:
authorRoeland Jago Douma <roeland@famdouma.nl>2021-02-13 15:27:11 +0100
committerRoeland Jago Douma <roeland@famdouma.nl>2021-02-15 10:07:13 +0100
commit507d5cf8ad79242ee309c18499f27ed5a81de59a (patch)
treeb9bd53d2469056f165e146dff74cc9e0ca66e85c /apps/dav/lib/CalDAV/Reminder
parent84de7a8d6356c4222f46b7b8139a4b6ac5f1aa96 (diff)
downloadnextcloud-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.php2
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;
}