code.ossystems Code Review - openembedded-core.git/blob

   1 gst-ffmpeg: qdm2: check array index before use, fix out of array
   2  accesses
   3
   4 Upstream-Status: Backport
   5
   6 Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
   7 Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
   8 ---
   9  libavcodec/qdm2.c |    5 +++++
  10  1 files changed, 5 insertions(+), 0 deletions(-)
  11
  12 diff --git a/libavcodec/qdm2.c b/libavcodec/qdm2.c
  13 index 4cf4b2f..1dfb8d5 100644
  14 --- a/gst-libs/ext/libav/libavcodec/qdm2.c
  15 +++ b/gst-libs/ext/libav/libavcodec/qdm2.c
  16 @@ -1257,6 +1257,11 @@ static void qdm2_decode_super_block (QDM2Context *q)
  17      for (i = 0; packet_bytes > 0; i++) {
  18          int j;
  19
  20 +        if (i>=FF_ARRAY_ELEMS(q->sub_packet_list_A)) {
  21 +            SAMPLES_NEEDED_2("too many packet bytes");
  22 +            return;
  23 +        }
  24 +
  25          q->sub_packet_list_A[i].next = NULL;
  26
  27          if (i > 0) {
  28 --
  29 1.7.5.4
  30