Use Buffer slice will have wrong result when end is larger than int32.
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- javascript, nodejs
- Área
- backend
Línea de trabajo
Reproduce el caso reportado Buffer.from([1,2,3]).slice(1, Math.pow(2,32)), luego sigue la implementación de Buffer.slice donde los offsets usan conversión bit a bit. Añade una prueba de regresión que cubra el valor final más grande junto con los ejemplos normales de slice, y confirma que el resultado incluye [2,3].
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
[1,2,3].slice(1,3) return [2, 3], it's ok.
Array.from( Buffer.from([1,2,3]).slice(1,3) ) return [2,3], it's ok.
Array.from( Buffer.from([1,2,3]).slice(1,Math.pow(2,32)) ) return [] , it's wrong result.
In node Buffer, there is comment:
// Use Math.trunc() to convert offset to an integer value that can be larger
// than an Int32. Hence, don't use offset | 0 or similar techniques.
In this package, Buffer use ~, can't work with larger than int32 number.
- Lenguaje dominante
- JavaScript
- Estrellas
- 1.9k
- Forks
- 260
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Aún no hemos revisado los archivos de configuración de este proyecto. 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 feross/buffer
-
React Native, unordered_map::at: key not found when passing a Buffer instead of a Uint8ArrayAbierto
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
-
Use `DataView`, remove dependency on `ieee754`Quizá libre de nuevo @dcousens la tomó hace 626 días y no hay ningún pull request abierto. Abierto
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
-
Expose `Buffer#utf8Slice`Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 48/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 38/100
Todos los issues de feross/buffer
Issues similares
-
bug good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
amponce/archive-movie-browser#354 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
saayam-for-all/webapp#1870 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Imageomics/OpenCite#66 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 64/100
chr15m/twiiit.com#20 ·
-
documentation
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
antropos17/Aegis#629 ·
Los mantenedores suelen responder en 4 días