aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRobin Appelman <robin@icewind.nl>2024-11-14 16:01:36 +0100
committerbackportbot[bot] <backportbot[bot]@users.noreply.github.com>2024-12-04 17:25:51 +0000
commit8f76edf64ab0424d289dbb853e0081659feeb058 (patch)
treef2dc16597b1a5f041a1ebeff277446ceb4d9b733
parentfc278e15cbbf9592bb3860f985a4c5c44ac007a3 (diff)
downloadnextcloud-server-backport/49288/stable29.tar.gz
nextcloud-server-backport/49288/stable29.zip
fix: smb: don't fail hard if we can't load acls for a filebackport/49288/stable29
Signed-off-by: Robin Appelman <robin@icewind.nl>
-rw-r--r--apps/files_external/lib/Lib/Storage/SMB.php7
1 files changed, 6 insertions, 1 deletions
diff --git a/apps/files_external/lib/Lib/Storage/SMB.php b/apps/files_external/lib/Lib/Storage/SMB.php
index 9330628d5ca..fe1d415e376 100644
--- a/apps/files_external/lib/Lib/Storage/SMB.php
+++ b/apps/files_external/lib/Lib/Storage/SMB.php
@@ -235,7 +235,12 @@ class SMB extends Common implements INotifyStorage {
* @return ACL|null
*/
private function getACL(IFileInfo $file): ?ACL {
- $acls = $file->getAcls();
+ try {
+ $acls = $file->getAcls();
+ } catch (Exception $e) {
+ $this->logger->error('Error while getting file acls', ['exception' => $e]);
+ return null;
+ }
foreach ($acls as $user => $acl) {
[, $user] = $this->splitUser($user); // strip domain
if ($user === $this->server->getAuth()->getUsername()) {