You've already forked FFmpeg
							
							
				mirror of
				https://github.com/FFmpeg/FFmpeg.git
				synced 2025-10-30 23:18:11 +02:00 
			
		
		
		
	qtpalette: make the color_* variables unsigned again
This fixes segmentation faults due to out of bounds writes, when color_start is interpreted as negative number. This regression was introduced in commit 57631f. Reviewed-by: Mats Peterson <matsp888@yahoo.com> Signed-off-by: Andreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>
This commit is contained in:
		| @@ -48,7 +48,7 @@ int ff_get_qtpalette(int codec_id, AVIOContext *pb, uint32_t *palette) | ||||
|  | ||||
|     /* If the depth is 1, 2, 4, or 8 bpp, file is palettized. */ | ||||
|     if ((bit_depth == 1 || bit_depth == 2 || bit_depth == 4 || bit_depth == 8)) { | ||||
|         int color_count, color_start, color_end; | ||||
|         uint32_t color_count, color_start, color_end; | ||||
|         uint32_t a, r, g, b; | ||||
|  | ||||
|         /* Ignore the greyscale bit for 1-bit video and sample | ||||
|   | ||||
		Reference in New Issue
	
	Block a user