Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cull primitives with invalid dimensions, to avoid overflow later. #1768

Merged
merged 1 commit into from Sep 29, 2017

Conversation

@glennw
Copy link
Member

glennw commented Sep 28, 2017

Primitives with invalid dimensions can cause overflow and general
badness in the texture and render task allocators. Detect them
and ensure they are not considered visible.

Fixes servo/servo#18655


This change is Reviewable

Primitives with invalid dimensions can cause overflow and general
badness in the texture and render task allocators. Detect them
and ensure they are not considered visible.

Fixes servo/servo#18655
@glennw
Copy link
Member Author

glennw commented Sep 28, 2017

r? @nical or anyone

@nical
nical approved these changes Sep 28, 2017
Copy link
Collaborator

nical left a comment

@glennw
Copy link
Member Author

glennw commented Sep 28, 2017

@bors-servo r=nical

@bors-servo
Copy link
Contributor

bors-servo commented Sep 28, 2017

📌 Commit 1f27f8e has been approved by nical

@bors-servo
Copy link
Contributor

bors-servo commented Sep 28, 2017

Testing commit 1f27f8e with merge e4c846a...

bors-servo added a commit that referenced this pull request Sep 28, 2017
Cull primitives with invalid dimensions, to avoid overflow later.

Primitives with invalid dimensions can cause overflow and general
badness in the texture and render task allocators. Detect them
and ensure they are not considered visible.

Fixes servo/servo#18655

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/webrender/1768)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Sep 29, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: nical
Pushing e4c846a to master...

@bors-servo bors-servo merged commit 1f27f8e into servo:master Sep 29, 2017
4 checks passed
4 checks passed
Taskcluster (pull_request) TaskGroup: success
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.