Mercurial > libavformat.hg
diff sierravmd.c @ 5207:f08a788606d5 libavformat
Fix overflow check insufficiently improved in r19840.
It assumes that sizeof(vmd_frame) < 64k, otherwise an additional
check to ensure sound_buffers <= UINT_MAX / sizeof(vmd_frame) would be necessary.
author | reimar |
---|---|
date | Wed, 16 Sep 2009 15:12:23 +0000 |
parents | 0394dceb230f |
children | 536e5527c1e0 |
line wrap: on
line diff
--- a/sierravmd.c Wed Sep 16 12:26:59 2009 +0000 +++ b/sierravmd.c Wed Sep 16 15:12:23 2009 +0000 @@ -161,7 +161,7 @@ vmd->frame_table = NULL; sound_buffers = AV_RL16(&vmd->vmd_header[808]); raw_frame_table_size = vmd->frame_count * 6; - if(vmd->frame_count * vmd->frames_per_block >= (UINT_MAX - sound_buffers) / sizeof(vmd_frame)){ + if(vmd->frame_count * vmd->frames_per_block >= UINT_MAX / sizeof(vmd_frame) - sound_buffers){ av_log(s, AV_LOG_ERROR, "vmd->frame_count * vmd->frames_per_block too large\n"); return -1; }