[P2][audio] Retenir le snapshot WAV validé plutôt que rouvrir le pathname #9

Open
opened 2026-07-31 21:05:11 +00:00 by mathieu · 1 comment
Owner

Constat

WavFile.Release ferme d'abord le snapshot scellé puis la rétention rouvre f.Path() (internal/audio/recorder.go:44-52, internal/audio/retention.go:112-176). Une mutation, substitution ou suffixe physique peut donc changer l'artefact retenu par rapport à l'audio réellement transcrit.

Tâche

Copier OpenRIFF() depuis le snapshot encore ouvert vers le store de rétention, puis fermer et supprimer la capture. Garder le chemin seulement pour une capture invalide sans snapshot.

Critères d'acceptation

  • Mutation ou remplacement après Stop ne change pas les octets retenus.
  • Les données après la taille RIFF sont exclues.
  • Tous les descripteurs ferment sur chaque erreur.

Validation

go test -race ./internal/audio ./internal/audio/wav

## Constat `WavFile.Release` ferme d'abord le snapshot scellé puis la rétention rouvre `f.Path()` (`internal/audio/recorder.go:44-52`, `internal/audio/retention.go:112-176`). Une mutation, substitution ou suffixe physique peut donc changer l'artefact retenu par rapport à l'audio réellement transcrit. ## Tâche Copier `OpenRIFF()` depuis le snapshot encore ouvert vers le store de rétention, puis fermer et supprimer la capture. Garder le chemin seulement pour une capture invalide sans snapshot. ## Critères d'acceptation - Mutation ou remplacement après Stop ne change pas les octets retenus. - Les données après la taille RIFF sont exclues. - Tous les descripteurs ferment sur chaque erreur. ## Validation `go test -race ./internal/audio ./internal/audio/wav`

Correctif proposé et validé dans la PR #43 : #43.

Correctif proposé et validé dans la PR #43 : https://git.2027a.net/mathieu/murmur/pulls/43.
Sign in to join this conversation.
No labels
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
mathieu/murmur#9
No description provided.