Skip to content

Commit 827269f

Browse files
authored
Merge pull request #43764 from nextcloud/fix/user_ldap-ignore-unserialize-errors-in-migration
fix(user_ldap): Ignore unserialize error in group membership migration
2 parents 4a23308 + 875d0a2 commit 827269f

1 file changed

Lines changed: 4 additions & 0 deletions

File tree

apps/user_ldap/lib/Migration/Version1190Date20230706134108.php

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -99,6 +99,10 @@ protected function copyGroupMembershipData(): void {
9999
$result = $query->executeQuery();
100100
while ($row = $result->fetch()) {
101101
$knownUsers = unserialize($row['owncloudusers']);
102+
if (!is_array($knownUsers)) {
103+
/* Unserialize failed or data was incorrect in database, ignore */
104+
continue;
105+
}
102106
$knownUsers = array_unique($knownUsers);
103107
foreach ($knownUsers as $knownUser) {
104108
try {

0 commit comments

Comments
 (0)