php / php/php-src

Reconsider usages of `gethostbyname_r()`

Aperta
#15,531 7 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Category: Engine Refactoring
Lingua principale
C
Stelle
40.4k
Fork
8.1k
Merge medio
2g 13h
PR unite (30g)
96

Descrizione

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

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Direzione di ricerca

Cerca nel repository gli utilizzi di gethostbyname_r() e leggi ext/standard/file.h intorno alle righe 106-110, insieme al contesto di #15511. Confronta ogni utilizzo con getaddrinfo(), getnameinfo() e gai_strerror(); il lavoro è completo quando le chiamate obsolete e le variabili globali condizionali non più necessarie sono gestite in modo coerente.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
c
Ambito
networking
Tipo di issue
Refactoring
Difficoltà
4/5
Tempo stimato
3-5 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
45/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.