You've already forked FFmpeg
mirror of
https://github.com/FFmpeg/FFmpeg.git
synced 2025-08-10 06:10:52 +02:00
mov: Check the entries value when parsing dref boxes
And properly reset the entries count when resetting the entries. CC: libav-stable@libav.org Bug-Id: 929 Bug-Id: CVE-2016-3062
This commit is contained in:
@@ -473,9 +473,11 @@ static int mov_read_dref(MOVContext *c, AVIOContext *pb, MOVAtom atom)
|
|||||||
|
|
||||||
avio_rb32(pb); // version + flags
|
avio_rb32(pb); // version + flags
|
||||||
entries = avio_rb32(pb);
|
entries = avio_rb32(pb);
|
||||||
if (entries > (atom.size - 1) / MIN_DATA_ENTRY_BOX_SIZE + 1 ||
|
if (!entries ||
|
||||||
|
entries > (atom.size - 1) / MIN_DATA_ENTRY_BOX_SIZE + 1 ||
|
||||||
entries >= UINT_MAX / sizeof(*sc->drefs))
|
entries >= UINT_MAX / sizeof(*sc->drefs))
|
||||||
return AVERROR_INVALIDDATA;
|
return AVERROR_INVALIDDATA;
|
||||||
|
sc->drefs_count = 0;
|
||||||
av_free(sc->drefs);
|
av_free(sc->drefs);
|
||||||
sc->drefs = av_mallocz(entries * sizeof(*sc->drefs));
|
sc->drefs = av_mallocz(entries * sizeof(*sc->drefs));
|
||||||
if (!sc->drefs)
|
if (!sc->drefs)
|
||||||
|
Reference in New Issue
Block a user