summaryrefslogtreecommitdiffstats
path: root/sub/sd_ass.c
diff options
context:
space:
mode:
authorwm4 <wm4@nowhere>2015-02-25 13:31:37 +0100
committerwm4 <wm4@nowhere>2015-02-25 13:31:37 +0100
commit9349cfca910990d1126b973fd2ca55cb2351d35f (patch)
treecc517d182f879c0708f22fc070c9f6755b762466 /sub/sd_ass.c
parent02bd54c0ac0933bc2fe982666bbe4da75957c485 (diff)
downloadmpv-9349cfca910990d1126b973fd2ca55cb2351d35f.tar.bz2
mpv-9349cfca910990d1126b973fd2ca55cb2351d35f.tar.xz
demux_mkv_timeline: fix potential issue when enabling cache
If the cache is enabled, the demuxer is closed and opened again (because currently, the cache can not be enabled atfer data was already read). The call for opening a new demuxer uses the same params struct, which references the ctx->uids array. But there is a MP_TARRAY_GROW() invocation somewhere on the way, which can reallocate the ctx->uids array, making params.uids a dangling pointer. This issue probably existed for a longer time, probably since 5cd33853 (slightly more obvious since f50b105d).
Diffstat (limited to 'sub/sd_ass.c')
0 files changed, 0 insertions, 0 deletions