Skip to content

[stable34] fix(files_sharing): don't crash CleanupShareTarget when mount info is missing - #64368

Open
backportbot[bot] wants to merge 1 commit into
stable34from
backport/63495/stable34
Open

backportbot[bot] wants to merge 1 commit into
stable34from
backport/63495/stable34

Conversation

@backportbot

@backportbot backportbot Bot commented Sep 15, 2026

Copy link
Copy Markdown

Backport of PR #63495

… missing

The repair step "Cleanup share names with false conflicts" fatals with
"Call to a member function getRootId() on null" when a problem share's
old target has no entry in the recipient's cached mounts (e.g. the
recipient has not logged in since the share was created).

Guard the lookup the same way the generateUniqueTarget callback already
does and skip the mount cache refresh in that case - the share row is
already updated by moveShare() and the cache is rebuilt on the user's
next login.

Fixes #63494

Signed-off-by: Baki Burak Öğün <63836730+bakiburakogun@users.noreply.github.com>
@backportbot
backportbot Bot requested a review from a team as a code owner September 15, 2026 16:26
@backportbot
backportbot Bot requested review from Altahrim, CarlSchwan, DerDreschner, leftybournes, salmart-dev and susnux and removed request for a team September 15, 2026 16:26
@backportbot backportbot Bot added bug 3. to review Waiting for reviews feature: sharing feedback-requested community pull requests from community labels Sep 15, 2026
@backportbot backportbot Bot added this to the Nextcloud 34.0.5 milestone Sep 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants