Merge pull request #44030 from nextcloud/backport/43764/stable28

[stable28] fix(user_ldap): Ignore unserialize error in group membership migration
nextcloud · Mar 6, 2024 · d046bf6 · d046bf6
2 parents 3991365 + 17173d3
commit d046bf6
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/apps/user_ldap/lib/Migration/Version1190Date20230706134108.php b/apps/user_ldap/lib/Migration/Version1190Date20230706134108.php
@@ -99,6 +99,10 @@ protected function copyGroupMembershipData(): void {
 		$result = $query->executeQuery();
 		while ($row = $result->fetch()) {
 			$knownUsers = unserialize($row['owncloudusers']);
+			if (!is_array($knownUsers)) {
+				/* Unserialize failed or data was incorrect in database, ignore */
+				continue;
+			}
 			$knownUsers = array_unique($knownUsers);
 			foreach ($knownUsers as $knownUser) {
 				try {