Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Endlosschleife bei Sendcommand #267

Open
Devirex opened this issue Jan 30, 2023 · 1 comment · May be fixed by #268
Open

Endlosschleife bei Sendcommand #267

Devirex opened this issue Jan 30, 2023 · 1 comment · May be fixed by #268
Assignees

Comments

@Devirex
Copy link
Contributor

Devirex commented Jan 30, 2023

Hallo Zusammen,

ich habe einen Bug verfolgt der bei mir des öfteren aufgetreten ist. Bei der Übermittlung eines Sendcommands über Telnet von FHEM and den Signalduino, schleichen sich merkwürdige Zeichen ein:
SR;R=3,.......
� SR;R=3,.......
SR;R=3.......

Die signalesp.h erkennt ein Sendcommand anhand des ";". Die send.h überprüft ob das Command mit S anfängt. Normalerweise korrekt, aber bei dem zweiten command mit Artefakt nicht. Der Buffer springt am Ende der Loop einen Schritt weiter. die Repeats werden auf 255 gesetzt und das ganze landet in einer Endlosschleife.

Klar normalerweise sollten keine derartigen SendCommands ankommen, aber dennoch ist es denke ich nicht schlecht eine Routine einzubauen die den msg_beginptr so lange verschiebt bis ein S kommt solange noch nie eines aufgetreten ist und andernfalls ein MSG_PRINTLN(FPSTR(TXT_CORRUPT)); zu senden.

So kann das ganze reproduziert werden:
set (sduino) raw %SR;R=3;P0=500;P1=-9000;P2=-4000;P3=-2000;D=0302030;

@Devirex
Copy link
Contributor Author

Devirex commented Jan 30, 2023

Lösungvorschlag: #268

@sidey79 sidey79 linked a pull request Jan 30, 2023 that will close this issue
@sidey79 sidey79 self-assigned this Mar 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants