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

uv_free_interface_addresses is not needed if uv_interface_addresses returns an error #20014

Merged

vtjnash merged 1 commit into master from jn/uv_free_interface_addresses

Jan 14, 2017

Sponsor Member

vtjnash commented Jan 13, 2017


          uv_free_interface_addresses is not needed if uv_interface_addresses r…

7eafe1b

…eturns an error

fix #20012

kshyatt added the domain:io label

Keno approved these changes

View reviewed changes

Contributor

tkelman commented Jan 14, 2017

the c version from before 627ed54 would only free if non-null, would that be worth doing?

vtjnash merged commit 496ca31 into master

vtjnash deleted the jn/uv_free_interface_addresses branch

January 14, 2017 23:36

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels