Reconsider usages of `gethostbyname_r()`
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- C
- Sterne
- 40.4k
- Forks
- 8.1k
- Ø Merge
- 2 T. 13 Std.
- Gemergte PRs (30 T.)
- 96
Beschreibung
Description
According to some sources, this function is obsolete and should be replaced with:
Applications should use getaddrinfo(3), getnameinfo(3), and gai_strerror(3) instead.
Moreover, getting rid of them would allow us to get remove some conditionally defined globals:
https://github.com/php/php-src/blob/69d9c12df64e829befd843175bfc9617aabb7450/ext/standard/file.h#L106-L110
Initially found while working on #15511
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Durchsuche das Repository nach Verwendungen von gethostbyname_r() und lies ext/standard/file.h im Bereich der Zeilen 106–110 sowie den Kontext aus #15511. Vergleiche jede Verwendung mit getaddrinfo(), getnameinfo() und gai_strerror(); die Aufgabe ist abgeschlossen, wenn die veralteten Aufrufe und die nicht mehr benötigten bedingten globalen Variablen konsistent behandelt sind.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- c
- Bereich
- networking
- Issue-Typ
- Refactoring
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 45/100