False positive: This expression has no effect (because _function with multiple implementations_ has no external side effects [in incorrectly selected version]).
- Lingua principale
- CodeQL
- Stelle
- 10.1k
- Fork
- 2.1k
- Merge medio
- 2g 15h
- PR unite (30g)
- 141
Descrizione
**Description of the false positive**
https://github.com/PowerDNS/pdns/pull/16363#discussion_r2455671092
> This expression has no effect (because sendUDPResponse has no external side effects).
codeql appears to have found https://github.com/PowerDNS/pdns/blob/dec9583d885713a0d0ecb55a74ef83cde5f8a235/pdns/dnsdistdist/test-dnsdist_cc.cc#L83 (which is indeed side effect free) instead of https://github.com/PowerDNS/pdns/blob/dec9583d885713a0d0ecb55a74ef83cde5f8a235/pdns/dnsdistdist/dnsdist.cc#L637 (which shouldn't be)
**Code samples or links to source code**
https://github.com/pieterlexis/pdns/blob/064c4c9f2ea60c59d4b0c1a95daad4e17cbe2ce4/pdns/dnsdistdist/dnsdist-udp.cc#L131
https://github.com/PowerDNS/pdns/blob/dec9583d885713a0d0ecb55a74ef83cde5f8a235/pdns/dnsdistdist/test-dnsdist_cc.cc#L83
https://github.com/PowerDNS/pdns/blob/dec9583d885713a0d0ecb55a74ef83cde5f8a235/pdns/dnsdistdist/dnsdist.cc#L637
**URL to the alert on GitHub code scanning (optional)**
I don't have access to this, but I'll see if I can get someone to provide it.
Guida per i contributori
Apri la guida per i contributori
Direzione di ricerca
Inizia confrontando la chiamata collegata in pdns/dnsdistdist/dnsdist-udp.cc con le implementazioni in test-dnsdist_cc.cc e dnsdist.cc. Esamina l’avviso CodeQL correlato e la discussione collegata per determinare perché viene selezionata l’implementazione errata. Il lavoro è completato quando il falso positivo citato è stato risolto per la chiamata segnalata e verificato rispetto a quelle posizioni nel codice sorgente.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- cpp
- Ambito
- security
- Tipo di issue
- Bug
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Ferma
- Chiarezza
- Da chiarire
- Idoneità per principianti
- 35/100