Page MenuHomeFreeBSD
Differential D45601

ppp: remove gethostby*() calls
Needs ReviewPublic
Actions

Authored by yanhaowang on Jun 16 2024, 3:29 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 16, 4:55 PM
Unknown Object (File)
Sat, Dec 13, 2:20 AM
Unknown Object (File)
Fri, Nov 28, 9:05 AM
Unknown Object (File)
Fri, Nov 28, 7:35 AM
Unknown Object (File)
Thu, Nov 27, 1:11 PM
Unknown Object (File)
Nov 25 2025, 4:01 PM
Unknown Object (File)
Nov 19 2025, 9:35 AM
Unknown Object (File)
Nov 17 2025, 9:01 AM
View All Files
Subscribers
imp

Details

Reviewers
hrs
lwhsu
Summary
  1. Use getaddrinfo to replace gethostbyname, use for loop to loop the address list and copy the first meet ipv4 address.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 58542
Build 55430: arc lint + arc unit

Event Timeline

yanhaowang created this revision.Jun 16 2024, 3:29 AM
Herald added a subscriber: imp. · View Herald TranscriptJun 16 2024, 3:29 AM
yanhaowang requested review of this revision.Jun 16 2024, 3:29 AM
Harbormaster completed remote builds in B58186: Diff 139880.Jun 16 2024, 3:29 AM
yanhaowang added reviewers: hrs, lwhsu.Jun 16 2024, 3:34 AM
yanhaowang updated this revision to Diff 139883.Jun 16 2024, 7:42 AM
  • Remove unused variable
Harbormaster completed remote builds in B58187: Diff 139883.Jun 16 2024, 7:42 AM
yanhaowang updated this revision to Diff 140203.Jun 25 2024, 2:44 AM
  • Remove for loop
Harbormaster completed remote builds in B58341: Diff 140203.Jun 25 2024, 2:44 AM
yanhaowang updated this revision to Diff 140648.Jul 7 2024, 7:15 AM

Update according Hiroki advise in D45594

  1. Use "error" instead of "status" name to store the value of getaddrinfo() function.
  2. Use compound literals instead of "hints" variable.
Harbormaster completed remote builds in B58542: Diff 140648.Jul 7 2024, 7:15 AM
yanhaowang updated this revision to Diff 140654.Jul 7 2024, 10:49 AM

Remove unused variable

Harbormaster completed remote builds in B58547: Diff 140654.Jul 7 2024, 10:49 AM
yanhaowang updated this revision to Diff 140673.Jul 8 2024, 7:31 AM

Fix format

Harbormaster completed remote builds in B58558: Diff 140673.Jul 8 2024, 7:31 AM

Revision Contents
Changeset List

PathSize
usr.sbin/
ppp/
19 lines
14 lines
12 lines
18 lines

Diff 140648

View Options

usr.sbin/ppp/defs.c

Loading...
View Options

usr.sbin/ppp/ipcp.c

Loading...
View Options

usr.sbin/ppp/nat_cmd.c

Loading...
View Options

usr.sbin/ppp/radius.c

Loading...