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

Guard against null pointers (which can exist if the size is zero) in #10

Merged
merged 1 commit into from Sep 30, 2015

Conversation

@pcwalton
Copy link
Collaborator

pcwalton commented Sep 30, 2015

MachSharedMemory::clone() and MachSharedMemory::deref().

Part of the fix for servo/servo#7422.

@pcwalton
Copy link
Collaborator Author

pcwalton commented Sep 30, 2015

`MachSharedMemory::clone()` and `MachSharedMemory::deref()`.

Part of the fix for servo/servo#7422.
@larsbergstrom
Copy link
Contributor

larsbergstrom commented Sep 30, 2015

r+

pcwalton added a commit that referenced this pull request Sep 30, 2015
Guard against null pointers (which can exist if the size is zero) in
@pcwalton pcwalton merged commit 5e71fe6 into master Sep 30, 2015
@pcwalton pcwalton deleted the mach-shared-memory-null-guard branch Sep 30, 2015
pcwalton referenced this pull request in pcwalton/servo Oct 1, 2015
incorporate the fix for zero-sized shmem on Mac.

Depends on servo/rust-azure#201 and pcwalton/ipc-channel#10.

Closes servo#7422.
bors-servo referenced this pull request in servo/servo Oct 1, 2015
Handle zero-sized source surfaces properly, and upgrade `ipc-channel` to incorporate the fix for zero-sized shmem on Mac.

Depends on servo/rust-azure#201 and pcwalton/ipc-channel#10.

Closes #7422.

r? @mbrubeck (don't r+ until the upstream dependencies have landed)

cc @jdm

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/7802)
<!-- Reviewable:end -->
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

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