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

Fix two leaks in the Mac platform code. #56

Merged
merged 1 commit into from Mar 24, 2016
Merged

Conversation

@pcwalton
Copy link
Collaborator

pcwalton commented Mar 24, 2016

Related: servo/servo#10029

r? @metajack

@@ -591,6 +591,8 @@ fn select(port: mach_port_t, blocking_mode: BlockingMode)
os_result => return Err(MachError(os_result)),

This comment has been minimized.

@metajack

metajack Mar 24, 2016

Can we leak here in the case that we get a bad os_result?

@metajack
Copy link

metajack commented Mar 24, 2016

Looks good aside from the one comment.

@pcwalton pcwalton force-pushed the pcwalton:mac-leak branch from 929929a to e876837 Mar 24, 2016
@pcwalton
Copy link
Collaborator Author

pcwalton commented Mar 24, 2016

Fixed. r? @metajack

@metajack
Copy link

metajack commented Mar 24, 2016

@bors-servo
Copy link
Contributor

bors-servo commented Mar 24, 2016

📌 Commit e876837 has been approved by metajack

@bors-servo
Copy link
Contributor

bors-servo commented Mar 24, 2016

Testing commit e876837 with merge dbe1874...

bors-servo added a commit that referenced this pull request Mar 24, 2016
Fix two leaks in the Mac platform code.

Related: servo/servo#10029

r? @metajack
@bors-servo
Copy link
Contributor

bors-servo commented Mar 24, 2016

☀️ Test successful - travis

@bors-servo bors-servo merged commit e876837 into servo:master Mar 24, 2016
2 of 3 checks passed
2 of 3 checks passed
continuous-integration/appveyor/pr Waiting for AppVeyor build to complete
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

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