From 2f559ac70b63141545992e934cc41941cf658b0e Mon Sep 17 00:00:00 2001
From: wm4 <wm4@nowhere>
Date: Sun, 19 Oct 2014 22:34:00 +0200
Subject: [PATCH] player: shutdown all clients before actual uninit

This seems safer. It might be possible that commands sent by the clients
could recreate e.g. audio or video outputs.
---
 player/main.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/player/main.c b/player/main.c
index b54b59674b..90841f5fdf 100644
--- a/player/main.c
+++ b/player/main.c
@@ -119,6 +119,8 @@ static void shutdown_clients(struct MPContext *mpctx)
 
 void mp_destroy(struct MPContext *mpctx)
 {
+    shutdown_clients(mpctx);
+
     uninit_audio_out(mpctx);
     uninit_video_out(mpctx);
 
@@ -129,8 +131,6 @@ void mp_destroy(struct MPContext *mpctx)
 
     mpctx->encode_lavc_ctx = NULL;
 
-    shutdown_clients(mpctx);
-
 #if !defined(__MINGW32__)
     mp_uninit_ipc(mpctx->ipc_ctx);
     mpctx->ipc_ctx = NULL;