shithub: dav1d

Info  •  Files  •  Log  •  Branches

call dav1d_get_picture() until it returns -EAGAIN for draining

With the decoupled decoding data there might be remaining input data during
draining which can cause bitstream parsing errors.

--- a/tests/libfuzzer/dav1d_fuzzer.c

+++ b/tests/libfuzzer/dav1d_fuzzer.c

@@ -121,7 +121,7 @@

         err = dav1d_get_picture(ctx, &pic);

         if (err == 0)

             dav1d_picture_unref(&pic);

-    } while (err == 0);

+    } while (err != -EAGAIN);

 cleanup:

     dav1d_flush(ctx);

--- a/tools/dav1d.c

+++ b/tools/dav1d.c

@@ -151,9 +151,10 @@

             if (res != -EAGAIN) {

                 fprintf(stderr, "Error decoding frame: %s\n",

                         strerror(-res));

-            } else

+            } else {

                 res = 0;

-            break;

+                break;

+            }

         } else {

             if (!n_out) {

                 if ((res = output_open(&out, cli_settings.muxer,