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
F160196073: D45601.id140648.diff
Mon, Jun 22, 2:14 AM
Unknown Object (File)
Sun, Jun 21, 2:19 AM
Unknown Object (File)
Thu, Jun 18, 1:02 PM
Unknown Object (File)
Sat, Jun 6, 9:45 AM
Unknown Object (File)
May 17 2026, 4:48 PM
Unknown Object (File)
May 17 2026, 4:48 PM
Unknown Object (File)
May 17 2026, 2:07 PM
Unknown Object (File)
May 17 2026, 12:26 PM
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 58187
Build 55075: 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/
27 lines
20 lines
22 lines
34 lines

Diff 139883

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...