channels: fix reentrancy guard in channel_class_changed()

atomic_dec(&ch->ch_changed_ref, 0) subtracts 0 instead of 1, so
the reentrancy counter never resets after the first call. All
subsequent calls see atomic_add() > 0 and return early, permanently
suppressing HTSP channelUpdate notifications, bouquet propagation,
and tag updates for that channel.

Fix by decrementing by 1 so the guard properly resets to 0.

diff --git a/src/channels.c b/src/channels.c
index 82adbe4485357beb3d0e60b0c42e77b689a218ca..5535b46e2e50b8cbf52f775ef77b68bd1799fb1e 100644 (file)
--- a/src/channels.c
+++ b/src/channels.c
@@ -92,7 +92,7 @@ channel_class_changed ( idnode_t *self )
   htsp_channel_update(ch);
 
 end:
-  atomic_dec(&ch->ch_changed_ref, 0);
+  atomic_dec(&ch->ch_changed_ref, 1);
 }
 
 static htsmsg_t *