False positive: This expression has no effect (because _function with multiple implementations_ has no external side effects [in incorrectly selected version]).
- Lenguaje dominante
- CodeQL
- Estrellas
- 10.1k
- Forks
- 2.1k
- Merge medio
- 2 d 15 h
- PR fusionados (30 d)
- 141
Descripción
**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.
Guía de contribución
Línea de trabajo
Comienza comparando la llamada enlazada en pdns/dnsdistdist/dnsdist-udp.cc con las implementaciones de test-dnsdist_cc.cc y dnsdist.cc. Revisa la alerta de CodeQL relacionada y la discusión enlazada para determinar por qué se selecciona la implementación incorrecta. Se considera terminado cuando el falso positivo citado se haya resuelto para la llamada reportada y se haya verificado frente a esas ubicaciones del código fuente.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- cpp
- Área
- security
- Tipo de issue
- Error
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Necesita aclaración
- Aptitud para principiantes
- 35/100