commit 239029b61f575847650021a5b4904ed426a2e9e4
from: Omar Polo <op@omarpolo.com>
date: Mon May 09 18:32:02 2022 UTC

don't call player_sendeof on IMSG_STOP

the refactoring introduced this error where we call report an EOF upon
IMSG_STOP, making the player infinitely loop.

commit - bf19b03e6400fa3a7573b0ba4fd057767f0adc22
commit + 239029b61f575847650021a5b4904ed426a2e9e4
blob - 8a11967a55fb6e34c043591ae23d80109c613d8c
blob + 4c839c50b4dc882266974106bff240a3a41bb793
--- player.c
+++ player.c
@@ -274,7 +274,7 @@ play(const void *buf, size_t len)
 int
 player(int debug, int verbose)
 {
-	int flags;
+	int flags, r;
 	log_init(debug, LOG_DAEMON);
 	log_setverbose(verbose);
 
@@ -314,9 +314,10 @@ player(int debug, int verbose)
 		while (nextfd == -1)
 			player_dispatch();
 
-		if (player_playnext() == -1)
+		r = player_playnext();
+		if (r == -1)
 			player_senderr();
-		else
+		if (r == 0)
 			player_sendeof();
 	}