From c8d078f9c7a82d3f0b5406c5ac96fbe76de6d740 Mon Sep 17 00:00:00 2001 From: Leah Rowe Date: Thu, 12 Mar 2026 21:33:46 +0000 Subject: util/spkmodem-recv: reset char precisely on timeout instead of when it goes above, do it precisely on the timeout. otherwise, if by sheer chance the signal pauses and we reset the byte - sure, ok, but it's a bit tight and we run the risk of advancing another frame, depending on the timing. this is a minor edge case, probably rarely ever triggered in practise. Signed-off-by: Leah Rowe --- util/spkmodem_decode/spkmodem-decode.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'util') diff --git a/util/spkmodem_decode/spkmodem-decode.c b/util/spkmodem_decode/spkmodem-decode.c index 820fa61e..9ba060d7 100644 --- a/util/spkmodem_decode/spkmodem-decode.c +++ b/util/spkmodem_decode/spkmodem-decode.c @@ -259,7 +259,7 @@ handle_audio(struct decoder_state *st) * If the modem signal disappears for several frames, * discard the partially assembled character. */ - if (st->sample_count > (3 * SAMPLES_PER_FRAME)) + if (st->sample_count >= (3 * SAMPLES_PER_FRAME)) reset_char(st); if (!valid_signal(st)) { decode_pulse(st); -- cgit v1.2.1