diff options
| author | Ronald S. Bultje <rsbultje@gmail.com> | 2017-03-29 09:03:49 -0400 |
|---|---|---|
| committer | Ronald S. Bultje <rsbultje@gmail.com> | 2017-03-31 08:26:24 -0400 |
| commit | b5300c8ad8c5384ab3654d6cb27693422bc424e7 (patch) | |
| tree | eba983d3d8089f65535213de8057b22265fc2c7c /libavcodec/h264_picture.c | |
| parent | 081c21ca55d72921125848c8c2c191a6ff8b5f88 (diff) | |
| download | ffmpeg-b5300c8ad8c5384ab3654d6cb27693422bc424e7.tar.gz | |
h264: don't write to source picture object in ff_h264_ref_picture().
Doing so is analogous to writing to source data in memcpy(), and causes
(harmless) tsan warnings in fate-h264.
Diffstat (limited to 'libavcodec/h264_picture.c')
| -rw-r--r-- | libavcodec/h264_picture.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/libavcodec/h264_picture.c b/libavcodec/h264_picture.c index f634d2a1a7..db9673793a 100644 --- a/libavcodec/h264_picture.c +++ b/libavcodec/h264_picture.c @@ -70,8 +70,8 @@ int ff_h264_ref_picture(H264Context *h, H264Picture *dst, H264Picture *src) av_assert0(!dst->f->buf[0]); av_assert0(src->f->buf[0]); + av_assert0(src->tf.f == src->f); - src->tf.f = src->f; dst->tf.f = dst->f; ret = ff_thread_ref_frame(&dst->tf, &src->tf); if (ret < 0) |