`jamulusclient/setFaderLevel` accepts a channel index one past the end
I maintainer di solito rispondono entro 3 giorni
@mcfnord ci sta già lavorando.
Dal 22/8/2026.
Valutazione
Questa issue non è ancora stata valutata.
Descrizione
🤖 AI: Split out of #3884 at pljones's request — same inclusive-vs-exclusive family as the customdirectoryindex crash also split from that issue, at a different boundary; this one cannot crash.
Valid fader indices are 0..MAX_NUM_CHANNELS-1 = 0..149 (vecpChanFader is sized MAX_NUM_CHANNELS at src/audiomixerboard.cpp:1020; MAX_NUM_CHANNELS is 150 at src/global.h:233). The validator rejects only channelIndex > MAX_NUM_CHANNELS (src/clientrpc.cpp:397), so 150 is accepted. Driven over the JSON-RPC socket:
| channelIndex | response |
|---|---|
| 149 | {"result":"ok"} |
| 150 | {"result":"ok"} |
| 151 | {"error":{"code":-32602,"message":"...out-of-range"}} |
This cannot crash: every consumer re-guards with < MAX_NUM_CHANNELS (src/audiomixerboard.cpp:1461, and the headless path at src/client.cpp:956), so channel 150 is silently dropped while the caller is told "ok". Fix: > → >=. The house pattern elsewhere is exclusive already — MathUtils::InRange<int> ( iCurChanID, 0, iMaxNumChannels ) at src/server.cpp:1386.
Measured on main @ 8b667a3a; line numbers re-verified against origin/main @ 11a28d57e33506fc84cb9b5ee02097931c1e35a3 on 2026-08-21 — two of the five shifted since the original measurement (clientrpc.cpp 371→397, client.cpp 954→956); audiomixerboard.cpp:1461 and the rest are unchanged, code and behavior unchanged throughout.
🤖 This message was written by AI and reviewed by @mcfnord.
- Lingua principale
- C
- Stelle
- 1.1k
- Fork
- 248
- Merge medio
- 2g 22h
- PR unite (30g)
- 6
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di jamulussoftware/jamulus
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
jamulussoftware/jamulus#3953 · 2 commenti ·
I maintainer di solito rispondono entro 3 giorni
-
AI bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
jamulussoftware/jamulus#3901 · 4 commenti · 1 reazione ·
I maintainer di solito rispondono entro 3 giorni
-
AI
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
jamulussoftware/jamulus#3846 ·
I maintainer di solito rispondono entro 3 giorni
-
Qt6 moving towards cmakeApertafeature request
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
jamulussoftware/jamulus#3964 · 3 commenti ·
I maintainer di solito rispondono entro 3 giorni
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 65/100
jamulussoftware/jamulus#3961 ·
I maintainer di solito rispondono entro 3 giorni
Tutte le issue di jamulussoftware/jamulus
Issue simili
-
After `require "openssl"`, a top-level `Digest` is `OpenSSL::Digest`, not the `Digest` moduleAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 92/100
-
enhancement
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 92/100
-
enhancement
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 94/100