diff options
| author | Ronald S. Bultje <rsbultje@gmail.com> | 2012-03-07 13:48:41 -0800 |
|---|---|---|
| committer | Ronald S. Bultje <rsbultje@gmail.com> | 2012-03-07 13:48:41 -0800 |
| commit | e97efecec82ca8458a9bbd75a91ebf556abde362 (patch) | |
| tree | ecad099ec93859cdb5d6838bc36118a2e94bb782 /libavcodec/dvdata.c | |
| parent | 6e2821160b151f402f82b16171a1cdc27b4a9b83 (diff) | |
| download | ffmpeg-e97efecec82ca8458a9bbd75a91ebf556abde362.tar.gz | |
dv: check buffer size before reading profile.
Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC: libav-stable@libav.org
Diffstat (limited to 'libavcodec/dvdata.c')
| -rw-r--r-- | libavcodec/dvdata.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/libavcodec/dvdata.c b/libavcodec/dvdata.c index e9929d0997..ac6e993d2f 100644 --- a/libavcodec/dvdata.c +++ b/libavcodec/dvdata.c @@ -286,11 +286,13 @@ static const DVprofile dv_profiles[] = { const DVprofile* avpriv_dv_frame_profile(const DVprofile *sys, const uint8_t* frame, unsigned buf_size) { - int i; + int i, dsf, stype; - int dsf = (frame[3] & 0x80) >> 7; + if (buf_size < 80*5 + 48 + 4) + return NULL; - int stype = frame[80*5 + 48 + 3] & 0x1f; + dsf = (frame[3] & 0x80) >> 7; + stype = frame[80*5 + 48 + 3] & 0x1f; /* 576i50 25Mbps 4:1:1 is a special case */ if (dsf == 1 && stype == 0 && frame[4] & 0x07 /* the APT field */) { |