shithub: dav1d

Download patch

ref: 3746b6e128324fff7468463aeac332279eb5d59c
parent: a5afcbe6f3e4c1d805e31871375f53684409fde2
author: Janne Grunau <[email protected]>
date: Sat Dec 8 14:03:56 EST 2018

memory allocation error checks

--- a/src/decode.c
+++ b/src/decode.c
@@ -2637,6 +2637,7 @@
                 dav1d_alloc_aligned(sizeof(int32_t) * 3 *
                                     f->sb128w * f->sb128h * 128 * 128, 32);
             if (!f->frame_thread.b || !f->frame_thread.pal_idx ||
+                !f->frame_thread.pal || !f->frame_thread.cbi ||
                 !f->frame_thread.cf)
             {
                 goto error;
@@ -3242,6 +3243,10 @@
             // put the new values. Allocate them here (the data
             // actually gets set elsewhere)
             f->cur_segmap_ref = dav1d_ref_create(f->b4_stride * 32 * f->sb128h);
+            if (!f->cur_segmap_ref) {
+                res = -ENOMEM;
+                goto error;
+            }
             f->cur_segmap = f->cur_segmap_ref->data;
         } else if (f->prev_segmap_ref) {
             // We're not updating an existing map, and we have a valid
@@ -3252,6 +3257,10 @@
         } else {
             // We need to make a new map. Allocate one here and zero it out.
             f->cur_segmap_ref = dav1d_ref_create(f->b4_stride * 32 * f->sb128h);
+            if (!f->cur_segmap_ref) {
+                res = -ENOMEM;
+                goto error;
+            }
             f->cur_segmap = f->cur_segmap_ref->data;
             memset(f->cur_segmap_ref->data, 0, f->b4_stride * 32 * f->sb128h);
         }