Loop up to MAX_THREADS instead of h->s.avctx->thread_count to free the thread

contexts, this avoids a crash when freeing the H.264 parser context introduced in
r18406, since h->s.avctx is NULL there.

Originally committed as revision 18418 to svn://svn.ffmpeg.org/ffmpeg/trunk

diff --git a/libavcodec/h264.c b/libavcodec/h264.c
index 99dd52b..b3c01a9 100644 (file)
--- a/libavcodec/h264.c
+++ b/libavcodec/h264.c
@@ -1989,7 +1989,7 @@ static void free_tables(H264Context *h){
     av_freep(&h->mb2b_xy);
     av_freep(&h->mb2b8_xy);
 
     av_freep(&h->mb2b_xy);
     av_freep(&h->mb2b8_xy);
 

-    for(i = 0; i < h->s.avctx->thread_count; i++) {
+    for(i = 0; i < MAX_THREADS; i++) {

         hx = h->thread_context[i];
         if(!hx) continue;
         av_freep(&hx->top_borders[1]);
         hx = h->thread_context[i];
         if(!hx) continue;
         av_freep(&hx->top_borders[1]);