encode: bail out on missing A or V stream

In mplayer2, it was valid to try to start encoding before all streams
were initialized. mpv avoids this situation and thus allows us to
properly bail out on some kinds of failures.

Also, this commit fixes a missing check in ao uninit which could cause
heap corruption when ao initialization did not complete.

1 files changed, 8 insertions, 2 deletions

diff --git a/encode_lavc.c b/encode_lavc.c
index e38ca5bca8..66f434410a 100644
--- a/encode_lavc.c
+++ b/encode_lavc.c
@@ -233,15 +233,21 @@ int encode_lavc_start(struct encode_lavc_context *ctx)
         for (i = 0; i < ctx->avc->nb_streams; ++i)
             if (ctx->avc->streams[i]->codec->codec_type == AVMEDIA_TYPE_VIDEO)
                 break;
-        if (i >= ctx->avc->nb_streams)
+        if (i >= ctx->avc->nb_streams) {
+            encode_lavc_fail(ctx,
+                    "encode-lavc: video stream missing, invalid codec?\n");
             return 0;
+        }
     }
     if (ctx->expect_audio) {
         for (i = 0; i < ctx->avc->nb_streams; ++i)
             if (ctx->avc->streams[i]->codec->codec_type == AVMEDIA_TYPE_AUDIO)
                 break;
-        if (i >= ctx->avc->nb_streams)
+        if (i >= ctx->avc->nb_streams) {
+            encode_lavc_fail(ctx,
+                    "encode-lavc: audio stream missing, invalid codec?\n");
             return 0;
+        }
     }
 
     ctx->header_written = -1;