Merge pull request #36387 from nextcloud/backport/36210/stable25

[stable25] fix(dav): check if principal has email address before accessing it
nextcloud · Jan 27, 2023 · 18e0118 · 18e0118
2 parents 49a3df3 + f5a6a69
commit 18e0118
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
@@ -617,8 +617,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'])) {