Skip to content

Fix closed flag from #6287 #6315

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.

Sign up for GitHub

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

Jump to bottom
Merged
scw00 merged 1 commit into from
Jan 9, 2020
Merged

Fix closed flag from #6287 #6315

scw00 merged 1 commit into from
Jan 9, 2020

Conversation

@scw00
Copy link
Member

@scw00 scw00 commented Jan 9, 2020 •
edited
Loading

closed flag should be int since do_io_close(-1) is valid.

introduce in #6287

@scw00 scw00 self-assigned this Jan 9, 2020
@scw00 scw00 added the Network label Jan 9, 2020
@scw00 scw00 added this to the 10.0.0 milestone Jan 9, 2020
@scw00
Copy link
Member Author

scw00 commented Jan 9, 2020

[approve ci autest]

1 similar comment
@oknet
Copy link
Member

oknet commented Jan 9, 2020

[approve ci autest]

@scw00 scw00 merged commit bd72eae into apache:master Jan 9, 2020
@scw00 scw00 deleted the fix_closed branch January 9, 2020 06:42
@zwoop
Copy link
Contributor

zwoop commented May 14, 2020

Cherry-picked to v9.0.x branch.

@zwoop zwoop modified the milestones: 10.0.0, 9.0.0 May 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@oknet oknet oknet approved these changes

Assignees

@scw00 scw00

Labels

Network

Projects

None yet

Milestone

9.0.0

Development

Successfully merging this pull request may close these issues.

3 participants

@scw00 @oknet @zwoop