From efade6294d7420ecd123d5641385c27c40794fca Mon Sep 17 00:00:00 2001
From: Max Kellermann <max.kellermann@gmail.com>
Date: Sat, 25 Mar 2017 20:20:30 +0100
Subject: [PATCH] aconnect: assign variable first, then start connect

Fixes bogus assertion failure.
---
 NEWS           | 1 +
 src/aconnect.c | 4 ++--
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/NEWS b/NEWS
index 2e36c8a..6956a1e 100644
--- a/NEWS
+++ b/NEWS
@@ -1,5 +1,6 @@
 ncmpc 0.27 - not yet released
 * remove the status bar clock (option "display-time")
+* fix assertion failure after connect failure
 
 ncmpc 0.26 - (2017-03-20)
 * require libmpdclient 2.9
diff --git a/src/aconnect.c b/src/aconnect.c
index 345bc52..a0f7315 100644
--- a/src/aconnect.c
+++ b/src/aconnect.c
@@ -144,10 +144,10 @@ aconnect_start(struct aconnect **acp,
 	ac->handler = handler;
 	ac->handler_ctx = ctx;
 
+	*acp = ac;
+
 	async_rconnect_start(&ac->rconnect, host, port,
 			     &aconnect_rconnect_handler, ac);
-
-	*acp = ac;
 }
 
 void
-- 
2.30.2