Skip to content

[stable-34.0] fix(discovery): use QString to avoid dangling QStringBuilder. - #10557

Open
backportbot[bot] wants to merge 1 commit into
stable-34.0from
backport/10550/stable-34.0
Open

[stable-34.0] fix(discovery): use QString to avoid dangling QStringBuilder.#10557
backportbot[bot] wants to merge 1 commit into
stable-34.0from
backport/10550/stable-34.0

Conversation

@backportbot

@backportbot backportbot Bot commented Aug 8, 2026

Copy link
Copy Markdown

Backport of PR #10550

Fix for #10535.

auto on a QString concat keeps a QStringBuilder referencing destroyed
temporaries, giving a garbage size allocation that crashes on Qt 6.4.2.

Assisted-by: Claude Code:claude-opus-4-8
Signed-off-by: Camila Ayres <hello@camilasan.com>
@backportbot
backportbot Bot requested review from camilasan and i2h3 as code owners August 8, 2026 09:43
@backportbot
backportbot Bot requested a review from claucambra August 8, 2026 09:43
@backportbot backportbot Bot added this to the 34.0.2 milestone Aug 8, 2026
@github-actions

github-actions Bot commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

Artifact containing the AppImage: nextcloud-appimage-pr-10557.zip

Digest: sha256:423919a49e347643cdd73ed6842976a425f54f549fa924357d02c3e846b9ab2e

To test this change/fix you can download the above artifact file, unzip it, and run it.

Please make sure to quit your existing Nextcloud app and backup your data.

@sonarqubecloud

sonarqubecloud Bot commented Aug 8, 2026

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants