`jamulusclient/setFaderLevel` accepts a channel index one past the end
@mcfnord já está trabalhando nisso.
Desde 22/8/2026.
Avaliação
Esta issue ainda não foi avaliada.
Descrição
🤖 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.
- Linguagem predominante
- C
- Estrelas
- 1.1k
- Forks
- 250
- Merge médio
- 7d 7h
- PRs com merge (30d)
- 2
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Tem um modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de jamulussoftware/jamulus
-
Move translation checker (and potentially other runners) to ARM runnerTalvez já em andamento @ann0see assumiu há 42 dias. Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100
jamulussoftware/jamulus#3953 · 2 comentários ·
-
AI
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
jamulussoftware/jamulus#3846 ·
-
Qt6 moving towards cmakeAbertafeature request
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 25/100
jamulussoftware/jamulus#3964 · 3 comentários ·
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 65/100
jamulussoftware/jamulus#3961 ·
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 76/100
jamulussoftware/jamulus#3958 ·
Todas as issues de jamulussoftware/jamulus
Issues semelhantes
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 65/100
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
HexaCluster/credcheck#86 ·
-
Wheels for Python 3.15Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 66/100
mongodb/winkerberos#175 ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 63/100
raspberrypi/linux#7718 · 2 comentários ·
Mantenedores costumam responder em até 1 dia
-
bug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 74/100
AOMediaCodec/SVT-AV1#3320 ·