Hacktoberfest 2026：メンテナが10月に向けて印を付けた、オープンで初心者向けの issue。 Hacktoberfest の issue を見る

USB and COM port share one command buffer

オープン
  • 何か問題がありますか？
#615 コメント 2 件 リアクション 0 件 担当者 1 名 GitHub で見る

@Yukti007 がすでに取り組んでいます。

2026年10月13日 から。

評価

この issue はまだ評価されていません。

説明

High Priority regression test type: bug

Describe the bug
The Krake reads text commands from two ports: the USB serial port and the RS-232 COM port. Both ports put their characters into the same command buffer, with the same character counter. If one port has sent part of a command but not the end-of-line yet, characters from the other port get added to the end of it. The Krake then runs one wrong command made from pieces of both.

Short Summary of the expected behavior:
Each port's commands are handled separately. A command from one port is never mixed with characters from the other port.

Short Summary of the buggy behavior:
A half-finished line from one port gets finished by characters from the other port. This can give the wrong alarm level, the wrong message text, or a lost alarm.

To Reproduce
Found by reading the code. Not yet tested on a real Krake board. Suggested steps to confirm:

  1. Connect a Krake to a PC by USB and open a serial monitor at 115200 baud, with line ending set to "No line ending".
  2. Connect a second serial terminal to the COM port (DB9, RS-232).
  3. On USB, type a2Pump and send it (no newline is sent).
  4. On the COM port, send a5Fire with a newline.
  5. See error: the Krake shows a level 2 alarm with the text "Pumpa5Fire" instead of a level 5 alarm with the text "Fire".

Expected behavior
The COM port command a5Fire should give a level 5 alarm with the text "Fire". The unfinished text on USB should stay with the USB port and not affect the COM port.

Screenshots
None. Not yet tested on hardware.

Hardware description
Model: Krake (GPAD_API firmware), checked against main at commit b6ea4b3 (2026-09-20)
Serial Number: N/A (found by reading the code)
Other: Affects any Krake with both the USB serial and the COM port connected.

Test Equipment
Model: N/A (not yet tested)
Serial Number: N/A

Additional context
Where in the code:

Why it happens: processSerial() keeps a half-finished line in the buffer between calls, but there is only one buffer and one counter for both ports. Nothing records which port the half-finished line came from. It is not two tasks running at once: everything runs in loop(), one call after the other.

Side effect: if one port sends a line that is too long, the buffer is cleared. That also throws away the other port's half-finished command and cancels pending alarm audio (gpad_serial.cpp lines 107–113).

How likely: low to medium. Most setups use one port at a time, but it is easy to hit while debugging with a serial monitor open.

Possible fix: give each port its own buffer and its own character counter, passed into processSerial(). The comment at gpad_serial.cpp lines 45–46 already suggests making the buffer a parameter.

Related: #283 (shared global variables).

主要言語
C++
スター
9
フォーク
6
30日以内にマージされた PR はありません

環境構築

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

PubInv/krake のほかの issue

PubInv/krake の issue をすべて見る

似ている issue

C++ の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。