Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

`jamulusclient/setFaderLevel` accepts a channel index one past the end

Đang mở
  • Có gì không ổn?
#3,918 0 bình luận 0 reaction 1 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 6 ngày

@mcfnord đang làm issue này rồi.

Từ ngày 22/8/2026.

Đánh giá

Issue này chưa được đánh giá.

Mô tả

bug

🤖 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.

Ngôn ngữ chính
C
Star
1.1k
Fork
248
Merge trung bình
11 ngày 40 phút
Pull request đã merge (30 ngày)
6

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của jamulussoftware/jamulus

Tất cả issue của jamulussoftware/jamulus

Issue tương tự

Thêm issue về C

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.