Replace deprecated gethostbyname(3) with getaddrinfo(3)
AbandonedPublic
Actions

Authored by linimon on Sep 10 2019, 2:54 PM.

Tags

None

Referenced Files

	F83046191: D21585.id.diff
	Sun, May 5, 12:41 PM

	Unknown Object (File)
	Sat, May 4, 7:26 AM

	Unknown Object (File)
	Thu, May 2, 12:38 PM

	Unknown Object (File)
	Thu, May 2, 9:35 AM

	Unknown Object (File)
	Thu, May 2, 9:35 AM

	Unknown Object (File)
	Thu, May 2, 9:35 AM

	Unknown Object (File)
	Thu, May 2, 6:45 AM

	Unknown Object (File)
	Fri, Apr 19, 11:36 PM

View All 24 Files

Subscribers

imp

Details

Reviewers

luthramihir708_gmail.com

Summary

The commit just changes definition of getaddr() function which was using gethostbyname(3)
before and uses getaddrinfo(3) instead.

Test Plan

I ran a separate test program to confirm that the replacement produces the same effect.
I didn't test it with arp command itself because I am not that much familiar with it.

Diff Detail

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 26381
Build 24841: arc lint + arc unit

Event Timeline

luthramihir708_gmail.com created this revision.Sep 10 2019, 2:54 PM

Harbormaster completed remote builds in B26381: Diff 61885.Sep 10 2019, 2:54 PM

I have made some corrections in the above code.
It can be found on https://reviews.freebsd.org/D21589
Apologies for creating a new review request. I made this phabricator review through arc command line and got a little jumbled while trying to update it.
I will be careful from next time.

luthramihir708_gmail.com updated this revision to Diff 61923.Sep 11 2019, 4:29 AM

Herald added a subscriber: imp. · View Herald TranscriptSep 11 2019, 4:29 AM