[BUG] Memory leak in free_sub_track(): blockaddition and message buffer never freed for WebVTT tracks
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 1/5
- Tempo stimato
- Meno di un'ora
- Idoneità per principianti
- 86/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Tranquilla
- Stack tecnologico
- c
- Ambito
- audio-video-rtc
Direzione di ricerca
Inizia in src/lib_ccx/matroska.c, in free_sub_track(), poi esamina parse_segment_cluster_block_group_block_additions() per confermare le allocazioni di block_addition e del messaggio di backing. Libera entrambe le allocazioni durante la liberazione delle frasi WebVTT, quindi elabora un MKV con BlockAdditions WebVTT sotto Valgrind e verifica che le perdite segnalate siano scomparse.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Description
In src/lib_ccx/matroska.c, the function free_sub_track()
iterates over all sentences and frees sentence->text and
the sentence itself, but never frees sentence->blockaddition
or the backing message buffer that was allocated in
parse_segment_cluster_block_group_block_additions().
Affected Code
src/lib_ccx/matroska.c — function free_sub_track()
Root Cause
In parse_segment_cluster_block_group_block_additions(),
a message buffer is allocated via read_bytes_signed()
and a block_addition struct is allocated via calloc().
The fields cue_settings_list, cue_identifier, and
comment are pointers into that message buffer.
In free_sub_track(), neither the block_addition struct
nor the message buffer are ever freed, causing a memory
leak on every WebVTT subtitle that contains BlockAdditions.
Steps to Reproduce
Process any .mkv file containing WebVTT subtitle tracks
with BlockAdditions (cue settings, identifiers, or comments).
Run with a memory checker such as Valgrind — leaks will be
reported for every such subtitle block.
Expected Behavior
All allocated memory should be freed when a subtitle track
is released.
Fix
Free sentence->blockaddition->cue_settings_list (the
backing message buffer) and sentence->blockaddition
inside the loop in free_sub_track().
- Lingua principale
- C
- Stelle
- 902
- Fork
- 591
- Merge medio
- 4g 19h
- PR unite (30g)
- 10
Preparare l'ambiente
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di CCExtractor/ccextractor
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
CCExtractor/ccextractor#2352 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 65/100
CCExtractor/ccextractor#2355 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 48/100
CCExtractor/ccextractor#2350 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 72/100
CCExtractor/ccextractor#2348 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 35/100
CCExtractor/ccextractor#2345 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di CCExtractor/ccextractor
Issue simili
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
johnsonjh/emu2-cpm86#68 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Zenmap CrashApertaZenmap
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
I maintainer di solito rispondono entro 2 giorni
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
fastfetch-cli/fastfetch#2619 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
I maintainer di solito rispondono entro 1 giorno
-
Template: Bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
betaflight/betaflight#15759 ·
I maintainer di solito rispondono entro 1 giorno