[FFmpeg-cvslog] avcodec/h264_slice: assert that the first slice implies unfinished setup

[Michael Niedermayer]

ffmpeg | branch: master | Michael Niedermayer <michaelni at gmx.at> | Sat Jun 27 23:41:00 2015 +0200| [9ea256814dc290866b85f56a1ba213e80c4a5f7a] | committer: Michael Niedermayer

avcodec/h264_slice: assert that the first slice implies unfinished setup

If this assert fails there is very likely a bug in the code

Signed-off-by: Michael Niedermayer <michaelni at gmx.at>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=9ea256814dc290866b85f56a1ba213e80c4a5f7a
---

 libavcodec/h264_slice.c |    6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/libavcodec/h264_slice.c b/libavcodec/h264_slice.c
index ddde9e6..d6ad9f6 100644
--- a/libavcodec/h264_slice.c
+++ b/libavcodec/h264_slice.c
@@ -1155,6 +1155,9 @@ int ff_h264_decode_slice_header(H264Context *h, H264SliceContext *sl)
     int mb_aff_frame, last_mb_aff_frame;
     PPS *pps;
 
+    if (first_slice)
+        av_assert0(!h->setup_finished);
+
     h->qpel_put = h->h264qpel.put_h264_qpel_pixels_tab;
     h->qpel_avg = h->h264qpel.avg_h264_qpel_pixels_tab;
 
@@ -1162,6 +1165,7 @@ int ff_h264_decode_slice_header(H264Context *h, H264SliceContext *sl)
 
     if (first_mb_in_slice == 0) { // FIXME better field boundary detection
         if (h->current_slice) {
+            av_assert0(!h->setup_finished);
             if (h->cur_pic_ptr && FIELD_PICTURE(h) && h->first_field) {
                 ff_h264_field_end(h, h->slice_ctx, 1);
                 h->current_slice = 0;
@@ -1249,7 +1253,7 @@ int ff_h264_decode_slice_header(H264Context *h, H264SliceContext *sl)
         return AVERROR_INVALIDDATA;
     }
 
-    if (first_slice && !h->setup_finished) {
+    if (first_slice) {
         h->pps = *h->pps_buffers[pps_id];
     } else if (h->setup_finished && h->dequant_coeff_pps != pps_id) {
         av_log(h->avctx, AV_LOG_ERROR, "PPS changed between slices\n");