envoyproxy/envoy

dns: use handles to prevent use-after-free problems between dns queries and callback targets

Open

#10.079 aperta il 17 feb 2020

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)C++ (5373 fork)batch import
area/dnsenhancementhelp wanted

Metriche repository

Star
 (27.997 star)
Metriche merge PR
 (Merge medio 8g) (378 PR mergiate in 30 g)

Descrizione

Description: ActiveDnsQueries have independent lifetimes from their callback targets (strict/logical dns cluster, dns cache). Right now there is an implicit contract where the callback target sets their query pointer to null once the query is resolved, and the query only fires callbacks if it has not been cancelled. However, this implicit contract can create problems when it's not adhered to (e.g #10006).

An enhancement would be to use handles with RAII constraints similar to how the init manager does with watchers and targets.

Guida contributor

Direzione di ricerca
Studia il ciclo di vita esistente di ActiveDnsQueries e il pattern handle del gestore di inizializzazione, quindi implementa handle RAII per le query DNS per prevenire use after free.
Tech stack
cpp
Dominio
backendapi
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++Envoy internals
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
35