Skip to content

Fix crash on vector tile style parsing - #66533

Merged
nyalldawson merged 1 commit into
qgis:masterfrom
elpaso:bugfix-gh_66466-vectortiles-crash-on-style
Jun 25, 2026
Merged

Fix crash on vector tile style parsing#66533
nyalldawson merged 1 commit into
qgis:masterfrom
elpaso:bugfix-gh_66466-vectortiles-crash-on-style

Conversation

@elpaso

@elpaso elpaso commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Fix #66466

@elpaso elpaso added Bug Either a bug report, or a bug fix. Let's hope for the latter! backport queued_ltr_backports Queued Backports labels Jun 23, 2026
@github-actions github-actions Bot added this to the 4.2.0 milestone Jun 23, 2026
@nirvn

nirvn commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

@elpaso , do you feel it'd be worth adding a minimal test to cover this? I've approved as is, I'll let you decide 😉

@elpaso

elpaso commented Jun 24, 2026

Copy link
Copy Markdown
Contributor Author

@elpaso , do you feel it'd be worth adding a minimal test to cover this? I've approved as is, I'll let you decide 😉

Honestly: it's just an obvious array range check, I wouldn't waste time on a test since there is no risk of someone reverting (besides that I'm not that familiar with the vector style styles).

@nyalldawson
nyalldawson merged commit 92c7450 into qgis:master Jun 25, 2026
39 of 40 checks passed
@qgis-bot

Copy link
Copy Markdown
Collaborator

The backport to queued_ltr_backports failed:

The process '/usr/bin/git' failed with exit code 1
stderr
error: could not apply 048943a7d75... Fix crash on vector tile style parsing
hint: After resolving the conflicts, mark them with
hint: "git add/rm <pathspec>", then run
hint: "git cherry-pick --continue".
hint: You can instead skip this commit with "git cherry-pick --skip".
hint: To abort and get back to the state before "git cherry-pick",
hint: run "git cherry-pick --abort".
hint: Disable this message with "git config set advice.mergeConflict false"

stdout
Auto-merging src/core/vectortile/qgsmapboxglstyleconverter.cpp
CONFLICT (content): Merge conflict in src/core/vectortile/qgsmapboxglstyleconverter.cpp

To backport manually, run these commands in your terminal:

# Fetch latest updates from GitHub
git fetch
# Create a new working tree
git worktree add .worktrees/backport-queued_ltr_backports queued_ltr_backports
# Navigate to the new working tree
cd .worktrees/backport-queued_ltr_backports
# Create a new branch
git switch --create backport-66533-to-queued_ltr_backports
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick 048943a7d750a401485e2434ade4d2281e83317a
# Push it to GitHub
git push --set-upstream origin backport-66533-to-queued_ltr_backports
# Go back to the original working tree
cd ../..
# Delete the working tree
git worktree remove .worktrees/backport-queued_ltr_backports

Then, create a pull request where the base branch is queued_ltr_backports and the compare/head branch is backport-66533-to-queued_ltr_backports.

@qgis-bot qgis-bot added the failed backport The automated backport attempt failed, needs a manual backport label Jun 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport queued_ltr_backports Queued Backports Bug Either a bug report, or a bug fix. Let's hope for the latter! failed backport The automated backport attempt failed, needs a manual backport Vector tiles

4 participants