BUG: set agc.reset.interval command parser does not accept full input range
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 67/100
Línea de trabajo
Comienza en src/helpers/CommonCLI.cpp, en el analizador agc_reset_interval enlazado en la incidencia, y luego rastrea cómo se almacena y comprueba el intervalo analizado. Reproduce el desbordamiento con entradas cercanas a 252 y 256, y confirma que el analizador acepta valores de hasta 1000 segundos sin desbordarse antes de la conversión.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
agc_reset_interval is a uint8, and stored divided by four. As such, it should accept a value up to 255*4=1000 seconds.
The CLI parser stores interval = atoi(string) / 4 at https://github.com/meshcore-dev/MeshCore/blob/0679dbeffc504d562d2f09eb072fdc223f8ffc2a/src/helpers/CommonCLI.cpp#L475. It appears the compiler sees the interval is a unit8 and invokes a version of atoi() that already returns a unit8. The stored value becomes (input mod 256) / 4, making 252 the largest input accepted. At 256, it rolls over to 0, and so on.
- Lenguaje dominante
- C
- Estrellas
- 3.9k
- Forks
- 1.5k
- Merge medio
- 18 h 59 min
- PR fusionados (30 d)
- 28
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de meshcore-dev/MeshCore
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 63/100
meshcore-dev/MeshCore#3622 ·
Los mantenedores suelen responder en 1 día
-
[Feature Request] Set Australia (Mid) Preset to automatically enable 2 byte default path hash sizeAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
meshcore-dev/MeshCore#3571 · 5 reacciones ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
meshcore-dev/MeshCore#3567 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
meshcore-dev/MeshCore#3549 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
meshcore-dev/MeshCore#3513 ·
Los mantenedores suelen responder en 1 día
Todos los issues de meshcore-dev/MeshCore
Issues similares
-
Missing options in yara man pageAbiertobug
Dificultad 1/5 Menos de una hora Aptitud para principiantes 84/100
VirusTotal/yara#2255 ·
-
[BUG] Filename sequence expansion (`+`) panics on numeric values exceeding `i32::MAX`Posiblemente ocupada @muntasir-396 la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
CCExtractor/ccextractor#2402 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
FujiNetWIFI/fujinet-lib#204 ·
Los mantenedores suelen responder en 1 día
-
os:linux
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
mpv-player/mpv#18649 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
facebookincubator/muse-gadget-sdk#249 ·
Los mantenedores suelen responder en 1 día