diff options
author | Andreas Rheinhardt <andreas.rheinhardt@outlook.com> | 2024-02-18 15:19:49 +0100 |
---|---|---|
committer | Andreas Rheinhardt <andreas.rheinhardt@outlook.com> | 2024-02-18 18:09:33 +0100 |
commit | 44620ade251f705b1395d91a540c51768b2d9c8b (patch) | |
tree | 1795c1db1e0a7443993a5f68c7b0dbcfbbfce4e3 /libavcodec/libcodec2.c | |
parent | 32538dafca37cadad143ca28ceae39b3ea64bb5f (diff) | |
download | ffmpeg-44620ade251f705b1395d91a540c51768b2d9c8b.tar.gz |
fftools/ffmpeg_mux_init: Fix attachment_filename use-after-free
The filename is freed with the OptionsContext and therefore
there will be a use-after-free when reporting the filename
in print_stream_maps(). So create a copy of the string.
This is a regression since 8aed3911fc454e79697e183660bf30d31334a64b.
fate-lavf-mkv_attachment exhibits it (and reports a random nonsense
filename here), but this does not make the test fail (not even with
valgrind; only with ASAN, as it aborts on use-after-free).
Reviewed-by: Anton Khirnov <anton@khirnov.net>
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
Diffstat (limited to 'libavcodec/libcodec2.c')
0 files changed, 0 insertions, 0 deletions