merge: Multicast-Fehler behoben - htonl() zerriss die Gruppenadresse

* sacn-multicast-fix:
  fix(sacn): Multicast kam nie an - htonl() zerriss die Gruppenadresse

src/net/nl_e131.c

@@ -69,3 +69,8 @@ bool nl_e131_parse(const uint8_t *buf, size_t len, nl_e131_packet_t *out)
     out->slots = buf + NL_E131_MIN_PACKET;
     return true;
 }
+
+uint32_t nl_e131_group_addr(uint16_t universe)
+{
+    return 0xefff0000u | ((uint32_t)(universe >> 8) << 8) | (uint32_t)(universe & 0xffu);
+}

src/net/nl_e131.h

@@ -23,4 +23,10 @@ typedef struct {
  * `out` wird nur bei true beschrieben. */
 bool nl_e131_parse(const uint8_t *buf, size_t len, nl_e131_packet_t *out);
 
+/* Die Multicast-Gruppe eines Universe nach E1.31 Anhang A: 239.255.<hi>.<lo>, als Zahl in
+ * HOST-Byte-Reihenfolge. Steht hier und nicht im Empfangs-Task, damit sie im Host-Test
+ * nachgerechnet werden kann - der Wert war schon einmal falsch, und man sieht es einer
+ * Lampe nicht an: sie tritt dann sauber einer Gruppe bei, auf die niemand sendet. */
+uint32_t nl_e131_group_addr(uint16_t universe);
+
 #endif

src/net/nl_sacn.c

@@ -57,6 +57,7 @@ static int open_socket(uint16_t universe, char ip_seen[16])
     struct ip_mreq mreq;
     struct timeval tv;
     nl_wifi_status_t w;
+    uint32_t group;
     int on = 1;
     int fd = socket(AF_INET, SOCK_DGRAM, 0);
 
@@ -90,8 +91,20 @@ static int open_socket(uint16_t universe, char ip_seen[16])
      * nicht beitritt oder das WLAN die Gruppe nicht weiterreicht. Genau diese Frage
      * beantwortet das Feld. */
     memset(&mreq, 0, sizeof mreq);
-    mreq.imr_multiaddr.s_addr =
-        htonl(0xefff0000u | ((uint32_t)(universe >> 8) << 8) | (universe & 0xffu));
+    /* Die Gruppenadresse wird ZUERST in eine Variable gerechnet und erst dann umgedreht.
+     * Das ist kein Schoenheitsfehler: htonl() ist in diesem SDK ein Makro, das sein
+     * Argument NICHT klammert (sdk/Include/Net/wm_sockets2.0.3.h):
+     *
+     *   #define htonl(n) (((n & 0xff) << 24) | ((n & 0xff00) << 8) | ...)
+     *
+     * Ein Ausdruck mit | wird dabei zerrissen, weil & staerker bindet als | - die Maske
+     * trifft nur den letzten Term. Frueher stand der ganze Ausdruck direkt im Aufruf, und
+     * fuer Universe 1 kam 239.255.239.255 statt 239.255.0.1 heraus. Die Lampe trat also
+     * sauber einer Gruppe bei, auf die niemand sendete: Multicast kam nie an, waehrend
+     * Unicast lief, und der Beitritt meldete trotzdem Erfolg. Gefunden am 2026-09-21 am
+     * EVK ueber die serielle Konsole. */
+    group = nl_e131_group_addr(universe);
+    mreq.imr_multiaddr.s_addr = htonl(group);
     mreq.imr_interface.s_addr = ip_seen[0] ? inet_addr(ip_seen) : htonl(INADDR_ANY);
     s_joined = setsockopt(fd, IPPROTO_IP, IP_ADD_MEMBERSHIP, &mreq, sizeof mreq) == 0;
     printf("sacn: universe %u, multicast join %s\r\n", (unsigned)universe,

test/test_e131.c

@@ -192,9 +192,23 @@ static void test_universe_too_high_rejected(void)
     TEST_ASSERT_FALSE(nl_e131_parse(pkt, pkt_len, &p));
 }
 
+static void test_group_address_per_annex_a(void)
+{
+    /* 239.255.<hi>.<lo>. Der Fehler, den dieser Test festhaelt: stand die Rechnung direkt
+     * im htonl()-Aufruf, zerriss das nicht geklammerte SDK-Makro sie, und aus Universe 1
+     * wurde 239.255.239.255 - eine Gruppe, auf die niemand sendet. */
+    TEST_ASSERT_EQUAL_HEX32(0xefff0001u, nl_e131_group_addr(1));
+    TEST_ASSERT_EQUAL_HEX32(0xefff000cu, nl_e131_group_addr(12));
+    TEST_ASSERT_EQUAL_HEX32(0xefff012cu, nl_e131_group_addr(300));
+    TEST_ASSERT_EQUAL_HEX32(0xefff00ffu, nl_e131_group_addr(255));
+    TEST_ASSERT_EQUAL_HEX32(0xefff0100u, nl_e131_group_addr(256));
+    TEST_ASSERT_EQUAL_HEX32(0xeffff9ffu, nl_e131_group_addr(63999));
+}
+
 int main(void)
 {
     UNITY_BEGIN();
+    RUN_TEST(test_group_address_per_annex_a);
     RUN_TEST(test_valid_packet);
     RUN_TEST(test_full_universe);
     RUN_TEST(test_empty_slot_set);