Re: [PATCH] serial: 8250_pci1xxxx: off by one in pci1xxxx_process_read_data()

From: Dan Carpenter
Date: Wed Jan 10 2024 - 14:20:08 EST


On Wed, Jan 10, 2024 at 02:11:46PM -0500, Hugo Villeneuve wrote:
> On Wed, 10 Jan 2024 21:52:28 +0300
> Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote:
>
> > These > comparisons should be >= to prevent writing one element beyond
> > the end of the rx_buff[] array. The buffer has RX_BUF_SIZE[] elements.
>
> Hi,
> your commit title message is very confusing and doesn't hint that this
> is a bug fix (or a potential bug fix)...
>

Most C programmers know what an "off by one" is... But sure I can add
"fix" to the subject. I debated either way, but left it off because the
subject was already too long.

regards,
dan carpenter