fix(dav): check if principal has email address before accessing it

Ref nextcloud/calendar#4811 Signed-off-by: Richard Steinmetz <richard@steinmetz.cloud>
nextcloud · Jan 26, 2023 · 6728b4b · 6728b4b
1 parent fb3261a
commit 6728b4b
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/apps/dav/lib/Connector/Sabre/Principal.php b/apps/dav/lib/Connector/Sabre/Principal.php
@@ -601,8 +601,8 @@ public function getCircleMembership($principal):array {
 	public function getEmailAddressesOfPrincipal(array $principal): array {
 		$emailAddresses = [];
 
-		if (($primaryAddress = $principal['{http://sabredav.org/ns}email-address'])) {
-			$emailAddresses[] = $primaryAddress;
+		if (isset($principal['{http://sabredav.org/ns}email-address'])) {
+			$emailAddresses[] = $principal['{http://sabredav.org/ns}email-address'];
 		}
 
 		if (isset($principal['{DAV:}alternate-URI-set'])) {