client/Expire: cancel the BackgroundCommand before deleting it

Fixes assertion failure. Closes https://github.com/MusicPlayerDaemon/MPD/issues/774

client/Expire: cancel the BackgroundCommand before deleting it
2229e866 · Max Kellermann · f24c274f · 2229e866
Commit 2229e866 authored Mar 12, 2020 by Max Kellermann
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 1 deletion

Expire.cxx src/client/Expire.cxx +4 -1

No files found.
--- a/src/client/Expire.cxx
+++ b/src/client/Expire.cxx
@@ -28,7 +28,10 @@ Client::SetExpired() noexcept
 	if (IsExpired())
 		return;

-	background_command.reset();
+	if (background_command) {
+		background_command->Cancel();
+		background_command.reset();
+	}

 	FullyBufferedSocket::Close();
 	timeout_event.Schedule(std::chrono::steady_clock::duration::zero());