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

msg/async: remove useless close function #13286

Merged
merged 1 commit into from Feb 8, 2017

Conversation

Projects
None yet
2 participants
@Liuchang0812
Copy link
Contributor

Liuchang0812 commented Feb 7, 2017

fix coverity cid 1395314: it's invalid to pass a negative value to close function, and we have closed socket in net.connect/nonblock_connect in error cases indeed

Signed-off-by: liuchang0812 liuchang0812@gmail.com

@yuyuyu101 Please take a look, thanks

msg/async: remove useless close function
fix coverity cid 1395314: it's invalid to pass a negative value to close function, and we have closed socket in net.connect/nonblock_connect in error cases indeed

Signed-off-by: liuchang0812 <liuchang0812@gmail.com>

@tchaikov tchaikov self-assigned this Feb 7, 2017

@tchaikov tchaikov merged commit 75434ff into ceph:master Feb 8, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details
@tchaikov

This comment has been minimized.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.