PlayHead: divide by zero on the audio thread when looping is turned on during playback

Abierto Apto para principiantes
  • ¿Algo no está bien?
#482 0 comentarios 0 reacciones 0 asignados Ver en GitHub

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
76/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
cpp
Área
audio-video-rtc, backend

Línea de trabajo

Empieza por PlayHead::setLoopRange y PlayHead::linearPositionToLoopPosition, mencionados en la incidencia, y sigue cómo PlayHeadState::update lee el rango del bucle durante la reproducción. Comprueba si ya existen pruebas para PlayHead y añade cobertura para activar y desactivar el bucle con un rango vacío; ejecuta la prueba pertinente. Se considera terminado cuando el hilo de audio no pueda pasar un rango de longitud cero a modulo.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

graph::PlayHead can divide by zero on the audio thread when looping is switched on during playback.

Stack: juce::negativeAwareModulo ← PlayHead::linearPositionToLoopPosition ← PlayHead::referenceSamplePositionToTimelinePosition ← PlayHeadState::update.

Cause: PlayHead::setLoopRange sets looping = loop before timelinePlayRange.store (loopRange). Turning the loop off stores an empty range (TransportControl: setLoopTimes (false, {})), so when it's turned back on the audio thread can briefly see looping == true with that zero-length range, and negativeAwareModulo divides by its length. The reverse can happen too: a reader holding looping == true from just before it's turned off, then reading the empty range.

Fix we're carrying (seen rarely, as a crash under load on Windows):

 inline void PlayHead::setLoopRange (bool loop, juce::Range<int64_t> loopRange, bool updatePosition)
 ...
-        looping = loop;
-        timelinePlayRange.store (loopRange);
+        if (loop)
+        {
+            timelinePlayRange.store (loopRange);
+            looping = true;
+        }
+        else
+        {
+            looping = false;
+            timelinePlayRange.store (loopRange);
+        }

 inline int64_t PlayHead::linearPositionToLoopPosition (int64_t position, juce::Range<int64_t> loopRange)
 {
+    if (loopRange.getLength() <= 0)
+        return position;
+
     const auto loopStart = loopRange.getStart();

The guard alone is enough to stop the crash, since the two atomics can't be read together; the ordering just makes it rarer. Seen on 38d5a27.

Lenguaje dominante
C++
Estrellas
1.5k
Forks
222
Merge medio
8 h 7 min
PR fusionados (30 d)
17

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

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de Tracktion/tracktion_engine

Todos los issues de Tracktion/tracktion_engine

Issues similares

Más issues de C++

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.