Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

[core] Check if pattern dependencies or pattern positions are missing #14696

Merged
merged 1 commit into from
May 20, 2019

Conversation

alexshalamov
Copy link
Contributor

Add check for optional pattern dependencies and don't bind empty vertex buffers if pattern positions are missing.

Fixes: #14518

@alexshalamov alexshalamov marked this pull request as ready for review May 17, 2019 05:07
Add check for optional pattern dependencies and don't bind empty vertex
buffers if pattern positions are missing.
@alexshalamov alexshalamov merged commit 521ac5c into master May 20, 2019
@alexshalamov alexshalamov deleted the alexshalamov_fix_for_14518 branch May 20, 2019 08:19
@friedbunny friedbunny added Core The cross-platform C++ core, aka mbgl needs changelog Indicates PR needs a changelog entry prior to merging. rendering labels May 20, 2019
@friedbunny
Copy link
Contributor

friedbunny commented May 20, 2019

Thanks for tracking this down, @alexshalamov! Since this fixed a crash, it should receive an entry in the platform changelogs.

@friedbunny friedbunny added this to the release-o milestone May 20, 2019
@friedbunny friedbunny removed needs changelog Indicates PR needs a changelog entry prior to merging. labels Jun 19, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Core The cross-platform C++ core, aka mbgl crash rendering
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Crash at mbgl::gl::Program<mbgl::FillPatternProgram>::draw
5 participants