locallibrary: an M4A claiming a huge sample count aborts mix, check or the player
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 47/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- rust
- Área
- audio-video-rtc, backend
Línea de trabajo
Start with src/atoms/stsz.rs in symphonia-format-isomp4 0.5.5 and the dependency constraint in crates/loclib-analysis/Cargo.toml. Reproduce with an M4A whose stsz sample count is 4294967295 and sample size is 0, then check whether symphonia 0.6.1 bounds the allocation. Done when the malformed file is reported as unreadable without aborting mix, check, or the player.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
What happened?
symphonia-format-isomp4 0.5.5 (locked) sizes a Vec from the sample count in the stsz atom with Vec::with_capacity(sample_count as usize) under // TODO: Apply a limit. (src/atoms/stsz.rs, line 44). An M4A claiming a count near u32::MAX asks for about 16 GiB, which aborts the process on Linux without overcommit and on Windows. mix, check and the player decode M4A files through symphonia. Listed in https://github.com/cutenode/localnetwork/blob/24d3176/apps/locallibrary/TODO.md#L219-L221. symphonia 0.6.1 is out, but stratum-dsp 1.0.0 still needs 0.5 (https://github.com/cutenode/localnetwork/blob/24d3176/crates/loclib-analysis/Cargo.toml#L21).
Steps to reproduce
- Take a small M4A, find its
stszatom, and set the sample count to 4294967295 with a sample size of 0. - Run
locallibrary check <drive> --audioorlocallibrary mixon it, on Linux withvm.overcommit_memory=2or on Windows. - The process aborts on the allocation.
What did you expect?
A damaged file is reported as unreadable. File the missing limit upstream with the repro, and check whether 0.6 bounds it.
- Lenguaje dominante
- Rust
- Estrellas
- 0
- Forks
- 0
- Merge medio
- 8 h 46 min
- PR fusionados (30 d)
- 112
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
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 cutenode/localnetwork
-
enhancement locallibrary locallibrary: mixes
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
cutenode/localnetwork#182 ·
Los mantenedores suelen responder en 1 día
-
bug locallibrary: core
Dificultad 2/5 1-3 horas Aptitud para principiantes 80/100
cutenode/localnetwork#169 ·
Los mantenedores suelen responder en 1 día
-
bug locallibrary: core
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
cutenode/localnetwork#158 ·
Los mantenedores suelen responder en 1 día
-
bug locallibrary locallibrary: core
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
cutenode/localnetwork#137 ·
Los mantenedores suelen responder en 1 día
-
locallibrary: a plugin's export plan adds up the file sizes it reports without an overflow checkAbiertobug locallibrary locallibrary: plugins
Dificultad 2/5 1-3 horas Aptitud para principiantes 83/100
cutenode/localnetwork#133 ·
Los mantenedores suelen responder en 1 día
Todos los issues de cutenode/localnetwork
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
farion1231/cc-switch#8829 ·
Los mantenedores suelen responder en 1 día
-
bug ci good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 92/100
longbridge/gpui-kit#3677 ·
Los mantenedores suelen responder en 1 día
-
E-pr-welcome
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
seanmonstar/reqwest#3121 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 68/100
ilya-zlobintsev/LACT#1235 ·
Los mantenedores suelen responder en 1 día