Merge pull request #50448 from nextcloud/backport/50447/stable30

[stable30] fix: Ensure `label` is always a string

Author: AndyScherzinger

apps/files_sharing/lib/Controller/ShareAPIController.php

@@ -733,7 +733,10 @@ public function createShare(
 			}
 
 			// If we have a label, use it
-			if (!empty($label)) {
+			if ($label !== '') {
+				if (strlen($label) > 255) {
+					throw new OCSBadRequestException('Maximum label length is 255');
+				}
 				$share->setLabel($label);
 			}
 

apps/sharebymail/lib/ShareByMailProvider.php

@@ -1005,7 +1005,7 @@ protected function createShareObject(array $data): IShare {
 		$share->setPassword($data['password']);
 		$passwordExpirationTime = \DateTime::createFromFormat('Y-m-d H:i:s', $data['password_expiration_time'] ?? '');
 		$share->setPasswordExpirationTime($passwordExpirationTime !== false ? $passwordExpirationTime : null);
-		$share->setLabel($data['label']);
+		$share->setLabel($data['label'] ?? '');
 		$share->setSendPasswordByTalk((bool)$data['password_by_talk']);
 		$share->setHideDownload((bool)$data['hide_download']);
 

lib/private/Share20/Share.php

@@ -66,13 +66,12 @@ class Share implements IShare {
 	private $shareTime;
 	/** @var bool */
 	private $mailSend;
-	/** @var string */
-	private $label = '';
 	/** @var ICacheEntry|null */
 	private $nodeCacheEntry;
 	/** @var bool */
 	private $hideDownload = false;
 
+	private string $label = '';
 	private bool $noExpirationDate = false;
 
 	public function __construct(