Page MenuHomeFreeBSD

Fix non-POSIX-compliant use of getaddrinfo in libfetch
ClosedPublic

Authored by grembo on Sep 23 2015, 2:42 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Nov 27, 10:53 AM
Unknown Object (File)
Nov 22 2024, 11:40 PM
Unknown Object (File)
Nov 22 2024, 7:29 AM
Unknown Object (File)
Nov 18 2024, 2:19 AM
Unknown Object (File)
Nov 17 2024, 4:02 PM
Unknown Object (File)
Nov 17 2024, 3:36 PM
Unknown Object (File)
Nov 17 2024, 2:28 PM
Unknown Object (File)
Nov 17 2024, 1:58 PM
Subscribers

Details

Summary

Boris Kolpackov brought this to my attention: libfetch uses
getaddrinfo in a non-compliant way, as it assumes that
res will be NULL when getaddrinfo fails. Even this
is true in reality right now, it's neither guaranteed by
POSIX, nor by the getaddrinfo man page. To prevent future
breakage and to be a good citizen, this patch aims
to fix this.

Test Plan

Build.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 591
Build 591: arc lint + arc unit

Event Timeline

grembo retitled this revision from to Fix non-POSIX-compliant use of getaddrinfo in libfetch.
grembo updated this object.
grembo edited the test plan for this revision. (Show Details)
grembo added reviewers: des, bapt.
bapt edited edge metadata.
This revision is now accepted and ready to land.Sep 23 2015, 10:03 PM
This revision was automatically updated to reflect the committed changes.