Merge pull request #36210 from nextcloud/fix/4811/principal-email

fix(dav): check if principal has email address before accessing it
nextcloud · Jan 26, 2023 · 0a54817 · 0a54817
2 parents 033b03d + bbda231
commit 0a54817
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'])) {