diff options
author | wm4 <wm4@nowhere> | 2013-10-31 18:12:39 +0100 |
---|---|---|
committer | wm4 <wm4@nowhere> | 2013-10-31 18:17:14 +0100 |
commit | 7100a1012760388f501c886d9209831183552061 (patch) | |
tree | bb93ad4875b049312bd30c26584826cfb12d59be /audio | |
parent | ee2ba635f266f71c38c21e8fffd0f7fbeb44944d (diff) | |
download | mpv-7100a1012760388f501c886d9209831183552061.tar.bz2 mpv-7100a1012760388f501c886d9209831183552061.tar.xz |
vd_lavc: add more ifdeffery and ffmpeg cargo culting for correctness
We mixed the "old" AVFrame management functions (avcodec_alloc_frame,
avcodec_free_frame) with reference counting. This doesn't work
correctly; you must use av_frame_alloc and av_frame_free. Of course
ffmpeg doesn't warn us about the bad usage, but will just mess up
things silently. (Thanks a lot...)
While the alloc function seems to be 100% compatible, the free function
will do bad things, such as freeing memory that might still be
referenced by another frame. I didn't experience any actual bugs, but
maybe that was pure luck.
Diffstat (limited to 'audio')
0 files changed, 0 insertions, 0 deletions