You've already forked FFmpeg
							
							
				mirror of
				https://github.com/FFmpeg/FFmpeg.git
				synced 2025-10-30 23:18:11 +02:00 
			
		
		
		
	Fixed segfault with wavpack decoder on corrupted decorrelation terms sub-blocks.
Signed-off-by: Martin Storsjö <martin@martin.st>
This commit is contained in:
		
				
					committed by
					
						 Martin Storsjö
						Martin Storsjö
					
				
			
			
				
	
			
			
			
						parent
						
							6a11686dfc
						
					
				
				
					commit
					8bfea4ab4e
				
			| @@ -863,12 +863,13 @@ static int wavpack_decode_block(AVCodecContext *avctx, int block_no, | |||||||
|         } |         } | ||||||
|         switch(id & WP_IDF_MASK){ |         switch(id & WP_IDF_MASK){ | ||||||
|         case WP_ID_DECTERMS: |         case WP_ID_DECTERMS: | ||||||
|             s->terms = size; |             if(size > MAX_TERMS){ | ||||||
|             if(s->terms > MAX_TERMS){ |  | ||||||
|                 av_log(avctx, AV_LOG_ERROR, "Too many decorrelation terms\n"); |                 av_log(avctx, AV_LOG_ERROR, "Too many decorrelation terms\n"); | ||||||
|  |                 s->terms = 0; | ||||||
|                 buf += ssize; |                 buf += ssize; | ||||||
|                 continue; |                 continue; | ||||||
|             } |             } | ||||||
|  |             s->terms = size; | ||||||
|             for(i = 0; i < s->terms; i++) { |             for(i = 0; i < s->terms; i++) { | ||||||
|                 s->decorr[s->terms - i - 1].value = (*buf & 0x1F) - 5; |                 s->decorr[s->terms - i - 1].value = (*buf & 0x1F) - 5; | ||||||
|                 s->decorr[s->terms - i - 1].delta = *buf >> 5; |                 s->decorr[s->terms - i - 1].delta = *buf >> 5; | ||||||
|   | |||||||
		Reference in New Issue
	
	Block a user