diff options
| author | Daniel Starke <daniel.starke@siemens.com> | 2022-04-14 02:42:12 -0700 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2022-05-12 12:17:08 +0200 |
| commit | da0fc1a6f2ea9068568b97d37d7faea3c8c65078 (patch) | |
| tree | cebb08c137a817d7aedde7f4431c3db0ca7362f7 | |
| parent | e5afac8d44e8d8a3dc107f40d786ecb5cdbb5d73 (diff) | |
| download | linux-da0fc1a6f2ea9068568b97d37d7faea3c8c65078.tar.gz linux-da0fc1a6f2ea9068568b97d37d7faea3c8c65078.tar.bz2 linux-da0fc1a6f2ea9068568b97d37d7faea3c8c65078.zip | |
tty: n_gsm: fix malformed counter for out of frame data
commit a24b4b2f660b7ddf3f484b37600bba382cb28a9d upstream.
The gsm_mux field 'malformed' represents the number of malformed frames
received. However, gsm1_receive() also increases this counter for any out
of frame byte.
Fix this by ignoring out of frame data for the malformed counter.
Fixes: e1eaea46bb40 ("tty: n_gsm line discipline")
Cc: stable@vger.kernel.org
Signed-off-by: Daniel Starke <daniel.starke@siemens.com>
Link: https://lore.kernel.org/r/20220414094225.4527-7-daniel.starke@siemens.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| -rw-r--r-- | drivers/tty/n_gsm.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/tty/n_gsm.c b/drivers/tty/n_gsm.c index bee92cd3f90a..cbeb8907a6cf 100644 --- a/drivers/tty/n_gsm.c +++ b/drivers/tty/n_gsm.c @@ -1973,7 +1973,8 @@ static void gsm1_receive(struct gsm_mux *gsm, unsigned char c) } /* Any partial frame was a runt so go back to start */ if (gsm->state != GSM_START) { - gsm->malformed++; + if (gsm->state != GSM_SEARCH) + gsm->malformed++; gsm->state = GSM_START; } /* A SOF in GSM_START means we are still reading idling or |