Skip to content
This repository was archived by the owner on Jun 30, 2025. It is now read-only.

Don't call close(2) more than once #508

Merged
shinh merged 1 commit into from
Dec 24, 2019
Merged

Don't call close(2) more than once #508

shinh merged 1 commit into from
Dec 24, 2019

Conversation

@xiaosuo
Copy link
Contributor

@xiaosuo xiaosuo commented Dec 21, 2019

Don't call close(2) more then once, even though close(2) returns -1.

See the manual page for details.

@xiaosuo
Don't call close(2) more than once
a35e612 
Don't call close(2) more then once, even though close(2) returns -1.

See the manual page for details.
shinh
shinh approved these changes Dec 24, 2019
View reviewed changes
Copy link
Contributor

@shinh shinh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@shinh shinh merged commit 445af7e into google:master Dec 24, 2019
@xiaosuo xiaosuo deleted the close branch January 2, 2020 03:36
@sergiud sergiud added this to the 0.5 milestone Mar 30, 2021
@sergiud sergiud mentioned this pull request May 6, 2021
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@shinh shinh shinh approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

cla: yes

Projects

None yet

Milestone

0.5

Development

Successfully merging this pull request may close these issues.

4 participants

@xiaosuo @shinh @sergiud @googlebot